Merge pull request swiftlang#245 from allevato/fix-linuxmain

Update LinuxMain and the generated pipeline.

Author: allevato

Sources/SwiftFormat/Pipelines+Generated.swift

@@ -250,7 +250,6 @@ class LintPipeline: SyntaxVisitor {
 
   override func visit(_ node: SwitchCaseLabelSyntax) -> SyntaxVisitorContinueKind {
     visitIfEnabled(NoLabelsInCasePatterns.visit, for: node)
-    visitIfEnabled(UseLetInEveryBoundCaseVariable.visit, for: node)
     return .visitChildren
   }
 
@@ -282,6 +281,11 @@ class LintPipeline: SyntaxVisitor {
     return .visitChildren
   }
 
+  override func visit(_ node: ValueBindingPatternSyntax) -> SyntaxVisitorContinueKind {
+    visitIfEnabled(UseLetInEveryBoundCaseVariable.visit, for: node)
+    return .visitChildren
+  }
+
   override func visit(_ node: VariableDeclSyntax) -> SyntaxVisitorContinueKind {
     visitIfEnabled(AllPublicDeclarationsHaveDocumentation.visit, for: node)
     visitIfEnabled(AlwaysUseLowerCamelCase.visit, for: node)

Tests/SwiftFormatPrettyPrintTests/XCTestManifests.swift

@@ -164,6 +164,7 @@ extension CommentTests {
         ("testDoesNotInsertExtraNewlinesAfterTrailingComments", testDoesNotInsertExtraNewlinesAfterTrailingComments),
         ("testLineCommentAtEndOfMemberDeclList", testLineCommentAtEndOfMemberDeclList),
         ("testLineComments", testLineComments),
+        ("testTrailingSpacesInComments", testTrailingSpacesInComments),
     ]
 }
 
@@ -302,6 +303,7 @@ extension FunctionCallTests {
         ("testDiscretionaryLineBreakBeforeTrailingClosure", testDiscretionaryLineBreakBeforeTrailingClosure),
         ("testDiscretionaryLineBreaksAreSelfCorrecting", testDiscretionaryLineBreaksAreSelfCorrecting),
         ("testGroupsTrailingComma", testGroupsTrailingComma),
+        ("testMultipleTrailingClosures", testMultipleTrailingClosures),
         ("testNestedFunctionCallExprSequences", testNestedFunctionCallExprSequences),
         ("testSingleUnlabeledArgumentWithDelimiters", testSingleUnlabeledArgumentWithDelimiters),
     ]
@@ -728,6 +730,8 @@ extension SwitchStmtTests {
     // to regenerate.
     static let __allTests__SwitchStmtTests = [
         ("testBasicSwitch", testBasicSwitch),
+        ("testConditionalCases", testConditionalCases),
+        ("testConditionalCasesIndenting", testConditionalCasesIndenting),
         ("testLabeledSwitchStmt", testLabeledSwitchStmt),
         ("testNestedSwitch", testNestedSwitch),
         ("testNewlinesDisambiguatingWhereClauses", testNewlinesDisambiguatingWhereClauses),
@@ -765,6 +769,7 @@ extension TryCatchTests {
         ("testDoTryCatch_noBreakBeforeCatch", testDoTryCatch_noBreakBeforeCatch),
         ("testMultipleCatchItems", testMultipleCatchItems),
         ("testNestedDo", testNestedDo),
+        ("testTryKeywordBreaking", testTryKeywordBreaking),
     ]
 }
 

Tests/SwiftFormatRulesTests/XCTestManifests.swift

@@ -15,6 +15,7 @@ extension AlwaysUseLowerCamelCaseTests {
     //   `swift test --generate-linuxmain`
     // to regenerate.
     static let __allTests__AlwaysUseLowerCamelCaseTests = [
+        ("testIgnoresUnderscoresInTestNames", testIgnoresUnderscoresInTestNames),
         ("testInvalidVariableCasing", testInvalidVariableCasing),
     ]
 }
@@ -47,6 +48,7 @@ extension DoNotUseSemicolonsTests {
         ("testNewlinesAfterSemicolons", testNewlinesAfterSemicolons),
         ("testSemicolonsInMemberLists", testSemicolonsInMemberLists),
         ("testSemicolonsInNestedStatements", testSemicolonsInNestedStatements),
+        ("testSemicolonsSeparatingDoWhile", testSemicolonsSeparatingDoWhile),
         ("testSemicolonUse", testSemicolonUse),
     ]
 }
@@ -216,6 +218,8 @@ extension NoParensAroundConditionsTests {
     // to regenerate.
     static let __allTests__NoParensAroundConditionsTests = [
         ("testParensAroundConditions", testParensAroundConditions),
+        ("testParensAroundNestedParenthesizedStatements", testParensAroundNestedParenthesizedStatements),
+        ("testParensAroundNestedUnparenthesizedStatements", testParensAroundNestedUnparenthesizedStatements),
     ]
 }
 
@@ -245,6 +249,11 @@ extension OneVariableDeclarationPerLineTests {
     //   `swift test --generate-linuxmain`
     // to regenerate.
     static let __allTests__OneVariableDeclarationPerLineTests = [
+        ("testCommentPrecedingDeclIsNotRepeated", testCommentPrecedingDeclIsNotRepeated),
+        ("testCommentsPrecedingBindingsAreKept", testCommentsPrecedingBindingsAreKept),
+        ("testInvalidBindingsAreNotDestroyed", testInvalidBindingsAreNotDestroyed),
+        ("testMixedInitializedAndTypedBindings", testMixedInitializedAndTypedBindings),
+        ("testMultipleBindingsWithAccessorsAreCorrected", testMultipleBindingsWithAccessorsAreCorrected),
         ("testMultipleVariableBindings", testMultipleVariableBindings),
         ("testNestedVariableBindings", testNestedVariableBindings),
     ]
@@ -316,7 +325,11 @@ extension UseLetInEveryBoundCaseVariableTests {
     //   `swift test --generate-linuxmain`
     // to regenerate.
     static let __allTests__UseLetInEveryBoundCaseVariableTests = [
-        ("testInvalidLetBoundCase", testInvalidLetBoundCase),
+        ("testForCase", testForCase),
+        ("testGuardCase", testGuardCase),
+        ("testIfCase", testIfCase),
+        ("testSwitchCase", testSwitchCase),
+        ("testWhileCase", testWhileCase),
     ]
 }
 
@@ -398,9 +411,9 @@ extension ValidateDocumentationCommentsTests {
         ("testInitializer", testInitializer),
         ("testParameterDocumentation", testParameterDocumentation),
         ("testParametersName", testParametersName),
-        ("testThrowsDocumentation", testThrowsDocumentation),
         ("testReturnDocumentation", testReturnDocumentation),
         ("testSeparateLabelAndIdentifier", testSeparateLabelAndIdentifier),
+        ("testThrowsDocumentation", testThrowsDocumentation),
         ("testValidDocumentation", testValidDocumentation),
     ]
 }