databricks-cli

Commit Graph

Author	SHA1	Message	Date
Pieter Noordhuis	e3717ba1c4	Fix flaky test in `libs/process` (#1314 ) ## Changes The order of stdout and stderr being read into the buffer for combined output is not deterministic due to scheduling of the underlying goroutines that consume them. That's why this asserts on the contents and not the order.	2024-03-26 07:57:48 +00:00
Andrew Nester	ea4153e323	Fixed flaky TestBackgroundCombinedOutputFailure (#978 ) ## Changes `TestBackgroundCombinedOutputFailure` was occasionally failing because combined output could have been in different order, see https://github.com/databricks/cli/actions/runs/6823883271/job/18558675165?pr=928	2023-11-10 14:09:02 +00:00
Serge Smertin	7171874db0	Added `process.Background()` and `process.Forwarded()` (#804 ) ## Changes This PR adds higher-level wrappers for calling subprocesses. One of the steps to get https://github.com/databricks/cli/pull/637 in, as previously discussed. The reason to add `process.Forwarded()` is to proxy Python's `input()` calls from a child process seamlessly. Another use-case is plugging in `less` as a pager for the list results. ## Tests `make test`	2023-09-27 09:04:44 +00:00

Author

SHA1

Message

Date

Pieter Noordhuis

e3717ba1c4

Fix flaky test in `libs/process` (#1314 )

## Changes

The order of stdout and stderr being read into the buffer for combined
output is not deterministic due to scheduling of the underlying
goroutines that consume them. That's why this asserts on the contents
and not the order.

2024-03-26 07:57:48 +00:00

Andrew Nester

ea4153e323

Fixed flaky TestBackgroundCombinedOutputFailure (#978 )

## Changes
`TestBackgroundCombinedOutputFailure` was occasionally failing because
combined output could have been in different order, see


https://github.com/databricks/cli/actions/runs/6823883271/job/18558675165?pr=928

2023-11-10 14:09:02 +00:00

Serge Smertin

7171874db0

Added `process.Background()` and `process.Forwarded()` (#804 )

## Changes
This PR adds higher-level wrappers for calling subprocesses. One of the
steps to get https://github.com/databricks/cli/pull/637 in, as
previously discussed.

The reason to add `process.Forwarded()` is to proxy Python's `input()`
calls from a child process seamlessly. Another use-case is plugging in
`less` as a pager for the list results.

## Tests
`make test`

2023-09-27 09:04:44 +00:00

3 Commits