Skip to content

Commit ea0fde3

Browse files
authored
Merge pull request #2893 from apple/nate/test_default
Use .none as the default test specification
2 parents 92acf14 + c4caace commit ea0fde3

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

Sources/Commands/SwiftTestTool.swift

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,13 @@ struct TestToolOptions: ParsableArguments {
114114

115115
var testCaseSkip: TestCaseSpecifier {
116116
// TODO: Remove this once the environment variable is no longer used.
117-
testCaseSkipOverride() ?? .skip(_testCaseSkip)
117+
if let override = testCaseSkipOverride() {
118+
return override
119+
}
120+
121+
return _testCaseSkip.isEmpty
122+
? .none
123+
: .skip(_testCaseSkip)
118124
}
119125

120126
@Option(name: .customLong("skip"),

0 commit comments

Comments
 (0)