-
Notifications
You must be signed in to change notification settings - Fork 8
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 --hash-by-class #6
Conversation
assert True | ||
|
||
class TC2(unittest.TestCase): |
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.
I changed TC2 and TC3 to not inherit from TC1 to avoid inheriting parent test methods.
Hm. I just tried this on our test suite and it missed some classes. |
Hah, I get different results locally compared to our ci cluster. I'll reopen. |
Oops, that was just me being a dingus 😁 Ready for review. |
I hadn't seen that before either. Nice. |
9278d13
to
e7c53fb
Compare
Would you mind updating the README to document this new option? |
Good call; will do. |
👍 updated the readme |
bump 😁 |
Looks great! Thanks again, Simon. |
Implements #2.
(This is the first time I've used
PluginTester
-- it's pretty nifty! You might want to refactor your other tests to use it).