feat: removed legacy command.Test function #67920

Signed-off-by: nikzayn <[email protected]>

Author: nikzayn

Diff for: gopls/internal/golang/code_lens.go

@@ -48,7 +48,11 @@ func runTestCodeLens(ctx context.Context, snapshot *cache.Snapshot, fh file.Hand
 	}
 	puri := fh.URI()
 	for _, fn := range testFuncs {
-		cmd, err := command.NewTestCommand("run test", puri, []string{fn.name}, nil)
+		cmd, err := command.NewRunTestsCommand("run test", command.RunTestsArgs{
+			URI:        puri,
+			Tests:      []string{fn.name},
+			Benchmarks: nil,
+		})
 		if err != nil {
 			return nil, err
 		}
@@ -57,7 +61,11 @@ func runTestCodeLens(ctx context.Context, snapshot *cache.Snapshot, fh file.Hand
 	}
 
 	for _, fn := range benchFuncs {
-		cmd, err := command.NewTestCommand("run benchmark", puri, nil, []string{fn.name})
+		cmd, err := command.NewRunTestsCommand("run benchmark", command.RunTestsArgs{
+			URI:        puri,
+			Tests:      nil,
+			Benchmarks: []string{fn.name},
+		})
 		if err != nil {
 			return nil, err
 		}
@@ -79,7 +87,11 @@ func runTestCodeLens(ctx context.Context, snapshot *cache.Snapshot, fh file.Hand
 		for _, fn := range benchFuncs {
 			benches = append(benches, fn.name)
 		}
-		cmd, err := command.NewTestCommand("run file benchmarks", puri, nil, benches)
+		cmd, err := command.NewRunTestsCommand("run file benchmarks", command.RunTestsArgs{
+			URI:        puri,
+			Tests:      nil,
+			Benchmarks: benches,
+		})
 		if err != nil {
 			return nil, err
 		}

Diff for: gopls/internal/protocol/command/command_gen.go

@@ -45,18 +45,6 @@ type Interface interface {
 	// Applies a fix to a region of source code.
 	ApplyFix(context.Context, ApplyFixArgs) (*protocol.WorkspaceEdit, error)
 
-	// Test: Run test(s) (legacy)
-	//
-	// Runs `go test` for a specific set of test or benchmark functions.
-	//
-	// This command is asynchronous; wait for the 'end' progress notification.
-	//
-	// This command is an alias for RunTests; the only difference
-	// is the form of the parameters.
-	//
-	// TODO(adonovan): eliminate it.
-	Test(context.Context, protocol.DocumentURI, []string, []string) error
-
 	// Test: Run test(s)
 	//
 	// Runs `go test` for a specific set of test or benchmark functions.