Reword the skip message for the abstract test classes

swiftlang · Mar 10, 2025 · 153eb90 · 153eb90
1 parent ea9f97f
commit 153eb90
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 7 deletions.
diff --git a/Tests/BuildTests/BuildPlanTests.swift b/Tests/BuildTests/BuildPlanTests.swift
@@ -47,7 +47,7 @@ extension Build.BuildPlan {
 
 class BuildPlanTestCase: BuildSystemProviderTestCase {
     override func setUpWithError() throws {
-        try XCTSkipIf(type(of: self) == BuildPlanTestCase.self, "Pay no attention to the class behind the curtain.")
+        try XCTSkipIf(type(of: self) == BuildPlanTestCase.self, "Skipping this test since it will be run in subclasses that will provide different build systems to test.")
     }
 
     let inputsDir = AbsolutePath(#file).parentDirectory.appending(components: "Inputs")

diff --git a/Tests/CommandsTests/APIDiffTests.swift b/Tests/CommandsTests/APIDiffTests.swift
@@ -27,7 +27,7 @@ import XCTest
 
 class APIDiffTestCase: CommandsBuildProviderTestCase {
     override func setUpWithError() throws {
-        try XCTSkipIf(type(of: self) == APIDiffTestCase.self, "Pay no attention to the class behind the curtain.")
+        try XCTSkipIf(type(of: self) == APIDiffTestCase.self, "Skipping this test since it will be run in subclasses that will provide different build systems to test.")
     }
 
     @discardableResult
@@ -481,4 +481,4 @@ class APIDiffSwiftBuildTests: APIDiffTestCase {
     override func skipIfApiDigesterUnsupportedOrUnset() throws {
         try super.skipIfApiDigesterUnsupportedOrUnset()
     }
-}
+}
diff --git a/Tests/CommandsTests/BuildCommandTests.swift b/Tests/CommandsTests/BuildCommandTests.swift
@@ -34,7 +34,7 @@ struct BuildResult {
 class BuildCommandTestCases: CommandsBuildProviderTestCase {
 
     override func setUpWithError() throws {
-        try XCTSkipIf(type(of: self) == BuildCommandTestCases.self, "Pay no attention to the class behind the curtain.")
+        try XCTSkipIf(type(of: self) == BuildCommandTestCases.self, "Skipping this test since it will be run in subclasses that will provide different build systems to test.")
     }
 
     @discardableResult

diff --git a/Tests/CommandsTests/PackageCommandTests.swift b/Tests/CommandsTests/PackageCommandTests.swift
@@ -33,7 +33,7 @@ import class Basics.AsyncProcess
 
 class PackageCommandTestCase: CommandsBuildProviderTestCase {
     override func setUpWithError() throws {
-        try XCTSkipIf(type(of: self) == PackageCommandTestCase.self, "Pay no attention to the class behind the curtain.")
+        try XCTSkipIf(type(of: self) == PackageCommandTestCase.self, "Skipping this test since it will be run in subclasses that will provide different build systems to test.")
     }
 
     @discardableResult

diff --git a/Tests/CommandsTests/RunCommandTests.swift b/Tests/CommandsTests/RunCommandTests.swift
@@ -21,7 +21,7 @@ import class Basics.AsyncProcess
 
 class RunCommandTestCase: CommandsBuildProviderTestCase {
     override func setUpWithError() throws {
-        try XCTSkipIf(type(of: self) == RunCommandTestCase.self, "Pay no attention to the class behind the curtain.")
+        try XCTSkipIf(type(of: self) == RunCommandTestCase.self, "Skipping this test since it will be run in subclasses that will provide different build systems to test.")
     }
 
     private func execute(

diff --git a/Tests/CommandsTests/TestCommandTests.swift b/Tests/CommandsTests/TestCommandTests.swift
@@ -20,7 +20,7 @@ import XCTest
 
 class TestCommandTestCase: CommandsBuildProviderTestCase {
     override func setUpWithError() throws {
-        try XCTSkipIf(type(of: self) == TestCommandTestCase.self, "Pay no attention to the class behind the curtain.")
+        try XCTSkipIf(type(of: self) == TestCommandTestCase.self, "Skipping this test since it will be run in subclasses that will provide different build systems to test.")
     }
 
     private func execute(