-
-
Notifications
You must be signed in to change notification settings - Fork 208
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
Silence two tests #1245
Silence two tests #1245
Conversation
Guilty! That test is not doing what I expected... I can fix it instead of silencing it. May I? |
Yes, just use the branch (and update/alter what I did as needed). I poked around with |
Now it's me who requests your review, but GitHub doesn't allow me to add you, I suppose because you started this PR. :) |
|
||
* inst/tinytest/test_xptr.R: Condition one test writing to stdout | ||
on test verbosity environment variable being set | ||
* inst/tinytest/test_xptr.R: Fix a couple of tests writing to stdout |
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.
'A couple' ? 'A pair' 😜
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.
They're married by their purpose. :)
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.
LGTM!
This trivial PR makes one test (that otherwise barks on
stdout
) conditional on the env var making some tests quiet. As we usesystem()
the stdout capture of tinytest does not work for us.Checklist
R CMD check
still passes all tests