Remove @testable annotations where possible

Author: natecook1000

Tests/RegexBuilderTests/RegexDSLTests.swift

@@ -11,7 +11,7 @@
 
 import XCTest
 import _StringProcessing
-@testable import RegexBuilder
+import RegexBuilder
 
 class RegexDSLTests: XCTestCase {
   func _testDSLCaptures<Content: RegexComponent, MatchType>(

Tests/RegexTests/AlgorithmsInternalsTests.swift

@@ -0,0 +1,47 @@
+//===----------------------------------------------------------------------===//
+//
+// This source file is part of the Swift.org open source project
+//
+// Copyright (c) 2021-2022 Apple Inc. and the Swift project authors
+// Licensed under Apache License v2.0 with Runtime Library Exception
+//
+// See https://swift.org/LICENSE.txt for license information
+//
+//===----------------------------------------------------------------------===//
+
+@testable import _StringProcessing
+import XCTest
+
+// TODO: Protocol-powered testing
+extension AlgorithmTests {
+  func testAdHoc() {
+    let r = try! Regex("a|b+")
+
+    XCTAssert("palindrome".contains(r))
+    XCTAssert("botany".contains(r))
+    XCTAssert("antiquing".contains(r))
+    XCTAssertFalse("cdef".contains(r))
+
+    let str = "a string with the letter b in it"
+    let first = str.firstRange(of: r)
+    let last = str.lastRange(of: r)
+    let (expectFirst, expectLast) = (
+      str.index(atOffset: 0)..<str.index(atOffset: 1),
+      str.index(atOffset: 25)..<str.index(atOffset: 26))
+    output(str.split(around: first!))
+    output(str.split(around: last!))
+
+    XCTAssertEqual(expectFirst, first)
+    XCTAssertEqual(expectLast, last)
+
+    XCTAssertEqual(
+      [expectFirst, expectLast], Array(str.ranges(of: r)))
+
+    XCTAssertTrue(str.starts(with: r))
+    XCTAssertFalse(str.ends(with: r))
+
+    XCTAssertEqual(str.dropFirst(), str.trimmingPrefix(r))
+    XCTAssertEqual("x", "axb".trimming(r))
+    XCTAssertEqual("x", "axbb".trimming(r))
+  }
+}

Tests/RegexTests/AlgorithmsTests.swift

@@ -9,7 +9,7 @@
 //
 //===----------------------------------------------------------------------===//
 
-@testable import _StringProcessing
+import _StringProcessing
 import XCTest
 
 // TODO: Protocol-powered testing
@@ -107,37 +107,6 @@ class RegexConsumerTests: XCTestCase {
     expectReplace("aab", "a*", "X", "XXbX")
   }
 
-  func testAdHoc() {
-    let r = try! Regex("a|b+")
-
-    XCTAssert("palindrome".contains(r))
-    XCTAssert("botany".contains(r))
-    XCTAssert("antiquing".contains(r))
-    XCTAssertFalse("cdef".contains(r))
-
-    let str = "a string with the letter b in it"
-    let first = str.firstRange(of: r)
-    let last = str.lastRange(of: r)
-    let (expectFirst, expectLast) = (
-      str.index(atOffset: 0)..<str.index(atOffset: 1),
-      str.index(atOffset: 25)..<str.index(atOffset: 26))
-    output(str.split(around: first!))
-    output(str.split(around: last!))
-
-    XCTAssertEqual(expectFirst, first)
-    XCTAssertEqual(expectLast, last)
-
-    XCTAssertEqual(
-      [expectFirst, expectLast], Array(str.ranges(of: r)))
-
-    XCTAssertTrue(str.starts(with: r))
-    XCTAssertFalse(str.ends(with: r))
-
-    XCTAssertEqual(str.dropFirst(), str.trimmingPrefix(r))
-    XCTAssertEqual("x", "axb".trimming(r))
-    XCTAssertEqual("x", "axbb".trimming(r))
-  }
-  
   func testSubstring() throws {
     let s = "aaa | aaaaaa | aaaaaaaaaa"
     let s1 = s.dropFirst(6)  // "aaaaaa | aaaaaaaaaa"

Tests/RegexTests/AllScalarsTests.swift

@@ -10,7 +10,7 @@
 //===----------------------------------------------------------------------===//
 
 import XCTest
-@testable import _StringProcessing
+import _StringProcessing
 
 let allScalars = Unicode.Scalar.allScalars
 

Tests/RegexTests/CompileTests.swift

@@ -9,7 +9,6 @@
 //
 //===----------------------------------------------------------------------===//
 
-
 @testable import _RegexParser
 @testable import _StringProcessing