testlens-playwright-reporter 0.2.6 → 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 */

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,
@@ -316,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
@@ -333,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
       }
     });
@@ -345,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) {
@@ -386,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',
@@ -412,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);
             }

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 */
@@ -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,
@@ -445,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
@@ -462,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> {
@@ -500,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) {
@@ -514,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',
@@ -530,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);
         }

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,
@@ -320,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;
@@ -335,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 {
@@ -369,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`);
@@ -381,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',
@@ -395,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);

package/package.json

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