Avoid useless or redundant db operations. Faster.

Moving operations into the "with self._connect" means less opening and
closing of the database.  Returning early if there is no data to write
avoids writing empty contexts.

1 files changed, 6 insertions, 4 deletions

diff --git a/tests/test_debug.py b/tests/test_debug.py
index 4eaba92e..351ef919 100644
--- a/tests/test_debug.py
+++ b/tests/test_debug.py
@@ -136,16 +136,18 @@ class DebugTraceTest(CoverageTest):
         frames = re_lines(out_lines, frame_pattern).splitlines()
         self.assertEqual(len(real_messages), len(frames))
 
-        # The last message should be "Writing data", and the last frame should
-        # be _write_file in data.py.
         last_line = out_lines.splitlines()[-1]
+
+        # The details of what to expect on the stack are empirical, and can change
+        # as the code changes. This test is here to ensure that the debug code
+        # continues working. It's ok to adjust these details over time.
         from coverage.data import STORAGE
         if STORAGE == "json":
             self.assertRegex(real_messages[-1], r"^\s*\d+\.\w{4}: Writing data")
             self.assertRegex(last_line, r"\s+_write_file : .*coverage[/\\]data.py @\d+$")
         else:
-            self.assertRegex(real_messages[-1], r"^\s*\d+\.\w{4}: Creating data file")
-            self.assertRegex(last_line, r"\s+_create_db : .*coverage[/\\]sqldata.py @\d+$")
+            self.assertRegex(real_messages[-1], r"^\s*\d+\.\w{4}: Adding file tracers: 0 files")
+            self.assertRegex(last_line, r"\s+add_file_tracers : .*coverage[/\\]sqldata.py @\d+$")
 
     def test_debug_config(self):
         out_lines = self.f1_debug_output(["config"])