2023-09-27 09:04:44 +00:00
|
|
|
package process
|
|
|
|
|
|
|
|
import (
|
|
|
|
"context"
|
|
|
|
"fmt"
|
|
|
|
"io"
|
|
|
|
"os/exec"
|
|
|
|
"strings"
|
|
|
|
|
|
|
|
"github.com/databricks/cli/libs/env"
|
|
|
|
"github.com/databricks/cli/libs/log"
|
|
|
|
)
|
|
|
|
|
|
|
|
func Forwarded(ctx context.Context, args []string, src io.Reader, outWriter, errWriter io.Writer, opts ...execOption) error {
|
|
|
|
commandStr := strings.Join(args, " ")
|
|
|
|
log.Debugf(ctx, "starting: %s", commandStr)
|
|
|
|
cmd := exec.CommandContext(ctx, args[0], args[1:]...)
|
|
|
|
|
|
|
|
// empirical tests showed buffered copies being more responsive
|
|
|
|
cmd.Stdout = outWriter
|
|
|
|
cmd.Stderr = errWriter
|
|
|
|
cmd.Stdin = src
|
|
|
|
// we pull the env through lib/env such that we can run
|
|
|
|
// parallel tests with anything using libs/process.
|
|
|
|
for k, v := range env.All(ctx) {
|
|
|
|
cmd.Env = append(cmd.Env, fmt.Sprintf("%s=%s", k, v))
|
|
|
|
}
|
|
|
|
|
|
|
|
// apply common options
|
|
|
|
for _, o := range opts {
|
|
|
|
err := o(ctx, cmd)
|
|
|
|
if err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
Added process stubbing for easier testing of launched subprocesses (#963)
## Changes
This PR makes unit testing with subprocesses fast.
```
ctx := context.Background()
ctx, stub := process.WithStub(ctx)
stub.WithDefaultOutput("meeee")
ctx = env.Set(ctx, "FOO", "bar")
out, err := process.Background(ctx, []string{"/usr/local/bin/meeecho", "1", "--foo", "bar"})
require.NoError(t, err)
require.Equal(t, "meeee", out)
require.Equal(t, 1, stub.Len())
require.Equal(t, []string{"meeecho 1 --foo bar"}, stub.Commands())
allEnv := stub.CombinedEnvironment()
require.Equal(t, "bar", allEnv["FOO"])
require.Equal(t, "bar", stub.LookupEnv("FOO"))
```
This should make further iterations of
https://github.com/databricks/cli/pull/914 easier
## Tests
`make test`
2023-11-09 14:24:05 +00:00
|
|
|
return runCmd(ctx, cmd)
|
2023-09-27 09:04:44 +00:00
|
|
|
}
|