test-reporting 3.6.1__tar.gz → 3.6.3__tar.gz

{test_reporting-3.6.1/test_reporting.egg-info → test_reporting-3.6.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: test-reporting
-Version: 3.6.1
+Version: 3.6.3
 Summary: Multi-project test reporting dashboard — collect results locally or push to S3
 Home-page: https://github.com/amahdy77/test-reporting.git
 Author: Ashfaqur Mahdy

{test_reporting-3.6.1 → test_reporting-3.6.3}/reporting/plugin.py

@@ -248,6 +248,9 @@ class TestReportingPlugin:
                 logging.info(f"[Reporting] Published run: {run_id}")
             except Exception as e:
                 logging.warning(f"[Reporting] Auto-publish failed: {e}")
+        
+        # Close database connection to prevent ResourceWarning
+        self.storage.close()
     
     def _save_latest_json(self):
         """Save latest run data as JSON for dashboard."""

{test_reporting-3.6.1 → test_reporting-3.6.3}/reporting/publisher.py

@@ -493,8 +493,12 @@ class Publisher:
                 # Regression grouping
                 if tag not in regressions:
                     regressions[tag] = {'tag': tag, 'date': tag[:10], 'suites': [],
-                                        'total_tests': 0, 'passed': 0, 'failed': 0, 'pass_rate': 0}
+                                        'total_tests': 0, 'passed': 0, 'failed': 0, 'pass_rate': 0,
+                                        'timestamp': ''}
                 reg = regressions[tag]
+                # Track the latest timestamp for this regression
+                if ts > reg.get('timestamp', ''):
+                    reg['timestamp'] = ts
                 existing = next((s for s in reg['suites'] if s['suite'] == suite), None)
                 if existing:
                     if ts > existing.get('timestamp', ''):
@@ -525,8 +529,10 @@ class Publisher:
                 s['runs'] = sorted(s['runs'], key=lambda r: r['timestamp'], reverse=True)
                 s['runs'] = s['runs'][:self.config.DASHBOARD_MAX_RUNS_PER_SUITE]
 
-            # Sort regressions newest first
-            regressions = dict(sorted(regressions.items(), key=lambda x: x[0], reverse=True))
+            # Sort regressions by timestamp (newest first), fallback to tag name
+            regressions = dict(sorted(regressions.items(), 
+                                     key=lambda x: (x[1].get('timestamp', ''), x[0]), 
+                                     reverse=True))
 
             # Overall health from latest regression
             last_reg = next(iter(regressions.values()), None) if regressions else None
@@ -745,7 +751,9 @@ class Publisher:
                         s3.put_object(Bucket=bucket, Key=dest_key, Body=f.read(),
                                     ContentType='image/png')
                     print(f"[Reporting] ✓ Uploaded screenshot: {src.name}")
-                    test['screenshot_path'] = f"artifacts/{src.name}"
+                    # Use full S3 URL so it works when dashboard is served from S3
+                    s3_url = f"https://{bucket}.s3.{self.config.S3_REGION}.amazonaws.com/{dest_key}"
+                    test['screenshot_path'] = s3_url
                     artifact_count += 1
                 else:
                     print(f"[Reporting] ✗ Screenshot not found: {src}")
@@ -761,7 +769,9 @@ class Publisher:
                         s3.put_object(Bucket=bucket, Key=dest_key, Body=f.read(),
                                     ContentType='application/zip')
                     print(f"[Reporting] ✓ Uploaded trace: {src.name}")
-                    test['trace_path'] = f"artifacts/{src.name}"
+                    # Use full S3 URL so it works when dashboard is served from S3
+                    s3_url = f"https://{bucket}.s3.{self.config.S3_REGION}.amazonaws.com/{dest_key}"
+                    test['trace_path'] = s3_url
                     artifact_count += 1
                 else:
                     print(f"[Reporting] ✗ Trace not found: {src}")

{test_reporting-3.6.1 → test_reporting-3.6.3}/reporting/storage.py

@@ -18,15 +18,49 @@ class TestResultStorage:
         self.config = config or ReportingConfig()
         self.db_path = db_path or self.config.DB_PATH
         self.db_path.parent.mkdir(parents=True, exist_ok=True)
+        self._connection = None
         self._init_database()
     
+    def __enter__(self):
+        """Context manager entry."""
+        return self
+    
+    def __exit__(self, exc_type, exc_val, exc_tb):
+        """Context manager exit - ensure connection is closed."""
+        self.close()
+        return False
+    
+    def close(self):
+        """Close any open database connections."""
+        if self._connection is not None:
+            try:
+                self._connection.close()
+            except Exception:
+                pass
+            finally:
+                self._connection = None
+    
+    def _execute_with_connection(self, func):
+        """Execute a function with a database connection, ensuring it's closed."""
+        conn = sqlite3.connect(self.db_path)
+        try:
+            result = func(conn)
+            conn.commit()
+            return result
+        finally:
+            conn.close()
+    
     def _init_database(self):
         """Initialize database with WAL mode and create tables."""
-        with sqlite3.connect(self.db_path) as conn:
+        conn = sqlite3.connect(self.db_path)
+        try:
             # Enable WAL mode for concurrent access
             conn.execute('PRAGMA journal_mode=WAL')
             conn.execute('PRAGMA busy_timeout=5000')  # Wait 5s if locked
             self._create_tables(conn)
+            conn.commit()
+        finally:
+            conn.close()
     
     def _create_tables(self, conn: sqlite3.Connection):
         """Create database tables if they don't exist."""
@@ -156,7 +190,8 @@ class TestResultStorage:
     
     def save_test_run(self, run_data: Dict[str, Any]) -> int:
         """Save a test run and return the run ID."""
-        with sqlite3.connect(self.db_path) as conn:
+        conn = sqlite3.connect(self.db_path)
+        try:
             cursor = conn.execute('''
                 INSERT INTO test_runs (
                     project_name, timestamp, suite_name, build_number, build_url, run_tag,
@@ -180,10 +215,13 @@ class TestResultStorage:
             ))
             conn.commit()
             return cursor.lastrowid
+        finally:
+            conn.close()
     
     def save_test_result(self, run_id: int, result_data: Dict[str, Any]) -> int:
         """Save a test result and return the result ID."""
-        with sqlite3.connect(self.db_path) as conn:
+        conn = sqlite3.connect(self.db_path)
+        try:
             cursor = conn.execute('''
                 INSERT INTO test_results (
                     run_id, project_name, file_name, test_name, full_name, status,
@@ -211,6 +249,8 @@ class TestResultStorage:
             ))
             conn.commit()
             return cursor.lastrowid
+        finally:
+            conn.close()
     
     def save_test_step(self, result_id: int, step_data: Dict[str, Any]):
         """Save a test step."""

{test_reporting-3.6.1 → test_reporting-3.6.3}/reporting/templates/project.html

@@ -392,6 +392,14 @@
     return new Date(iso).toLocaleTimeString(undefined, { hour: '2-digit', minute: '2-digit', hour12: false });
   }
 
+  function fmtDateTime(iso) {
+    if (!iso) return '-';
+    const date = new Date(iso);
+    const dateStr = date.toLocaleDateString(undefined, { month: 'short', day: 'numeric', year: 'numeric' });
+    const timeStr = date.toLocaleTimeString(undefined, { hour: '2-digit', minute: '2-digit', hour12: false });
+    return dateStr + ' ' + timeStr;
+  }
+
   function fmtDur(s) {
     if (!s || s === 0) return '-';
     if (s >= 60) return Math.floor(s / 60) + 'm ' + Math.floor(s % 60) + 's';
@@ -553,10 +561,12 @@
           }).join('');
 
           const isOpen = i === 0;
+          const regDateTime = reg.timestamp ? fmtDateTime(reg.timestamp) : '';
+          const regTitle = regDateTime ? reg.tag + ' · ' + regDateTime : reg.tag;
           return '<div class="regression">' +
             '<div class="reg-head" onclick="toggleReg(this)">' +
               '<div>' +
-                '<div class="reg-title">' + reg.tag + '</div>' +
+                '<div class="reg-title">' + regTitle + '</div>' +
                 '<div class="reg-sub">' + reg.suites.length + ' test ' + plural(reg.suites.length, 'file') + ' · ' + reg.total_tests + ' tests</div>' +
               '</div>' +
               '<div class="reg-right">' +
@@ -1207,7 +1217,9 @@
     }
 
     document.getElementById('pageTitle').textContent = proj.name;
-    document.getElementById('lastRun').textContent   = 'Last run ' + relTime(proj.last_run);
+    const lastRunEl = document.getElementById('lastRun');
+    lastRunEl.textContent = 'Last run ' + relTime(proj.last_run);
+    lastRunEl.title = 'All times shown in your local timezone';
     document.title = proj.name + ' - Test Dashboard';
 
     const analytics = proj.analytics || {};

{test_reporting-3.6.1 → test_reporting-3.6.3}/reporting/templates/run.html

@@ -413,7 +413,9 @@
     document.title = (run.suite_name || 'Run') + ' - Test Dashboard';
     document.getElementById('pageTitle').textContent =
       (run.suite_name || 'Run') + '  ·  ' + (run.project_name || '');
-    document.getElementById('runTime').textContent = fmtDate(run.timestamp);
+    var runTimeEl = document.getElementById('runTime');
+    runTimeEl.textContent = fmtDate(run.timestamp);
+    runTimeEl.title = 'Time shown in your local timezone';
 
     var backHref = projectName
       ? 'project.html?p=' + encodeURIComponent(projectName)

{test_reporting-3.6.1 → test_reporting-3.6.3}/setup.py

@@ -9,7 +9,7 @@ with open("README.md", "r", encoding="utf-8") as fh:
 
 setup(
     name='test-reporting',
-    version='3.6.1',
+    version='3.6.3',
     description='Multi-project test reporting dashboard — collect results locally or push to S3',
     long_description=long_description,
     long_description_content_type="text/markdown",

{test_reporting-3.6.1 → test_reporting-3.6.3/test_reporting.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: test-reporting
-Version: 3.6.1
+Version: 3.6.3
 Summary: Multi-project test reporting dashboard — collect results locally or push to S3
 Home-page: https://github.com/amahdy77/test-reporting.git
 Author: Ashfaqur Mahdy