testlens-playwright-reporter 0.2.5 → 0.2.7

package/index.d.ts

@@ -11,6 +11,10 @@ export interface TestLensReporterConfig {
   enableGitInfo?: boolean;
   /** Enable artifact processing */
   enableArtifacts?: boolean;
+  /** Enable video capture - defaults to true */
+  enableVideo?: boolean;
+  /** Enable screenshot capture - defaults to true */
+  enableScreenshot?: boolean;
   /** Batch size for API requests */
   batchSize?: number;
   /** Flush interval in milliseconds */
@@ -36,6 +40,10 @@ export interface TestLensReporterOptions {
   enableGitInfo?: boolean;
   /** Enable artifact processing */
   enableArtifacts?: boolean;
+  /** Enable video capture - defaults to true */
+  enableVideo?: boolean;
+  /** Enable screenshot capture - defaults to true */
+  enableScreenshot?: boolean;
   /** Batch size for API requests */
   batchSize?: number;
   /** Flush interval in milliseconds */
@@ -55,8 +63,8 @@ export interface GitInfo {
   commit: string;
   shortCommit: string;
   author: string;
-  commitMessage: string;
-  commitTimestamp: string;
+  message: string;
+  timestamp: string;
   isDirty: boolean;
   remoteName: string;
   remoteUrl: string;

package/index.js

@@ -27,6 +27,8 @@ class TestLensReporter {
       enableRealTimeStream: options.enableRealTimeStream !== undefined ? options.enableRealTimeStream : true,
       enableGitInfo: options.enableGitInfo !== undefined ? options.enableGitInfo : true,
       enableArtifacts: options.enableArtifacts !== undefined ? options.enableArtifacts : true,
+      enableVideo: options.enableVideo !== undefined ? options.enableVideo : true, // Default to true, override from config
+      enableScreenshot: options.enableScreenshot !== undefined ? options.enableScreenshot : true, // Default to true, override from config
       batchSize: options.batchSize || 10,
       flushInterval: options.flushInterval || 5000,
       retryAttempts: options.retryAttempts !== undefined ? options.retryAttempts : 0,
@@ -147,6 +149,13 @@ class TestLensReporter {
     // Collect Git information if enabled
     if (this.config.enableGitInfo) {
       this.runMetadata.gitInfo = await this.collectGitInfo();
+      if (this.runMetadata.gitInfo) {
+        console.log(`📦 Git info collected: branch=${this.runMetadata.gitInfo.branch}, commit=${this.runMetadata.gitInfo.shortCommit}, author=${this.runMetadata.gitInfo.author}`);
+      } else {
+        console.log(`⚠️  Git info collection returned null - not in a git repository or git not available`);
+      }
+    } else {
+      console.log(`ℹ️  Git info collection disabled (enableGitInfo: false)`);
     }
 
     // Add shard information if available
@@ -309,8 +318,10 @@ class TestLensReporter {
     // Calculate final stats
     const totalTests = Array.from(this.testMap.values()).length;
     const passedTests = Array.from(this.testMap.values()).filter(t => t.status === 'passed').length;
+    // failedTests already includes timedOut tests since normalizeTestStatus converts 'timedOut' to 'failed'
     const failedTests = Array.from(this.testMap.values()).filter(t => t.status === 'failed').length;
     const skippedTests = Array.from(this.testMap.values()).filter(t => t.status === 'skipped').length;
+    // Track timedOut separately for reporting purposes only (not for count)
     const timedOutTests = Array.from(this.testMap.values()).filter(t => t.originalStatus === 'timedOut').length;
 
     // Normalize run status - if there are timeouts, treat run as failed
@@ -326,9 +337,9 @@ class TestLensReporter {
         ...this.runMetadata,
         totalTests,
         passedTests,
-        failedTests: failedTests + timedOutTests, // Include timeouts in failed count
+        failedTests, // Already includes timedOut tests (normalized to 'failed')
         skippedTests,
-        timedOutTests,
+        timedOutTests, // For informational purposes
         status: normalizedRunStatus
       }
     });
@@ -338,7 +349,7 @@ class TestLensReporter {
     await new Promise(resolve => setTimeout(resolve, 10000));
 
     console.log(`📊 TestLens Report completed - Run ID: ${this.runId}`);
-    console.log(`🎯 Results: ${passedTests} passed, ${failedTests + timedOutTests} failed (${timedOutTests} timeouts), ${skippedTests} skipped`);
+    console.log(`🎯 Results: ${passedTests} passed, ${failedTests} failed (${timedOutTests} timeouts), ${skippedTests} skipped`);
   }
 
   async sendToApi(payload) {
@@ -379,17 +390,70 @@ class TestLensReporter {
         for (let i = 0; i < attachments.length; i++) {
           const attachment = attachments[i];
           if (attachment.path) {
+            // Check if attachment should be processed based on config
+            const artifactType = this.getArtifactType(attachment.name);
+            const isVideo = artifactType === 'video' || (attachment.contentType && attachment.contentType.startsWith('video/'));
+            const isScreenshot = artifactType === 'screenshot' || (attachment.contentType && attachment.contentType.startsWith('image/'));
+            
+            // Skip video if disabled in config
+            if (isVideo && !this.config.enableVideo) {
+              console.log(`⏭️  Skipping video artifact ${attachment.name} - video capture disabled in config`);
+              continue;
+            }
+            
+            // Skip screenshot if disabled in config
+            if (isScreenshot && !this.config.enableScreenshot) {
+              console.log(`⏭️  Skipping screenshot artifact ${attachment.name} - screenshot capture disabled in config`);
+              continue;
+            }
+            
             try {
               console.log(`📤 Processing ${attachment.name} asynchronously...`);
               
+              // Determine proper filename with extension
+              // Playwright attachment.name often doesn't have extension, so we need to derive it
+              let fileName = attachment.name;
+              const existingExt = path.extname(fileName);
+              
+              if (!existingExt) {
+                // Get extension from the actual file path
+                const pathExt = path.extname(attachment.path);
+                if (pathExt) {
+                  fileName = `${fileName}${pathExt}`;
+                } else if (attachment.contentType) {
+                  // Fallback: derive extension from contentType
+                  const mimeToExt = {
+                    'image/png': '.png',
+                    'image/jpeg': '.jpg',
+                    'image/gif': '.gif',
+                    'image/webp': '.webp',
+                    'video/webm': '.webm',
+                    'video/mp4': '.mp4',
+                    'application/zip': '.zip',
+                    'application/json': '.json',
+                    'text/plain': '.txt'
+                  };
+                  const ext = mimeToExt[attachment.contentType];
+                  if (ext) {
+                    fileName = `${fileName}${ext}`;
+                  }
+                }
+              }
+              
               // Upload to S3 first
-              const s3Data = await this.uploadArtifactToS3(attachment.path, testId, attachment.name);
+              const s3Data = await this.uploadArtifactToS3(attachment.path, testId, fileName);
+              
+              // Skip if upload failed or file was too large
+              if (!s3Data) {
+                console.log(`⏭️  Skipping artifact ${attachment.name} - upload failed or file too large`);
+                continue;
+              }
               
               const artifactData = {
                 testId,
                 type: this.getArtifactType(attachment.name),
                 path: attachment.path,
-                name: attachment.name,
+                name: fileName,
                 contentType: attachment.contentType,
                 fileSize: this.getFileSize(attachment.path),
                 storageType: 's3',
@@ -405,7 +469,7 @@ class TestLensReporter {
                 artifact: artifactData
               });
 
-              console.log(`📎 Processed artifact: ${attachment.name} (uploaded to S3)`);
+              console.log(`📎 Processed artifact: ${fileName} (uploaded to S3)`);
             } catch (error) {
               console.error(`❌ Failed to process ${attachment.name}:`, error.message);
             }
@@ -543,8 +607,8 @@ class TestLensReporter {
         commit,
         shortCommit,
         author,
-        commitMessage,
-        commitTimestamp,
+        message: commitMessage,
+        timestamp: commitTimestamp,
         isDirty,
         remoteName,
         remoteUrl

package/index.ts

@@ -32,6 +32,10 @@ export interface TestLensReporterConfig {
   enableGitInfo?: boolean;
   /** Enable artifact processing */
   enableArtifacts?: boolean;
+  /** Enable video capture - defaults to true */
+  enableVideo?: boolean;
+  /** Enable screenshot capture - defaults to true */
+  enableScreenshot?: boolean;
   /** Batch size for API requests */
   batchSize?: number;
   /** Flush interval in milliseconds */
@@ -57,6 +61,10 @@ export interface TestLensReporterOptions {
   enableGitInfo?: boolean;
   /** Enable artifact processing */
   enableArtifacts?: boolean;
+  /** Enable video capture - defaults to true */
+  enableVideo?: boolean;
+  /** Enable screenshot capture - defaults to true */
+  enableScreenshot?: boolean;
   /** Batch size for API requests */
   batchSize?: number;
   /** Flush interval in milliseconds */
@@ -76,8 +84,8 @@ export interface GitInfo {
   commit: string;
   shortCommit: string;
   author: string;
-  commitMessage: string;
-  commitTimestamp: string;
+  message: string;
+  timestamp: string;
   isDirty: boolean;
   remoteName: string;
   remoteUrl: string;
@@ -156,6 +164,8 @@ export class TestLensReporter implements Reporter {
       enableRealTimeStream: options.enableRealTimeStream !== undefined ? options.enableRealTimeStream : true,
       enableGitInfo: options.enableGitInfo !== undefined ? options.enableGitInfo : true,
       enableArtifacts: options.enableArtifacts !== undefined ? options.enableArtifacts : true,
+      enableVideo: options.enableVideo !== undefined ? options.enableVideo : true, // Default to true, override from config
+      enableScreenshot: options.enableScreenshot !== undefined ? options.enableScreenshot : true, // Default to true, override from config
       batchSize: options.batchSize || 10,
       flushInterval: options.flushInterval || 5000,
       retryAttempts: options.retryAttempts !== undefined ? options.retryAttempts : 0,
@@ -276,6 +286,13 @@ export class TestLensReporter implements Reporter {
     // Collect Git information if enabled
     if (this.config.enableGitInfo) {
       this.runMetadata.gitInfo = await this.collectGitInfo();
+      if (this.runMetadata.gitInfo) {
+        console.log(`📦 Git info collected: branch=${this.runMetadata.gitInfo.branch}, commit=${this.runMetadata.gitInfo.shortCommit}, author=${this.runMetadata.gitInfo.author}`);
+      } else {
+        console.log(`⚠️  Git info collection returned null - not in a git repository or git not available`);
+      }
+    } else {
+      console.log(`ℹ️  Git info collection disabled (enableGitInfo: false)`);
     }
 
     // Add shard information if available
@@ -438,8 +455,10 @@ export class TestLensReporter implements Reporter {
     // Calculate final stats
     const totalTests = Array.from(this.testMap.values()).length;
     const passedTests = Array.from(this.testMap.values()).filter(t => t.status === 'passed').length;
+    // failedTests already includes timedOut tests since normalizeTestStatus converts 'timedOut' to 'failed'
     const failedTests = Array.from(this.testMap.values()).filter(t => t.status === 'failed').length;
     const skippedTests = Array.from(this.testMap.values()).filter(t => t.status === 'skipped').length;
+    // Track timedOut separately for reporting purposes only (not for count)
     const timedOutTests = Array.from(this.testMap.values()).filter(t => t.originalStatus === 'timedOut').length;
 
     // Normalize run status - if there are timeouts, treat run as failed
@@ -455,15 +474,15 @@ export class TestLensReporter implements Reporter {
         ...this.runMetadata,
         totalTests,
         passedTests,
-        failedTests: failedTests + timedOutTests, // Include timeouts in failed count
+        failedTests, // Already includes timedOut tests (normalized to 'failed')
         skippedTests,
-        timedOutTests,
+        timedOutTests, // For informational purposes
         status: normalizedRunStatus
       }
     });
 
     console.log(`📊 TestLens Report completed - Run ID: ${this.runId}`);
-    console.log(`🎯 Results: ${passedTests} passed, ${failedTests + timedOutTests} failed (${timedOutTests} timeouts), ${skippedTests} skipped`);
+    console.log(`🎯 Results: ${passedTests} passed, ${failedTests} failed (${timedOutTests} timeouts), ${skippedTests} skipped`);
   }
 
   private async sendToApi(payload: any): Promise<void> {
@@ -493,9 +512,56 @@ export class TestLensReporter implements Reporter {
     
     for (const attachment of attachments) {
       if (attachment.path) {
+        // Check if attachment should be processed based on config
+        const artifactType = this.getArtifactType(attachment.name);
+        const isVideo = artifactType === 'video' || attachment.contentType?.startsWith('video/');
+        const isScreenshot = artifactType === 'screenshot' || attachment.contentType?.startsWith('image/');
+        
+        // Skip video if disabled in config
+        if (isVideo && !this.config.enableVideo) {
+          console.log(`⏭️  Skipping video artifact ${attachment.name} - video capture disabled in config`);
+          continue;
+        }
+        
+        // Skip screenshot if disabled in config
+        if (isScreenshot && !this.config.enableScreenshot) {
+          console.log(`⏭️  Skipping screenshot artifact ${attachment.name} - screenshot capture disabled in config`);
+          continue;
+        }
+        
         try {
+          // Determine proper filename with extension
+          // Playwright attachment.name often doesn't have extension, so we need to derive it
+          let fileName = attachment.name;
+          const existingExt = path.extname(fileName);
+          
+          if (!existingExt) {
+            // Get extension from the actual file path
+            const pathExt = path.extname(attachment.path);
+            if (pathExt) {
+              fileName = `${fileName}${pathExt}`;
+            } else if (attachment.contentType) {
+              // Fallback: derive extension from contentType
+              const mimeToExt: Record<string, string> = {
+                'image/png': '.png',
+                'image/jpeg': '.jpg',
+                'image/gif': '.gif',
+                'image/webp': '.webp',
+                'video/webm': '.webm',
+                'video/mp4': '.mp4',
+                'application/zip': '.zip',
+                'application/json': '.json',
+                'text/plain': '.txt'
+              };
+              const ext = mimeToExt[attachment.contentType];
+              if (ext) {
+                fileName = `${fileName}${ext}`;
+              }
+            }
+          }
+
           // Upload to S3 first
-          const s3Data = await this.uploadArtifactToS3(attachment.path, testId, attachment.name);
+          const s3Data = await this.uploadArtifactToS3(attachment.path, testId, fileName);
           
           // Skip if upload failed or file was too large
           if (!s3Data) {
@@ -507,7 +573,7 @@ export class TestLensReporter implements Reporter {
             testId,
             type: this.getArtifactType(attachment.name),
             path: attachment.path,
-            name: attachment.name,
+            name: fileName,
             contentType: attachment.contentType,
             fileSize: this.getFileSize(attachment.path),
             storageType: 's3',
@@ -523,7 +589,7 @@ export class TestLensReporter implements Reporter {
             artifact: artifactData
           });
           
-          console.log(`📎 Processed artifact: ${attachment.name} (uploaded to S3)`);
+          console.log(`📎 Processed artifact: ${fileName} (uploaded to S3)`);
         } catch (error) {
           console.error(`❌ Failed to process artifact ${attachment.name}:`, (error as Error).message);
         }
@@ -657,8 +723,8 @@ export class TestLensReporter implements Reporter {
         commit,
         shortCommit,
         author,
-        commitMessage,
-        commitTimestamp,
+        message: commitMessage,
+        timestamp: commitTimestamp,
         isDirty,
         remoteName,
         remoteUrl

package/lib/index.js

@@ -63,6 +63,8 @@ class TestLensReporter {
             enableRealTimeStream: options.enableRealTimeStream !== undefined ? options.enableRealTimeStream : true,
             enableGitInfo: options.enableGitInfo !== undefined ? options.enableGitInfo : true,
             enableArtifacts: options.enableArtifacts !== undefined ? options.enableArtifacts : true,
+            enableVideo: options.enableVideo !== undefined ? options.enableVideo : true, // Default to true, override from config
+            enableScreenshot: options.enableScreenshot !== undefined ? options.enableScreenshot : true, // Default to true, override from config
             batchSize: options.batchSize || 10,
             flushInterval: options.flushInterval || 5000,
             retryAttempts: options.retryAttempts !== undefined ? options.retryAttempts : 0,
@@ -168,6 +170,15 @@ class TestLensReporter {
         // Collect Git information if enabled
         if (this.config.enableGitInfo) {
             this.runMetadata.gitInfo = await this.collectGitInfo();
+            if (this.runMetadata.gitInfo) {
+                console.log(`📦 Git info collected: branch=${this.runMetadata.gitInfo.branch}, commit=${this.runMetadata.gitInfo.shortCommit}, author=${this.runMetadata.gitInfo.author}`);
+            }
+            else {
+                console.log(`⚠️  Git info collection returned null - not in a git repository or git not available`);
+            }
+        }
+        else {
+            console.log(`ℹ️  Git info collection disabled (enableGitInfo: false)`);
         }
         // Add shard information if available
         if (config.shard) {
@@ -311,8 +322,10 @@ class TestLensReporter {
         // Calculate final stats
         const totalTests = Array.from(this.testMap.values()).length;
         const passedTests = Array.from(this.testMap.values()).filter(t => t.status === 'passed').length;
+        // failedTests already includes timedOut tests since normalizeTestStatus converts 'timedOut' to 'failed'
         const failedTests = Array.from(this.testMap.values()).filter(t => t.status === 'failed').length;
         const skippedTests = Array.from(this.testMap.values()).filter(t => t.status === 'skipped').length;
+        // Track timedOut separately for reporting purposes only (not for count)
         const timedOutTests = Array.from(this.testMap.values()).filter(t => t.originalStatus === 'timedOut').length;
         // Normalize run status - if there are timeouts, treat run as failed
         const hasTimeouts = timedOutTests > 0;
@@ -326,14 +339,14 @@ class TestLensReporter {
                 ...this.runMetadata,
                 totalTests,
                 passedTests,
-                failedTests: failedTests + timedOutTests, // Include timeouts in failed count
+                failedTests, // Already includes timedOut tests (normalized to 'failed')
                 skippedTests,
-                timedOutTests,
+                timedOutTests, // For informational purposes
                 status: normalizedRunStatus
             }
         });
         console.log(`📊 TestLens Report completed - Run ID: ${this.runId}`);
-        console.log(`🎯 Results: ${passedTests} passed, ${failedTests + timedOutTests} failed (${timedOutTests} timeouts), ${skippedTests} skipped`);
+        console.log(`🎯 Results: ${passedTests} passed, ${failedTests} failed (${timedOutTests} timeouts), ${skippedTests} skipped`);
     }
     async sendToApi(payload) {
         try {
@@ -360,9 +373,52 @@ class TestLensReporter {
         const attachments = result.attachments;
         for (const attachment of attachments) {
             if (attachment.path) {
+                // Check if attachment should be processed based on config
+                const artifactType = this.getArtifactType(attachment.name);
+                const isVideo = artifactType === 'video' || attachment.contentType?.startsWith('video/');
+                const isScreenshot = artifactType === 'screenshot' || attachment.contentType?.startsWith('image/');
+                // Skip video if disabled in config
+                if (isVideo && !this.config.enableVideo) {
+                    console.log(`⏭️  Skipping video artifact ${attachment.name} - video capture disabled in config`);
+                    continue;
+                }
+                // Skip screenshot if disabled in config
+                if (isScreenshot && !this.config.enableScreenshot) {
+                    console.log(`⏭️  Skipping screenshot artifact ${attachment.name} - screenshot capture disabled in config`);
+                    continue;
+                }
                 try {
+                    // Determine proper filename with extension
+                    // Playwright attachment.name often doesn't have extension, so we need to derive it
+                    let fileName = attachment.name;
+                    const existingExt = path.extname(fileName);
+                    if (!existingExt) {
+                        // Get extension from the actual file path
+                        const pathExt = path.extname(attachment.path);
+                        if (pathExt) {
+                            fileName = `${fileName}${pathExt}`;
+                        }
+                        else if (attachment.contentType) {
+                            // Fallback: derive extension from contentType
+                            const mimeToExt = {
+                                'image/png': '.png',
+                                'image/jpeg': '.jpg',
+                                'image/gif': '.gif',
+                                'image/webp': '.webp',
+                                'video/webm': '.webm',
+                                'video/mp4': '.mp4',
+                                'application/zip': '.zip',
+                                'application/json': '.json',
+                                'text/plain': '.txt'
+                            };
+                            const ext = mimeToExt[attachment.contentType];
+                            if (ext) {
+                                fileName = `${fileName}${ext}`;
+                            }
+                        }
+                    }
                     // Upload to S3 first
-                    const s3Data = await this.uploadArtifactToS3(attachment.path, testId, attachment.name);
+                    const s3Data = await this.uploadArtifactToS3(attachment.path, testId, fileName);
                     // Skip if upload failed or file was too large
                     if (!s3Data) {
                         console.log(`⏭️  Skipping artifact ${attachment.name} - upload failed or file too large`);
@@ -372,7 +428,7 @@ class TestLensReporter {
                         testId,
                         type: this.getArtifactType(attachment.name),
                         path: attachment.path,
-                        name: attachment.name,
+                        name: fileName,
                         contentType: attachment.contentType,
                         fileSize: this.getFileSize(attachment.path),
                         storageType: 's3',
@@ -386,7 +442,7 @@ class TestLensReporter {
                         timestamp: new Date().toISOString(),
                         artifact: artifactData
                     });
-                    console.log(`📎 Processed artifact: ${attachment.name} (uploaded to S3)`);
+                    console.log(`📎 Processed artifact: ${fileName} (uploaded to S3)`);
                 }
                 catch (error) {
                     console.error(`❌ Failed to process artifact ${attachment.name}:`, error.message);
@@ -507,8 +563,8 @@ class TestLensReporter {
                 commit,
                 shortCommit,
                 author,
-                commitMessage,
-                commitTimestamp,
+                message: commitMessage,
+                timestamp: commitTimestamp,
                 isDirty,
                 remoteName,
                 remoteUrl

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "testlens-playwright-reporter",
-  "version": "0.2.5",
+  "version": "0.2.7",
   "description": "Universal Playwright reporter for TestLens - works with both TypeScript and JavaScript projects",
   "main": "index.js",
   "types": "index.d.ts",