summaryrefslogtreecommitdiff
path: root/tests/coveragetest.py
diff options
context:
space:
mode:
authorNed Batchelder <ned@nedbatchelder.com>2021-01-31 08:16:50 -0500
committerNed Batchelder <ned@nedbatchelder.com>2021-01-31 09:20:49 -0500
commit716b31214126c4a06d8262a62153e5d1fa45fa54 (patch)
tree799b71514cf978927aa8f7144cda8ede25655c2a /tests/coveragetest.py
parent6e93714d655c87e62cd03a4ff3e1739245b684b9 (diff)
downloadpython-coveragepy-git-716b31214126c4a06d8262a62153e5d1fa45fa54.tar.gz
test: adapt to pytest assertion messages
Bare "assert" statements don't produce the same assertion message as self.assertEqual did: they don't include the two values compared. For some of our own asserts, add back the detailed message. For some checks of asserts, it's fine that the values are missing because the longer messsage includes the information.
Diffstat (limited to 'tests/coveragetest.py')
-rw-r--r--tests/coveragetest.py8
1 files changed, 4 insertions, 4 deletions
diff --git a/tests/coveragetest.py b/tests/coveragetest.py
index ed3f1839..2e6f1323 100644
--- a/tests/coveragetest.py
+++ b/tests/coveragetest.py
@@ -198,7 +198,7 @@ class CoverageTest(
if isinstance(lines[0], int):
# lines is just a list of numbers, it must match the statements
# found in the code.
- assert statements == lines
+ assert statements == lines, "{!r} != {!r}".format(statements, lines)
else:
# lines is a list of possible line number lists, one of them
# must match.
@@ -210,7 +210,7 @@ class CoverageTest(
missing_formatted = analysis.missing_formatted()
if isinstance(missing, string_class):
- assert missing_formatted == missing
+ assert missing_formatted == missing, "{!r} != {!r}".format(missing_formatted, missing)
else:
for missing_list in missing:
if missing_formatted == missing_list:
@@ -244,7 +244,7 @@ class CoverageTest(
frep = StringIO()
cov.report(mod, file=frep, show_missing=True)
rep = " ".join(frep.getvalue().split("\n")[2].split()[1:])
- assert report == rep
+ assert report == rep, "{!r} != {!r}".format(report, rep)
return cov
@@ -351,7 +351,7 @@ class CoverageTest(
"""
ret_actual = command_line(args)
- assert ret_actual == ret
+ assert ret_actual == ret, "{!r} != {!r}".format(ret_actual, ret)
# Some distros rename the coverage command, and need a way to indicate
# their new command name to the tests. This is here for them to override,