Merge pull request #2114 from DougGregor/educational-notes-diags-generalization-6.2

DougGregor · web-flow · commit 0310a3be27eb · 2025-04-16T08:59:36.000-07:00
[6.2] Generalize educational notes documentation test to not require the ".md" suffix
diff --git a/Tests/SourceKitLSPTests/LocalSwiftTests.swift b/Tests/SourceKitLSPTests/LocalSwiftTests.swift
@@ -379,7 +379,8 @@ final class LocalSwiftTests: XCTestCase {
     XCTAssertEqual(diags.diagnostics.count, 1)
     let diag = diags.diagnostics.first!
     XCTAssertEqual(diag.code, .string("property-wrapper-requirements"))
-    XCTAssertEqual(diag.codeDescription?.href.fileURL?.lastPathComponent, "property-wrapper-requirements.md")
+    let filename = try XCTUnwrap(diag.codeDescription?.href.fileURL?.lastPathComponent)
+    XCTAssert(filename.starts(with: "property-wrapper-requirements"))
   }
 
   func testFixitsAreIncludedInPublishDiagnostics() async throws {

Original file line number	Diff line number	Diff line change
`@@ -379,7 +379,8 @@ final class LocalSwiftTests: XCTestCase {`
`379`	`379`	`XCTAssertEqual(diags.diagnostics.count, 1)`
`380`	`380`	`let diag = diags.diagnostics.first!`
`381`	`381`	`XCTAssertEqual(diag.code, .string("property-wrapper-requirements"))`
`382`		`- XCTAssertEqual(diag.codeDescription?.href.fileURL?.lastPathComponent, "property-wrapper-requirements.md")`
	`382`	`+ let filename = try XCTUnwrap(diag.codeDescription?.href.fileURL?.lastPathComponent)`
	`383`	`+ XCTAssert(filename.starts(with: "property-wrapper-requirements"))`
`383`	`384`	`}`
`384`	`385`
`385`	`386`	`func testFixitsAreIncludedInPublishDiagnostics() async throws {`