Skip to content

Commit c6faab7

Browse files
authored
Reenable disabled tests (#5957)
* Revert "Pass `SWIFT_EXEC` to swift-test via bootstrap (#4213)" * Revert "Disable `FunctionalTests` temporarily (#5875)" * Reenable disabled tests These tests were disabled but proven working again. rdar://101868275
1 parent b268125 commit c6faab7

File tree

6 files changed

+6
-13
lines changed

6 files changed

+6
-13
lines changed

Package.swift

+2-3
Original file line numberDiff line numberDiff line change
@@ -507,8 +507,7 @@ let package = Package(
507507
name: "WorkspaceTests",
508508
dependencies: ["Workspace", "SPMTestSupport"]
509509
),
510-
// rdar://101868275 "error: cannot find 'XCTAssertEqual' in scope" can affect almost any functional test, so we flat out disable them all until we know what is going on
511-
/*.testTarget(
510+
.testTarget(
512511
name: "FunctionalTests",
513512
dependencies: [
514513
"swift-build",
@@ -517,7 +516,7 @@ let package = Package(
517516
"PackageModel",
518517
"SPMTestSupport"
519518
]
520-
),*/
519+
),
521520
.testTarget(
522521
name: "FunctionalPerformanceTests",
523522
dependencies: [

Tests/CommandsTests/BuildToolTests.swift

+1-1
Original file line numberDiff line numberDiff line change
@@ -280,7 +280,7 @@ final class BuildToolTests: CommandsTestCase {
280280
}
281281

282282
func testBuildCompleteMessage() throws {
283-
throw XCTSkip("This test fails to match the 'Compiling' regex; rdar://101815761")
283+
throw XCTSkip("This test isn't stable w.r.t. the build cache; rdar://101815761")
284284

285285
try fixture(name: "DependencyResolution/Internal/Simple") { fixturePath in
286286
do {

Tests/FunctionalTests/PluginTests.swift

+2
Original file line numberDiff line numberDiff line change
@@ -461,6 +461,7 @@ class PluginTests: XCTestCase {
461461
outputDirectory: pluginDir.appending(component: "output"),
462462
toolSearchDirectories: toolSearchDirectories,
463463
toolNamesToPaths: [:],
464+
toolNamesToTriples: [:],
464465
writableDirectories: [pluginDir.appending(component: "output")],
465466
readOnlyDirectories: [package.path],
466467
fileSystem: localFileSystem,
@@ -722,6 +723,7 @@ class PluginTests: XCTestCase {
722723
outputDirectory: pluginDir.appending(component: "output"),
723724
toolSearchDirectories: [try UserToolchain.default.swiftCompilerPath.parentDirectory],
724725
toolNamesToPaths: [:],
726+
toolNamesToTriples: [:],
725727
writableDirectories: [pluginDir.appending(component: "output")],
726728
readOnlyDirectories: [package.path],
727729
fileSystem: localFileSystem,

Tests/WorkspaceTests/InitTests.swift

-2
Original file line numberDiff line numberDiff line change
@@ -266,8 +266,6 @@ class InitTests: XCTestCase {
266266
}
267267

268268
func testNonC99NameExecutablePackage() throws {
269-
throw XCTSkip("This test fails to find XCTAssertEqual; rdar://101868275")
270-
271269
try withTemporaryDirectory(removeTreeOnDeinit: true) { tempDirPath in
272270
XCTAssertDirectoryExists(tempDirPath)
273271

Tests/WorkspaceTests/WorkspaceTests.swift

-2
Original file line numberDiff line numberDiff line change
@@ -4291,8 +4291,6 @@ final class WorkspaceTests: XCTestCase {
42914291

42924292
// This verifies that the simplest possible loading APIs are available for package clients.
42934293
func testSimpleAPI() throws {
4294-
throw XCTSkip("This test fails to find XCTAssertEqual; rdar://101868275")
4295-
42964294
try testWithTemporaryDirectory { path in
42974295
// Create a temporary package as a test case.
42984296
let packagePath = path.appending(component: "MyPkg")

Utilities/bootstrap

+1-5
Original file line numberDiff line numberDiff line change
@@ -361,11 +361,7 @@ def test(args):
361361

362362
note("Testing")
363363
parse_test_args(args)
364-
cmd = [
365-
"SWIFT_EXEC=" + args.swiftc_path,
366-
"SWIFT_DRIVER_SWIFT_EXEC=" + args.swiftc_path,
367-
os.path.join(args.bin_dir, "swift-test")
368-
]
364+
cmd = [os.path.join(args.bin_dir, "swift-test")]
369365
if args.parallel:
370366
cmd.append("--parallel")
371367
for arg in args.filter:

0 commit comments

Comments
 (0)