-
Notifications
You must be signed in to change notification settings - Fork 584
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
Allow labelling of values drawn from data strategy #428
Allow labelling of values drawn from data strategy #428
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.
Docs additions aside, I think this is good to go. 👍
@@ -49,6 +49,24 @@ def test(data): | |||
assert 'Draw 2: 0' in result | |||
|
|||
|
|||
def test_prints_labels_if_given_on_failure(): |
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 was going to ask if we need a test that labels aren’t printed when not specified, but I think that’s covered by the existing tests – since the fallback behaviour is not to print any labels, we’d notice if/when those tests broke.
👍 I like the feature. Pending docs change I'm happy to have it. |
a4148d0
to
51b3c72
Compare
OK, docs updated. |
Sorry, I didn't mean to drop this. I'm happy with this as is, but it needs reformatting ("make format" should fix it followed by "make lint" to check if it did) to make the build pass. Can you do that and then rebase off master and then I'm totally happy to merge. |
This results in more useful output in falsifying examples
51b3c72
to
f3d174b
Compare
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.
🎉
This results in more useful output in falsifying examples.
If you're happy with this I can update the documentation.