Skip to content

Commit 72acb5d

Browse files
committed
Don’t set --verbose twice when invoking SwiftPM
`invokeSwiftPM` already sets `--verbose`. No need to pass it again in the `additionalArguments`.
1 parent 5a55dbb commit 72acb5d

File tree

1 file changed

+2
-17
lines changed
  • SwiftSyntaxDevUtils/Sources/swift-syntax-dev-utils/commands

1 file changed

+2
-17
lines changed

SwiftSyntaxDevUtils/Sources/swift-syntax-dev-utils/commands/Test.swift

Lines changed: 2 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -32,18 +32,11 @@ struct Test: ParsableCommand, BuildCommand {
3232

3333
private func runTests() throws {
3434
logSection("Running SwiftSyntax Tests")
35-
var swiftpmCallArguments = [
36-
"--test-product", "swift-syntaxPackageTests",
37-
]
38-
39-
if arguments.verbose {
40-
swiftpmCallArguments += ["--verbose"]
41-
}
4235

4336
try invokeSwiftPM(
4437
action: "test",
4538
packageDir: Paths.packageDir,
46-
additionalArguments: swiftpmCallArguments,
39+
additionalArguments: ["--test-product", "swift-syntaxPackageTests"],
4740
additionalEnvironment: swiftPMEnvironmentVariables,
4841
captureStdout: false,
4942
captureStderr: false
@@ -52,18 +45,10 @@ struct Test: ParsableCommand, BuildCommand {
5245

5346
private func runCodeGenerationTests() throws {
5447
logSection("Running CodeGeneration Tests")
55-
var swiftpmCallArguments = [
56-
"--test-product", "CodeGenerationPackageTests",
57-
]
58-
59-
if arguments.verbose {
60-
swiftpmCallArguments += ["--verbose"]
61-
}
62-
6348
try invokeSwiftPM(
6449
action: "test",
6550
packageDir: Paths.codeGenerationDir,
66-
additionalArguments: swiftpmCallArguments,
51+
additionalArguments: ["--test-product", "CodeGenerationPackageTests"],
6752
additionalEnvironment: swiftPMEnvironmentVariables,
6853
captureStdout: false,
6954
captureStderr: false

0 commit comments

Comments
 (0)