Skip to content
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

feat: implement skip utility function for testing #181

Merged
merged 1 commit into from
May 13, 2020

Conversation

gianarb
Copy link
Collaborator

@gianarb gianarb commented Apr 15, 2020

I implement a new function to use as testing utility.
It skips the test if the provider is not reachable or well provisioned.

You have to declare it in your test, so if you depend on Docker and you
know you need it just keep doing as you are doing today.

In my use case we use testcontainers only for some particular tests, and
I do not like the idea to mark the testsuite as FAILED if a colleague
does not have Docker enabled in their local environment. Worst case if
something has to fail CI/CD will tell the truth.

Signed-off-by: Gianluca Arbezzano gianarb92@gmail.com

I implement a new function to use as testing utility.
It skips the test if the provider is not reachable or well provisioned.

You have to declare it in your test, so if you depend on Docker and you
know you need it just keep doing as you are doing today.

In my use case we use testcontainers only for some particular tests, and
I do not like the idea to mark the testsuite as `FAILED` if a colleague
does not have Docker enabled in their local environment. Worst case if
something has to fail CI/CD will tell the truth.

Signed-off-by: Gianluca Arbezzano <gianarb92@gmail.com>
@gianarb gianarb added feature New functionality or new behaviors on the existing one type/test-improvement labels Apr 15, 2020
@gianarb gianarb merged commit 93d5444 into master May 13, 2020
@mdelapenya mdelapenya deleted the feature/skip-testing-util branch November 15, 2022 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New functionality or new behaviors on the existing one
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant