diff options
| author | Ned Batchelder <ned@nedbatchelder.com> | 2021-10-27 08:02:44 -0400 |
|---|---|---|
| committer | Ned Batchelder <ned@nedbatchelder.com> | 2021-10-27 08:50:31 -0400 |
| commit | 15aff0ad7d6c92851bc21fc39863a3949edbb9c6 (patch) | |
| tree | 4c1bbef4e9e6d9d3f789fb3b52649c42100ccbf5 /tests/test_debug.py | |
| parent | 93c9ca9f1b2e5d0b45dbf4b82c77aaf05b458bac (diff) | |
| download | python-coveragepy-git-15aff0ad7d6c92851bc21fc39863a3949edbb9c6.tar.gz | |
refactor(test): re_lines is more useful if it returns a list
Diffstat (limited to 'tests/test_debug.py')
| -rw-r--r-- | tests/test_debug.py | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/tests/test_debug.py b/tests/test_debug.py index 740bd617..564c2c29 100644 --- a/tests/test_debug.py +++ b/tests/test_debug.py @@ -15,7 +15,7 @@ from coverage.debug import filter_text, info_formatter, info_header, short_id, s from coverage.debug import clipped_repr from tests.coveragetest import CoverageTest -from tests.helpers import re_line, re_lines +from tests.helpers import re_line, re_lines, re_lines_text class InfoFormatterTest(CoverageTest): @@ -141,7 +141,7 @@ class DebugTraceTest(CoverageTest): # Now our lines are always prefixed with the process id. pid_prefix = r"^%5d\.[0-9a-f]{4}: " % os.getpid() - pid_lines = re_lines(out_lines, pid_prefix) + pid_lines = re_lines_text(out_lines, pid_prefix) assert pid_lines == out_lines # We still have some tracing, and some not tracing. @@ -152,9 +152,9 @@ class DebugTraceTest(CoverageTest): out_lines = self.f1_debug_output(["pid", "dataop", "dataio", "callers"]) # For every real message, there should be a stack trace with a line like # "f1_debug_output : /Users/ned/coverage/tests/test_debug.py @71" - real_messages = re_lines(out_lines, r":\d+", match=False).splitlines() + real_messages = re_lines(out_lines, r":\d+", match=False) frame_pattern = r"\s+f1_debug_output : .*tests[/\\]test_debug.py:\d+$" - frames = re_lines(out_lines, frame_pattern).splitlines() + frames = re_lines(out_lines, frame_pattern) assert len(real_messages) == len(frames) last_line = out_lines.splitlines()[-1] @@ -178,7 +178,7 @@ class DebugTraceTest(CoverageTest): for label in labels: label_pat = r"^\s*%s: " % label msg = "Incorrect lines for %r" % label - assert 1 == len(re_lines(out_lines, label_pat).splitlines()), msg + assert 1 == len(re_lines(out_lines, label_pat)), msg def test_debug_sys(self): out_lines = self.f1_debug_output(["sys"]) @@ -192,7 +192,7 @@ class DebugTraceTest(CoverageTest): for label in labels: label_pat = r"^\s*%s: " % label msg = "Incorrect lines for %r" % label - assert 1 == len(re_lines(out_lines, label_pat).splitlines()), msg + assert 1 == len(re_lines(out_lines, label_pat)), msg def test_debug_sys_ctracer(self): out_lines = self.f1_debug_output(["sys"]) |