Added missing fix its

Author: kimdv

Tests/SwiftParserTest/Assertions.swift

@@ -434,7 +434,18 @@ func assertDiagnostic<T: SyntaxProtocol>(
       }
     }
   }
-  if let fixIts = spec.fixIts {
+
+  if diag.fixIts.count != (spec.fixIts?.count ?? 0) {
+    XCTFail(
+      """
+      Expected \(spec.fixIts?.count ?? 0) fix its but received \(diag.fixIts.count):
+      \(diag.fixIts.map { $0.message.message }.joined(separator: "\n"))
+      """,
+      file: file,
+      line: line
+    )
+  } else if let fixIts = spec.fixIts {
+    XCTAssertEqual(diag.fixIts.map { $0.message.message }, spec.fixIts ?? [], file: file, line: line)
     if fixIts != diag.fixIts.map(\.message.message) {
       failStringsEqualWithDiff(
         diag.fixIts.map(\.message.message).joined(separator: "\n"),

Tests/SwiftParserTest/AttributeTests.swift

@@ -60,9 +60,9 @@ final class AttributeTests: XCTestCase {
       @_specialize(e1️⃣
       """,
       diagnostics: [
-        DiagnosticSpec(message: "expected ':' in attribute argument"),
-        DiagnosticSpec(message: "expected ')' to end attribute"),
-        DiagnosticSpec(message: "expected declaration after attribute"),
+        DiagnosticSpec(message: "expected ':' in attribute argument", fixIts: ["insert ':'"]),
+        DiagnosticSpec(message: "expected ')' to end attribute", fixIts: ["insert ')'"]),
+        DiagnosticSpec(message: "expected declaration after attribute", fixIts: ["insert declaration"]),
       ]
     )
   }
@@ -73,9 +73,9 @@ final class AttributeTests: XCTestCase {
       @_specialize(e1️⃣, exported2️⃣)3️⃣
       """,
       diagnostics: [
-        DiagnosticSpec(locationMarker: "1️⃣", message: "expected ':' in attribute argument"),
-        DiagnosticSpec(locationMarker: "2️⃣", message: "expected ': false' in attribute argument"),
-        DiagnosticSpec(locationMarker: "3️⃣", message: "expected declaration after attribute"),
+        DiagnosticSpec(locationMarker: "1️⃣", message: "expected ':' in attribute argument", fixIts: ["insert ':'"]),
+        DiagnosticSpec(locationMarker: "2️⃣", message: "expected ': false' in attribute argument", fixIts: ["insert ': false'"]),
+        DiagnosticSpec(locationMarker: "3️⃣", message: "expected declaration after attribute", fixIts: ["insert declaration"]),
       ]
     )
   }
@@ -309,7 +309,7 @@ final class AttributeTests: XCTestCase {
   func testMissingDeclarationAfterAttributes() {
     assertParse(
       "@resultBuilder1️⃣",
-      diagnostics: [DiagnosticSpec(message: "expected declaration after attribute")],
+      diagnostics: [DiagnosticSpec(message: "expected declaration after attribute", fixIts: ["insert declaration"])],
       fixedSource: """
         @resultBuilder
         <#declaration#>
@@ -426,7 +426,7 @@ final class AttributeTests: XCTestCase {
       @_expose(Cxx, 1️⃣baz) func foo() {}
       """,
       diagnostics: [
-        DiagnosticSpec(message: "expected string literal to end @_expose arguments"),
+        DiagnosticSpec(message: "expected string literal to end @_expose arguments", fixIts: ["insert string literal"]),
         DiagnosticSpec(message: "unexpected code 'baz' in attribute"),
       ]
     )
@@ -476,7 +476,7 @@ final class AttributeTests: XCTestCase {
       func foo() {}
       """,
       diagnostics: [
-        DiagnosticSpec(message: "expected 'message' in @_unavailableFromAsync argument"),
+        DiagnosticSpec(message: "expected 'message' in @_unavailableFromAsync argument", fixIts: ["insert 'message'"]),
         DiagnosticSpec(message: "unexpected code 'nope' before @_unavailableFromAsync argument"),
       ]
     )
@@ -487,7 +487,7 @@ final class AttributeTests: XCTestCase {
       func foo() {}
       """,
       diagnostics: [
-        DiagnosticSpec(message: "expected ':' and string literal in @_unavailableFromAsync argument"),
+        DiagnosticSpec(message: "expected ':' and string literal in @_unavailableFromAsync argument", fixIts: ["insert ':' and string literal"]),
         DiagnosticSpec(message: #"unexpected code '= "abc"' in attribute"#),
       ]
     )
@@ -498,7 +498,7 @@ final class AttributeTests: XCTestCase {
       func foo() {}
       """,
       diagnostics: [
-        DiagnosticSpec(message: "expected string literal to end @_unavailableFromAsync argument"),
+        DiagnosticSpec(message: "expected string literal to end @_unavailableFromAsync argument", fixIts: ["insert string literal"]),
         DiagnosticSpec(message: "unexpected code 'abc' in attribute"),
       ]
     )

Tests/SwiftParserTest/AvailabilityTests.swift

@@ -149,7 +149,7 @@ final class AvailabilityTests: XCTestCase {
       func test() {}
       """,
       diagnostics: [
-        DiagnosticSpec(message: "expected version tuple in version restriction"),
+        DiagnosticSpec(message: "expected version tuple in version restriction", fixIts: ["insert version tuple"]),
         DiagnosticSpec(message: "unexpected code '10e10' in attribute"),
       ]
     )
@@ -160,7 +160,7 @@ final class AvailabilityTests: XCTestCase {
       func test() {}
       """,
       diagnostics: [
-        DiagnosticSpec(message: "expected integer literal in version tuple"),
+        DiagnosticSpec(message: "expected integer literal in version tuple", fixIts: ["insert integer literal"]),
         DiagnosticSpec(message: "unexpected code '0e10' in attribute"),
       ]
     )
@@ -171,7 +171,7 @@ final class AvailabilityTests: XCTestCase {
       func test() {}
       """,
       diagnostics: [
-        DiagnosticSpec(message: "expected version tuple in version restriction"),
+        DiagnosticSpec(message: "expected version tuple in version restriction", fixIts: ["insert version tuple"]),
         DiagnosticSpec(message: "unexpected code '0xff' in attribute"),
       ]
     )
@@ -182,7 +182,7 @@ final class AvailabilityTests: XCTestCase {
       func test() {}
       """,
       diagnostics: [
-        DiagnosticSpec(message: "expected integer literal in version tuple"),
+        DiagnosticSpec(message: "expected integer literal in version tuple", fixIts: ["insert integer literal"]),
         DiagnosticSpec(message: "unexpected code '0xff' in attribute"),
       ]
     )