Merge pull request #800 from bnbarham/add-test-message

bnbarham · web-flow · commit c0f14692afd2 · 2023-08-17T18:16:50.000-07:00
Add a message to some test assertions
diff --git a/Tests/SourceKitLSPTests/SwiftInterfaceTests.swift b/Tests/SourceKitLSPTests/SwiftInterfaceTests.swift
@@ -125,12 +125,12 @@ final class SwiftInterfaceTests: XCTestCase {
       return
     }
     let location = try XCTUnwrap(jump.first)
-    XCTAssertTrue(location.uri.pseudoPath.hasSuffix(swiftInterfaceFile))
+    XCTAssertTrue(location.uri.pseudoPath.hasSuffix(swiftInterfaceFile), "Path was: '\(location.uri.pseudoPath)'")
     // load contents of swiftinterface
     let contents = try XCTUnwrap(location.uri.fileURL.flatMap({ try String(contentsOf: $0, encoding: .utf8) }))
     let lineTable = LineTable(contents)
     let line = lineTable[location.range.lowerBound.line]
-    XCTAssert(line.hasPrefix(linePrefix))
+    XCTAssert(line.hasPrefix(linePrefix), "Full line was: '\(line)'")
     ws.closeDocument(testLoc.url)
   }
 

Original file line number	Diff line number	Diff line change
`@@ -125,12 +125,12 @@ final class SwiftInterfaceTests: XCTestCase {`
`125`	`125`	`return`
`126`	`126`	`}`
`127`	`127`	`let location = try XCTUnwrap(jump.first)`
`128`		`- XCTAssertTrue(location.uri.pseudoPath.hasSuffix(swiftInterfaceFile))`
	`128`	`+ XCTAssertTrue(location.uri.pseudoPath.hasSuffix(swiftInterfaceFile), "Path was: '\(location.uri.pseudoPath)'")`
`129`	`129`	`// load contents of swiftinterface`
`130`	`130`	`let contents = try XCTUnwrap(location.uri.fileURL.flatMap({ try String(contentsOf: $0, encoding: .utf8) }))`
`131`	`131`	`let lineTable = LineTable(contents)`
`132`	`132`	`let line = lineTable[location.range.lowerBound.line]`
`133`		`- XCTAssert(line.hasPrefix(linePrefix))`
	`133`	`+ XCTAssert(line.hasPrefix(linePrefix), "Full line was: '\(line)'")`
`134`	`134`	`ws.closeDocument(testLoc.url)`
`135`	`135`	`}`
`136`	`136`