Merge pull request #205 from dylansturg/fix_testable

Remove @testable imports, upgrading symbols to public visibility.

Author: allevato

Sources/SwiftFormat/SwiftFormatter.swift

@@ -101,7 +101,7 @@ public final class SwiftFormatter {
     syntax: SourceFileSyntax, assumingFileURL url: URL?, source: String?,
     to outputStream: inout Output
   ) throws {
-    if let position = firstInvalidSyntaxPosition(in: Syntax(syntax)) {
+    if let position = _firstInvalidSyntaxPosition(in: Syntax(syntax)) {
       throw SwiftFormatError.fileContainsInvalidSyntax(position: position)
     }
 

Sources/SwiftFormat/SwiftLinter.swift

@@ -82,7 +82,7 @@ public final class SwiftLinter {
   }
 
   private func lint(syntax: SourceFileSyntax, assumingFileURL url: URL, source: String?) throws {
-    if let position = firstInvalidSyntaxPosition(in: Syntax(syntax)) {
+    if let position = _firstInvalidSyntaxPosition(in: Syntax(syntax)) {
       throw SwiftFormatError.fileContainsInvalidSyntax(position: position)
     }
 

Sources/SwiftFormat/SyntaxValidatingVisitor.swift

@@ -68,7 +68,7 @@ fileprivate class SyntaxValidatingVisitor: SyntaxVisitor {
 /// syntax is valid.
 ///
 /// - Parameter syntax: The root of a tree of syntax nodes to check for compatibility.
-func firstInvalidSyntaxPosition(in syntax: Syntax) -> AbsolutePosition? {
+public func _firstInvalidSyntaxPosition(in syntax: Syntax) -> AbsolutePosition? {
   let visitor = SyntaxValidatingVisitor()
   visitor.walk(syntax)
   return visitor.invalidSyntaxStartPosition

Tests/SwiftFormatTests/SyntaxValidatingVisitorTests.swift

@@ -1,8 +1,7 @@
+import SwiftFormat
 import SwiftSyntax
 import XCTest
 
-@testable import SwiftFormat
-
 final class SyntaxValidatingVisitorTests: XCTestCase {
   func testValidSyntax() {
     let input =
@@ -18,7 +17,7 @@ final class SyntaxValidatingVisitorTests: XCTestCase {
       @unknown default: break
       }
       """
-    XCTAssertNil(firstInvalidSyntaxPosition(in: createSyntax(from: input)))
+    XCTAssertNil(_firstInvalidSyntaxPosition(in: createSyntax(from: input)))
   }
 
   func testInvalidSyntax() {
@@ -56,7 +55,7 @@ final class SyntaxValidatingVisitorTests: XCTestCase {
   private func assertInvalidSyntax(
     in source: String, atLine: Int, column: Int, file: StaticString = #file, line: UInt = #line
   ) {
-    guard let position = firstInvalidSyntaxPosition(in: createSyntax(from: source)) else {
+    guard let position = _firstInvalidSyntaxPosition(in: createSyntax(from: source)) else {
       XCTFail("No invalid syntax was found", file: file, line: line)
       return
     }