testlens-playwright-reporter 0.2.9 → 0.3.0

package/index.js

@@ -46,15 +46,12 @@ class TestLensReporter {
     if (this.config.ignoreSslErrors) {
       // Explicit configuration option
       rejectUnauthorized = false;
-      console.log('⚠️  SSL certificate validation disabled via ignoreSslErrors option');
     } else if (this.config.rejectUnauthorized === false) {
       // Explicit configuration option
       rejectUnauthorized = false;
-      console.log('⚠️  SSL certificate validation disabled via rejectUnauthorized option');
     } else if (process.env.NODE_TLS_REJECT_UNAUTHORIZED === '0') {
       // Environment variable override
       rejectUnauthorized = false;
-      console.log('⚠️  SSL certificate validation disabled via NODE_TLS_REJECT_UNAUTHORIZED environment variable');
     }
 
     // Set up axios instance with retry logic and enhanced SSL handling
@@ -144,18 +141,9 @@ class TestLensReporter {
   }
 
   async onBegin(config, suite) {
-    console.log(`🚀 TestLens Reporter starting - Run ID: ${this.runId}`);
-    
     // 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
@@ -252,11 +240,8 @@ class TestLensReporter {
     const testId = this.getTestId(test);
     let testData = this.testMap.get(testId);
 
-    console.log(`[TestLens] onTestEnd called for test: ${test.title}, status: ${result.status}, testData exists: ${!!testData}`);
-
     // For skipped tests, onTestBegin might not be called, so we need to create the test data here
     if (!testData) {
-      console.log(`[TestLens] Creating test data for skipped/uncreated test: ${test.title}`);
       // Create spec data if not exists (skipped tests might not have spec data either)
       const specPath = test.location.file;
       const specKey = `${specPath}-${test.parent.title}`;
@@ -422,7 +407,6 @@ class TestLensReporter {
       const isFinalAttempt = result.status === 'passed' || result.status === 'skipped' || result.retry >= test.retries;
       
       if (isFinalAttempt) {
-        console.log(`[TestLens] Sending testEnd - testId: ${testData.id}, status: ${testData.status}, originalStatus: ${testData.originalStatus}`);
         // Send test end event to API
         await this.sendToApi({
           type: 'testEnd',
@@ -508,19 +492,12 @@ class TestLensReporter {
     });
 
     // Wait for background artifact processing to complete (up to 10 seconds)
-    console.log('⏳ Waiting for background artifact processing to complete...');
     await new Promise(resolve => setTimeout(resolve, 10000));
-
-    console.log(`📊 TestLens Report completed - Run ID: ${this.runId}`);
-    console.log(`🎯 Results: ${passedTests} passed, ${failedTests} failed (${timedOutTests} timeouts), ${skippedTests} skipped`);
   }
 
   async sendToApi(payload) {
     try {
       const response = await this.axiosInstance.post('', payload);
-      if (this.config.enableRealTimeStream) {
-        console.log(`✅ Sent ${payload.type} event to TestLens`);
-      }
     } catch (error) {
       console.error(`❌ Failed to send ${payload.type} event to TestLens:`, {
         message: error?.message || 'Unknown error',
@@ -534,19 +511,10 @@ class TestLensReporter {
 
   async processArtifacts(testId, result) {
     const attachments = result.attachments;
-    console.log(`🔍 Processing artifacts for test ${testId}: ${attachments ? attachments.length : 0} attachments found`);
-
-    if (attachments && attachments.length > 0) {
-      console.log('📎 Attachment details:');
-      attachments.forEach((attachment, index) => {
-        console.log(`  ${index + 1}. ${attachment.name} (${attachment.contentType}) - Path: ${attachment.path}`);
-      });
-    }
 
     // Process artifacts with controlled async handling to ensure uploads complete
     if (attachments && attachments.length > 0) {
       // Process all artifacts asynchronously but track completion
-      console.log(`🔄 Processing ${attachments.length} artifacts asynchronously...`);
 
       // Use process.nextTick to defer processing to next event loop iteration
       process.nextTick(async () => {
@@ -560,18 +528,15 @@ class TestLensReporter {
             
             // 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
@@ -608,7 +573,6 @@ class TestLensReporter {
               
               // Skip if upload failed or file was too large
               if (!s3Data) {
-                console.log(`⏭️  Skipping artifact ${attachment.name} - upload failed or file too large`);
                 continue;
               }
               
@@ -631,8 +595,6 @@ class TestLensReporter {
                 timestamp: new Date().toISOString(),
                 artifact: artifactData
               });
-
-              console.log(`📎 Processed artifact: ${fileName} (uploaded to S3)`);
             } catch (error) {
               console.error(`❌ Failed to process ${attachment.name}:`, error.message);
             }
@@ -665,8 +627,6 @@ class TestLensReporter {
         codeBlocks,
         testSuiteName: path.basename(specPath).replace(/\.(spec|test)\.(js|ts)$/, '')
       });
-
-      console.log(`📝 Sent ${codeBlocks.length} code blocks for: ${path.basename(specPath)}`);
     } catch (error) {
       console.error('Failed to send spec code blocks:', error?.response?.data || error?.message || 'Unknown error');
     }
@@ -760,7 +720,6 @@ class TestLensReporter {
         }
       } catch (e) {
         // Remote info is optional - handle gracefully
-        console.log('ℹ️  No git remote configured, skipping remote info');
       }
       
       const isDirty = execSync('git status --porcelain', { encoding: 'utf-8' }).trim().length > 0;
@@ -828,8 +787,6 @@ class TestLensReporter {
       // Check file size first
       const fileSize = this.getFileSize(filePath);
       const fileSizeMB = (fileSize / (1024 * 1024)).toFixed(2);
-      
-      console.log(`📤 Uploading ${fileName} (${fileSizeMB}MB) directly to S3...`);
 
       const baseUrl = this.config.apiEndpoint.replace('/api/v1/webhook/playwright', '');
       
@@ -854,8 +811,6 @@ class TestLensReporter {
       const { uploadUrl, s3Key, metadata } = presignedResponse.data;
       
       // Step 2: Upload directly to S3 using presigned URL
-      console.log(`⬆️  Uploading ${fileName} directly to S3 (bypass server)...`);
-      
       const fileBuffer = fs.readFileSync(filePath);
       
       // IMPORTANT: When using presigned URLs, we MUST include exactly the headers that were signed
@@ -876,8 +831,6 @@ class TestLensReporter {
         throw new Error(`S3 upload failed with status ${uploadResponse.status}`);
       }
 
-      console.log(`✅ S3 direct upload completed for ${fileName}`);
-
       // Step 3: Confirm upload with server to save metadata
       const confirmEndpoint = `${baseUrl}/api/v1/artifacts/public/confirm-upload`;
       const confirmResponse = await this.axiosInstance.post(confirmEndpoint, {
@@ -895,7 +848,6 @@ class TestLensReporter {
 
       if (confirmResponse.status === 201 && confirmResponse.data.success) {
         const artifact = confirmResponse.data.artifact;
-        console.log(`✅ Upload confirmed and saved to database`);
         return {
           key: s3Key,
           url: artifact.s3Url,

package/package.json

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