Skip to content

Commit

Permalink
Merge pull request #5119 from blueyed/reportopts-A-order
Browse files Browse the repository at this point in the history
reportopts: A: put "Pp" in front
  • Loading branch information
blueyed committed May 24, 2019
2 parents b4d75ad + f9f41e6 commit 10ca84f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/_pytest/terminal.py
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ def getreportopt(config):
if char == "a":
reportopts = "sxXwEf"
elif char == "A":
reportopts = "sxXwEfpP"
reportopts = "PpsxXwEf"
break
elif char not in reportopts:
reportopts += char
Expand Down
2 changes: 1 addition & 1 deletion testing/test_terminal.py
Original file line number Diff line number Diff line change
Expand Up @@ -893,7 +893,7 @@ class Option(object):
assert getreportopt(config) == "sxXwEf" # NOTE: "w" included!

config.option.reportchars = "A"
assert getreportopt(config) == "sxXwEfpP"
assert getreportopt(config) == "PpsxXwEf"


def test_terminalreporter_reportopt_addopts(testdir):
Expand Down

0 comments on commit 10ca84f

Please sign in to comment.