diff options
author | Ned Batchelder <ned@nedbatchelder.com> | 2013-09-08 15:47:45 -0400 |
---|---|---|
committer | Ned Batchelder <ned@nedbatchelder.com> | 2013-09-08 15:47:45 -0400 |
commit | bcbe9da43a5e6564a33ec3d78098393cb5ecb3d0 (patch) | |
tree | 2f03989080b1cf653f378c6fdd22299d8887ca04 | |
parent | 7ee8de2756922ae14cd51bbbede2ca25be86d7f6 (diff) | |
download | python-coveragepy-git-bcbe9da43a5e6564a33ec3d78098393cb5ecb3d0.tar.gz |
Make the should_trace_cache test a little more bullet-proof.
-rw-r--r-- | tests/test_collector.py | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/tests/test_collector.py b/tests/test_collector.py index d0aa997e..29c39996 100644 --- a/tests/test_collector.py +++ b/tests/test_collector.py @@ -43,5 +43,12 @@ class CollectorTest(CoverageTest): # Grab all the filenames mentioned in debug output, there should be no # duplicates. - filenames = re.findall(r"'[^']+'", debug_out.getvalue()) + trace_lines = [ + l for l in debug_out.getvalue().splitlines() + if l.startswith("Tracing ") or l.startswith("Not tracing ") + ] + filenames = [re.search(r"'[^']+'", l).group() for l in trace_lines] self.assertEqual(len(filenames), len(set(filenames))) + + # Double-check that the tracing messages are in there somewhere. + self.assertTrue(len(filenames) > 5) |