ensure log file cleanup as finalizer

Signed-off-by: Jeffrey Martin <[email protected]>

tests/cli/test_cli.py

@@ -8,11 +8,24 @@
 
 
 @pytest.fixture(autouse=True)
-def close_report_file():
-    if _config.transient.reportfile is not None:
-        _config.transient.reportfile.close()
-        if os.path.exists(_config.transient.report_filename):
-            os.remove(_config.transient.report_filename)
+def cleanup(request):
+    """Cleanup a testing and report directory once we are finished."""
+
+    def remove_log_files():
+        files = []
+        if _config.transient.reportfile is not None:
+            _config.transient.reportfile.close()
+            report_html_file = _config.transient.report_filename.replace(
+                ".jsonl", ".html"
+            )
+            files.append(_config.transient.report_filename)
+            files.append(report_html_file)
+
+        for file in files:
+            if os.path.exists(file):
+                os.remove(file)
+
+    request.addfinalizer(remove_log_files)
 
 
 def test_version_command(capsys):

tests/test_config.py

@@ -482,7 +482,11 @@ def remove_log_files():
         ]
         if _config.transient.reportfile is not None:
             _config.transient.reportfile.close()
+            report_html_file = _config.transient.report_filename.replace(
+                ".jsonl", ".html"
+            )
             files.append(_config.transient.report_filename)
+            files.append(report_html_file)
 
         for file in files:
             if os.path.exists(file):