pytest-html-plus 1.2.0__tar.gz → 1.2.1__tar.gz

{pytest_html_plus-1.2.0 → pytest_html_plus-1.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pytest-html-plus
-Version: 1.2.0
+Version: 1.2.1
 Summary: Generate Actionable, automatic screenshots, unified Mobile friendly Pytest HTML report in less than 3 seconds — no hooks, merge plugins, no config, xdist-ready.
 License: MIT
 License-File: LICENSE

{pytest_html_plus-1.2.0 → pytest_html_plus-1.2.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "pytest-html-plus"
-version = "1.2.0"
+version = "1.2.1"
 description = "Generate Actionable, automatic screenshots, unified Mobile friendly Pytest HTML report in less than 3 seconds — no hooks, merge plugins, no config, xdist-ready."
 readme = "README.md"
 authors = ["reporterplus"]
@@ -78,13 +78,7 @@ html-output = "report_output_warnings"
 [tool.ruff]
 line-length = 88
 target-version = "py39"
-ignore = [
-    "E501",  # line too long — let black handle
-    "W291",  # trailing whitespace — let black handle
-    "W293",  # blank line whitespace — let black handle
-]
 exclude = [
-    "pytest_html_plus/generate_html_report.py",
     ".git"
 ]
 
@@ -96,4 +90,8 @@ select = [
     "I",   # isort
     "UP",  # pyupgrade
 ]
-ignore = []
+ignore = [
+    "E501",  # line too long — let black handle
+    "W291",  # trailing whitespace — let black handle
+    "W293",  # blank line whitespace — let black handle
+]

{pytest_html_plus-1.2.0 → pytest_html_plus-1.2.1}/pytest_html_plus/generate_html_report.py

@@ -5,7 +5,9 @@ import os
 import shutil
 from datetime import datetime, timezone
 from html import escape
+
 from pytest_html_plus.compute_filter_counts import compute_filter_count
+from pytest_html_plus.resolver_driver import sanitize_filename
 from pytest_html_plus.utils import extract_error_block, extract_trace_block
 
 
@@ -792,7 +794,14 @@ class JSONReporter:
                     else "error" if test["status"] == "error" else "skipped"
                 )
             )
-            screenshot_path = self.find_screenshot_and_copy(test["test"])
+            # NOTE: Parammetrized test name may contains unsupport filesystem-safe chars
+            # Use test_name will not match the screenshot.
+            # 1. use test["screenshot"] which is the the screenshot path will lost the wild pattern
+            # *2. use sanitize_filename which replace non-alphanumeric will match the screenshot filename pattern
+            # 3. try to split parametrized will be hard
+            screenshot_path = self.find_screenshot_and_copy(
+                sanitize_filename(test["test"])
+            )
             screenshot_html = (
                 f'<div class="details-screenshot"><img src="{screenshot_path}" alt="Screenshot" onclick="toggleFullscreen(this)"></div>'
                 if screenshot_path
@@ -847,7 +856,7 @@ class JSONReporter:
 
                     error_block = (
                         f'<div class="error-content"><strong>Error:</strong> '
-                        f'{self.generate_copy_button(error, "error")}'
+                        f"{self.generate_copy_button(error, 'error')}"
                         f"<pre>{error}</pre></div>"
                         if error
                         else ""
@@ -855,7 +864,7 @@ class JSONReporter:
 
                     trace_block = (
                         f'<div class="trace-content"><strong>Trace:</strong> '
-                        f'{self.generate_copy_button(trace, "trace")}'
+                        f"{self.generate_copy_button(trace, 'trace')}"
                         f"<pre>{trace}</pre></div>"
                         if trace
                         else ""
@@ -874,7 +883,7 @@ class JSONReporter:
                     """
 
                 attempts_html += "</div>"
-            
+
             search_error = ""
 
             if test.get("error"):
@@ -895,10 +904,8 @@ class JSONReporter:
                     <pre>{error_content}</pre></div>
                     """
                     search_error = (
-                        error_content.replace("\n", " ")
-                        .replace("\r", " ")
+                        error_content.replace("\n", " ").replace("\r", " ")
                     )[:1000]
-                   
 
             flaky_badge = ""
             if test.get("flaky"):
@@ -935,17 +942,18 @@ class JSONReporter:
                     '<span style="display:inline-block; min-width:45px;"></span>'
                 )
 
+            escaped_test_name = escape(test["test"])
             html += f"""    
-<div class="test test-card" data-name="{test["test"]}" data-link="{",".join(test.get("links") or [])}" data-markers="{marker_str}" data-error="{escape(search_error)}">
+<div class="test test-card" data-name="{escaped_test_name}" data-link="{",".join(test.get("links") or [])}" data-markers="{marker_str}" data-error="{escape(search_error)}">
   <div class="header {status_class}" onclick="toggleDetails(this)">
     <div class="header-section test-info">
       <span class="toggle"></span>
-      <strong>{test["test"]}</strong>
+      <strong>{escaped_test_name}</strong>
       <span>— {test["status"].upper()}</span>
     </div>
     <div class="header-section meta">
       <span class="nodeid-badge" style="display: flex; align-items: center; gap: 6px;">
-        <code style="font-size: 0.6em; color: #555;">{test["nodeid"]}</code>
+        <code style="font-size: 0.6em; color: #555;">{escape(test["nodeid"], quote=True)}</code>
           {self.generate_copy_button(test["nodeid"], "nodeid")}
       </span>
       <span class="worker-id" style="background: #ddd; border-radius: 3px; padding: 2px 5px; font-size: 0.85em; font-weight: bold;">{test["worker"]}</span>
@@ -990,20 +998,21 @@ class JSONReporter:
             "#e6f4ea" if failed_tests == 0 and error_tests == 0 else "#fdecea"
         }; 
             border: 1px solid {
-            "#2f7a33" if failed_tests == 0 and
-                         error_tests == 0
-            else "#a83232"
+            "#2f7a33" if failed_tests == 0 and error_tests == 0 else "#a83232"
         }; 
             border-radius: 5px; margin-bottom: 1rem;">
               {
             "<strong>Bingo!</strong> All your tests passed!"
             if failed_tests == 0 and error_tests == 0
-            else (f"Total tests: {total_tests}, "
-                  f"Failures: {failed_tests}, "
-                  f"Errors: {error_tests}.")
+            else (
+                f"Total tests: {total_tests}, "
+                f"Failures: {failed_tests}, "
+                f"Errors: {error_tests}."
+            )
         }
-              The slowest test was <strong>{slowest_test_name}</strong> at {
-            slowest_test_duration:.2f}s.
+              The slowest test was <strong>{
+            escape(slowest_test_name, quote=True)
+        }</strong> at {slowest_test_duration:.2f}s.
             </div>
             """
 

{pytest_html_plus-1.2.0 → pytest_html_plus-1.2.1}/pytest_html_plus/plugin.py

@@ -247,7 +247,6 @@ def pytest_runtest_makereport(item, call):
                 screenshot_path = take_screenshot_generic(screenshot_path, item, driver)
 
         worker_id = os.getenv("PYTEST_XDIST_WORKER") or "main"
-        test_name = "".join(c if c.isalnum() else "_" for c in item.name)
 
         status = report.outcome
         if report.when in ("setup", "teardown") and report.failed:
@@ -256,14 +255,13 @@ def pytest_runtest_makereport(item, call):
         output_policy = config.getoption(OUTPUT_OPTION)
         is_expected_failure = report.skipped and hasattr(report, "wasxfail")
         include_output = output_policy == "all" or (
-            output_policy == "failed-only"
-            and (report.failed or is_expected_failure)
+            output_policy == "failed-only" and (report.failed or is_expected_failure)
         )
         stdout = getattr(report, "capstdout", "") if include_output else ""
         stderr = getattr(report, "capstderr", "") if include_output else ""
 
         reporter.log_result(
-            test_name=test_name,
+            test_name=item.name,
             nodeid=item.nodeid,
             status=status,
             duration=report.duration,