Skip to content

Allow tests to be disabled #218

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
2 tasks done
ycombinator opened this issue Jan 12, 2021 · 1 comment
Closed
2 tasks done

Allow tests to be disabled #218

ycombinator opened this issue Jan 12, 2021 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@ycombinator
Copy link
Contributor

ycombinator commented Jan 12, 2021

In elastic/integrations#514, @adriansr had to disable some packages' system tests in order to make progress on his feature. To do this, he had to remove the system test configuration files entirely because that is the only way to disable tests at the moment.

Instead, it would be nice if the configuration files accepted a skip or similar field which, if present, would skip the test. Further, the value of this field could be an arbitrary string representing the reason for skipping/disabling the test. Finally, this capability could be applied not just to system tests but also other types of tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant