You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently Pipes tests arguments are a bit messy.
I am planning to clean them up and possible add more structure to CLI arguments.
Proposals:
Remove --env and --full flags. They don't really have any useful meaning.
Remove --job-name argument from all tests except test_components.
Remove test_context_reconstruction. It doesn't do anything which can't be part of test_components.
Rename --custom-payload-path to --custom-payload to be consistent with other arguments
Add a --test-name argument which will be set uniquely for each test. Remove obsolete flags like --throw-error, --logging (check if --test-name matches test_error_reporting and similar conditions instead).
Currently Pipes tests arguments are a bit messy.
I am planning to clean them up and possible add more structure to CLI arguments.
Proposals:
--env
and--full
flags. They don't really have any useful meaning.--job-name
argument from all tests excepttest_components
.test_context_reconstruction
. It doesn't do anything which can't be part oftest_components
.--custom-payload-path
to--custom-payload
to be consistent with other arguments--test-name
argument which will be set uniquely for each test. Remove obsolete flags like--throw-error
,--logging
(check if--test-name
matchestest_error_reporting
and similar conditions instead).This change will require at least 3 PRs:
What are your thoughts @GingerYouth @marijncv @wingyplus?
The text was updated successfully, but these errors were encountered: