logging: move log_file_handler cleanup from sessionend to unconfigure

It is set-up in configure, so match it.

Author: bluetech

src/_pytest/logging.py

@@ -678,10 +678,6 @@ def pytest_sessionfinish(self):
             with catching_logs(self.log_file_handler, level=self.log_file_level):
                 yield
 
-        # Close the FileHandler explicitly.
-        # (logging.shutdown might have lost the weakref?!)
-        self.log_file_handler.close()
-
     @pytest.hookimpl(hookwrapper=True, tryfirst=True)
     def pytest_sessionstart(self):
         self.log_cli_handler.set_when("sessionstart")
@@ -706,6 +702,12 @@ def pytest_runtestloop(self, session):
             with catching_logs(self.log_file_handler, level=self.log_file_level):
                 yield  # run all the tests
 
+    @pytest.hookimpl
+    def pytest_unconfigure(self):
+        # Close the FileHandler explicitly.
+        # (logging.shutdown might have lost the weakref?!)
+        self.log_file_handler.close()
+
 
 class _LiveLoggingStreamHandler(logging.StreamHandler):
     """