Skip to content
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

Make pytest in tox quite #21727

Merged
merged 1 commit into from Mar 7, 2019
Merged

Conversation

awarecan
Copy link
Contributor

@awarecan awarecan commented Mar 7, 2019

Description:

Current pytest output a beautiful progress bar, it is cool watching it in term. However it is meaningless in CI environment.

Add following options to pytest to quite the output as much as possible.

option
-qq quite and quiter
-o console_output_style=count change default percent progress to count
-p no:sugar disable beautiful pytest-sugar plugin

It will reduce some scrolling time I spent on finding the actual pytest error message on travis.

Reduced to 668 lines from 1200 lines of log

Related issue (if applicable): fixes #

Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>

Example entry for configuration.yaml (if applicable):

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.

@homeassistant homeassistant added small-pr PRs with less than 30 lines. cla-signed labels Mar 7, 2019
@ghost ghost assigned awarecan Mar 7, 2019
@ghost ghost added the in progress label Mar 7, 2019
@awarecan awarecan merged commit 38a93af into home-assistant:dev Mar 7, 2019
@ghost ghost removed the in progress label Mar 7, 2019
@awarecan awarecan deleted the quite-pytest branch March 7, 2019 06:27
@balloob balloob mentioned this pull request Mar 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed small-pr PRs with less than 30 lines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants