Use pytest_runtest_logstart to signal the start of a new test

This also simplifies the code a bit because we don't need to keep
a set of ids anymore

_pytest/logging.py

@@ -305,7 +305,7 @@ class LoggingPlugin(object):
         with catching_logs(LogCaptureHandler(),
                            formatter=self.formatter, level=self.log_level) as log_handler:
             if self.log_cli_handler:
-                self.log_cli_handler.reset(item, when)
+                self.log_cli_handler.set_when(when)
             if not hasattr(item, 'catch_log_handlers'):
                 item.catch_log_handlers = {}
             item.catch_log_handlers[when] = log_handler
@@ -337,6 +337,10 @@ class LoggingPlugin(object):
         with self._runtest_for(item, 'teardown'):
             yield
 
+    def pytest_runtest_logstart(self):
+        if self.log_cli_handler:
+            self.log_cli_handler.reset()
+
     @pytest.hookimpl(hookwrapper=True)
     def pytest_runtestloop(self, session):
         """Runs all collected test items."""
@@ -385,18 +389,17 @@ class _LiveLoggingStreamHandler(logging.StreamHandler):
         """
         logging.StreamHandler.__init__(self, stream=terminal_reporter)
         self.capture_manager = capture_manager
+        self.reset()
+        self.set_when(None)
+
+    def reset(self):
+        """Reset the handler; should be called before the start of each test"""
         self._first_record_emitted = False
 
-        self._section_name_shown = False
-        self._when = None
-        self._run_tests = set()
-
-    def reset(self, item, when):
+    def set_when(self, when):
+        """Prepares for the given test phase (setup/call/teardown)"""
         self._when = when
         self._section_name_shown = False
-        if item.name not in self._run_tests:
-            self._first_record_emitted = False
-            self._run_tests.add(item.name)
 
     def emit(self, record):
         if self.capture_manager is not None:

testing/logging/test_reporting.py

@@ -527,10 +527,7 @@ def test_live_logging_suspends_capture(has_capture_manager, request):
     out_file = DummyTerminal()
     capture_manager = MockCaptureManager() if has_capture_manager else None
     handler = _LiveLoggingStreamHandler(out_file, capture_manager)
-
-    class DummyItem:
-        name = 'test_foo'
-    handler.reset(DummyItem(), 'call')
+    handler.set_when('call')
 
     logger = logging.getLogger(__name__ + '.test_live_logging_suspends_capture')
     logger.addHandler(handler)