Merge pull request #1200 from ahoppen/skip-hover-tests-on-old-toolchains

ahoppen · web-flow · commit 30a3d6997d76 · 2024-04-23T20:34:13.000-07:00
Skip a few hover tests if sourcekitd doesn’t return raw documentation in cursor info
diff --git a/Tests/SourceKitLSPTests/HoverTests.swift b/Tests/SourceKitLSPTests/HoverTests.swift
@@ -87,6 +87,7 @@ final class HoverTests: XCTestCase {
   }
 
   func testMultiCursorInfoResultsHoverWithDocumentation() async throws {
+    try await SkipUnless.sourcekitdReturnsRawDocumentationResponse()
     try await assertHover(
       """
       /// A struct
@@ -118,6 +119,7 @@ final class HoverTests: XCTestCase {
   }
 
   func testHoverNameEscapingOnFunction() async throws {
+    try await SkipUnless.sourcekitdReturnsRawDocumentationResponse()
     try await assertHover(
       """
       /// this is **bold** documentation
@@ -134,6 +136,7 @@ final class HoverTests: XCTestCase {
   }
 
   func testHoverNameEscapingOnOperator() async throws {
+    try await SkipUnless.sourcekitdReturnsRawDocumentationResponse()
     try await assertHover(
       """
       /// this is *italic* documentation

Original file line number	Diff line number	Diff line change
`@@ -87,6 +87,7 @@ final class HoverTests: XCTestCase {`
`87`	`87`	`}`
`88`	`88`
`89`	`89`	`func testMultiCursorInfoResultsHoverWithDocumentation() async throws {`
	`90`	`+ try await SkipUnless.sourcekitdReturnsRawDocumentationResponse()`
`90`	`91`	`try await assertHover(`
`91`	`92`	`"""`
`92`	`93`	`/// A struct`
`@@ -118,6 +119,7 @@ final class HoverTests: XCTestCase {`
`118`	`119`	`}`
`119`	`120`
`120`	`121`	`func testHoverNameEscapingOnFunction() async throws {`
	`122`	`+ try await SkipUnless.sourcekitdReturnsRawDocumentationResponse()`
`121`	`123`	`try await assertHover(`
`122`	`124`	`"""`
`123`	`125`	`/// this is bold documentation`
`@@ -134,6 +136,7 @@ final class HoverTests: XCTestCase {`
`134`	`136`	`}`
`135`	`137`
`136`	`138`	`func testHoverNameEscapingOnOperator() async throws {`
	`139`	`+ try await SkipUnless.sourcekitdReturnsRawDocumentationResponse()`
`137`	`140`	`try await assertHover(`
`138`	`141`	`"""`
`139`	`142`	`/// this is italic documentation`