Skip to content

Reduce test spam by using unittest output buffering #819

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Reduce test spam by using unittest output buffering #819

wants to merge 1 commit into from

Conversation

awwad
Copy link
Contributor

@awwad awwad commented Jan 8, 2019

After merge, stdout should only appear if a test has failed. This functionality is provided by unittest.main argument buffer=True. This functions like the --buffer command line argument listed here. std out is discarded if a test succeeds.

@awwad
Reduce test spam by using unittest output buffering
d15c362 
stdout will only appear if a test has failed.  This is provided
by unittest.main argument buffer=True.

Signed-off-by: Sebastien Awwad <sebastien.awwad@gmail.com>
@lukpueh
Copy link
Member

lukpueh commented Jan 8, 2019

FWIF you can also configure buffer=True only once on your unittest.TextTestRunner instance in aggregate_tests.py#L94.

@awwad awwad mentioned this pull request Feb 19, 2019
Merged
@awwad
Copy link
Contributor Author

awwad commented Feb 19, 2019

Closing in favor of #825.

@awwad awwad closed this Feb 19, 2019
@trishankatdatadog trishankatdatadog deleted the quieter_tests branch November 5, 2019 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@awwad @lukpueh