Make sure swift test --show-code-cov-path works when swift-testing is enabled.

This PR ensures that the aforementioned option behaves correctly when `--enable-experimental-swift-testing` is passed.

As it stands, `--show-code-cov-path` is implemented as a subcommand but behaves like an option; this PR removes the subcommand so that it "just works" without regard for other options.

A longer-term solution could be to make it into a proper subcommand (e.g. `swift test show-path code-coverage`.) That would ensure that we would get appropriate output from Swift Argument Parser when mixing it with other (nonsensical) options.

Author: grynspan

Sources/Commands/SwiftTestTool.swift

@@ -346,8 +346,7 @@ public struct SwiftTestTool: SwiftCommand {
         }
 
         if self.options.shouldPrintCodeCovPath {
-            let command = try PrintCodeCovPath.parse()
-            try command.run(swiftTool)
+            try printCodeCovPath(swiftTool)
         } else if self.options._deprecated_shouldListTests {
             // backward compatibility 6/2022 for deprecation of flag into a subcommand
             let command = try List.parse()
@@ -542,36 +541,21 @@ public struct SwiftTestTool: SwiftCommand {
 }
 
 extension SwiftTestTool {
-     struct PrintCodeCovPath: SwiftCommand {
-         static let configuration = CommandConfiguration(
-             commandName: "show-codecov-path",
-             abstract: "Print the path of the exported code coverage JSON file"
-         )
-
-         @OptionGroup(visibility: .hidden)
-         var globalOptions: GlobalOptions
-
-         // for deprecated passthrough from SwiftTestTool (parse will fail otherwise)
-
-         @Flag(name: [.customLong("show-codecov-path"), .customLong("show-code-coverage-path"), .customLong("show-coverage-path")], help: .hidden)
-         var _deprecated_passthrough: Bool = false
-
-         func run(_ swiftTool: SwiftTool) throws {
-             let workspace = try swiftTool.getActiveWorkspace()
-             let root = try swiftTool.getWorkspaceRoot()
-             let rootManifests = try temp_await {
-                 workspace.loadRootManifests(
-                     packages: root.packages,
-                     observabilityScope: swiftTool.observabilityScope,
-                     completion: $0
-                 )
-             }
-             guard let rootManifest = rootManifests.values.first else {
-                 throw StringError("invalid manifests at \(root.packages)")
-             }
-             let buildParameters = try swiftTool.buildParametersForTest(enableCodeCoverage: true, library: .xctest)
-             print(buildParameters.codeCovAsJSONPath(packageName: rootManifest.displayName))
-         }
+    func printCodeCovPath(_ swiftTool: SwiftTool) throws {
+        let workspace = try swiftTool.getActiveWorkspace()
+        let root = try swiftTool.getWorkspaceRoot()
+        let rootManifests = try temp_await {
+            workspace.loadRootManifests(
+                packages: root.packages,
+                observabilityScope: swiftTool.observabilityScope,
+                completion: $0
+            )
+        }
+        guard let rootManifest = rootManifests.values.first else {
+            throw StringError("invalid manifests at \(root.packages)")
+        }
+        let buildParameters = try swiftTool.buildParametersForTest(enableCodeCoverage: true, library: .xctest)
+        print(buildParameters.codeCovAsJSONPath(packageName: rootManifest.displayName))
      }
  }