Test: use FORCE_COLOR=1 to ensure the test works #12036
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Subject: set the environment to get predictable test output
Feature or Bugfix
Purpose
Without this fix, one test fails when I run the test suite locally with tox:
The problem is that
color_terminal()
returns False because stdout is not a tty. The output then has\n
instead of\r
and the test assertion fails. By setting FORCE_COLOR=1 in the test, we remove the check on stdout, and the test produces predictable expected output.