-
Notifications
You must be signed in to change notification settings - Fork 14
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
New test filters #82
New test filters #82
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci @DurieuxPol !
Why one of the filters rejects while the other filter selects?
That will confuse people (starting by myself :D)
Because to me, you want to use pragma in tests as flags that specify which tests you don't want to run. |
I understand, maybe the name should be more explicit... But to me either the filters do filter out (reject) or select. |
I agree, I should rename it. |
assert: (analysis generalResult particularResults at: 1) runCount | ||
equals: 1 | ||
assert: (result mutantResults at: 1) selector | ||
equals: #test10Milliseconds |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh yes this seems nicer!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good! Thanks @DurieuxPol !
Added 2 new test filters:
MTBlockTestFilter
which selects tests based on the given blockMTPragmaRejectionTestFilter
which rejects tests that contain a given pragmaMTPragmaSelectionTestFilter
which selects tests that contain a given pragma