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

Doc about skipping some integration tests with '-check.f ConsulCatalogSuite' #418

Merged

Conversation

samber
Copy link
Contributor

@samber samber commented May 30, 2016

No description provided.

@@ -77,6 +77,11 @@ ok github.com/containous/traefik 0.005s coverage: 4.1% of statements
Test success
```

For dev purpose, you can skip some integration tests by using:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's more "you can specifiy which tests to run" instead of skipping 👼

TESTFLAGS="-check.f MyTestSuite.MyTest" make test-integration

MyTest can even be not the full name, like if if use -check.f MyTestSuite.My it will run all tests starting by My 😉

@samber samber force-pushed the doc--tooling--integration-test-filtering branch 2 times, most recently from 755d49e to c33736a Compare May 31, 2016 08:02
@samber
Copy link
Contributor Author

samber commented May 31, 2016

Corrected ;-) 👍

For development purpose, you can specifiy which tests to run by using:
```
# Run every tests in the ConsulCatalog suite
TESTFLAGS="-check.f MyTestSuite" make test-integration
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hum small nit, but the comment is a little bit wrong 😝 (MySuite vs ConsulCatalog suite)

@vdemeester
Copy link
Contributor

Other than the comment, LGTM 😉
(and needs a rebase, sorry 😓)

@samber samber force-pushed the doc--tooling--integration-test-filtering branch from c33736a to a255ea4 Compare May 31, 2016 09:39
@samber
Copy link
Contributor Author

samber commented May 31, 2016

Oops, sorry, i forgot to rebase.

It's alright ;-)

@samber samber force-pushed the doc--tooling--integration-test-filtering branch from a255ea4 to 46c2184 Compare May 31, 2016 10:08
@emilevauge
Copy link
Member

LGTM! Thanks @samber 👍

@emilevauge emilevauge merged commit 39fa8f7 into traefik:master May 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants