mirror of https://github.com/databricks/cli.git
70b7bbfd81
## Changes The `Setenv` helper function configures an environment variable and resets it to its original value when exiting the test scope. It is incompatible with running tests in parallel because it modifies process-wide state. The `libs/env` package defines functions to interact with the environment but records `Setenv` calls on a `context.Context`. This enables us to override/specialize the environment scoped to a context. Pre-requisites for removing the `t.Setenv` calls: * Make `cmdio.NewIO` accept a context and use it with `libs/env` * Make all `internal/testcli` functions use a context The rest of this change: * Modifies integration tests to initialize a context to use if there wasn't already one * Updates `t.Setenv` calls to use `env.Set` ## Tests n/a |
||
---|---|---|
.. | ||
templates | ||
testdata | ||
builtin.go | ||
builtin_test.go | ||
config.go | ||
config_test.go | ||
file.go | ||
file_test.go | ||
helpers.go | ||
helpers_test.go | ||
materialize.go | ||
materialize_test.go | ||
renderer.go | ||
renderer_test.go |