-
Notifications
You must be signed in to change notification settings - Fork 127
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
Add support for running tagged test files #2052
Add support for running tagged test files #2052
Conversation
To view this pull requests documentation preview, visit the following URL: Documentation is deployed and generated using docs.page. |
Hi @kostadin-damyanov-prime, thanks a lot for making this contribution! We have just discussed tags internally :) I have a few questions:
|
@mateuszwojtczak Hi.
|
Yes, look at this information in their readme. They also support I'm just worried that the community would assume that Patrol uses the same tags as Dart/Flutter tests. This is a very frequent assumption that ends with people filing issues, because they assume Patrol uses the same approach. Although we can't always be 100% compatible with that approach, we still at least try to do so. |
I agree that mplementation based on |
@kostadin-damyanov-prime We decided to implement tags and skips by passing them as arguments to the Thus, I'm closing this PR and we're gonna solve #492 with another approach :) |
Hello.
This PR adds support for
--tags
option for thetest
command.This way we can tag test files with the
@Tags
annotation like dart's test suite does.Using this option we can now run only the test files matching a given tag or a list of tags.