playwright-slack-report-burak 3.0.28 → 3.0.30

package/dist/src/ResultsParser.js

@@ -48,6 +48,66 @@ class ResultsParser {
         // Log package version
         console.log(`📦 [ResultsParser] playwright-slack-report-burak v${packageVersion}`);
     }
+    /**
+     * Recursively print directory tree structure
+     * @param {string} dirPath - Directory path to print
+     * @param {string} prefix - Prefix for tree visualization
+     * @param {number} maxDepth - Maximum depth to traverse (default: 5)
+     */
+    printDirectoryTree(dirPath, prefix = '', maxDepth = 5) {
+      if (maxDepth <= 0) return;
+      if (!fs.existsSync(dirPath)) {
+        console.log(`${prefix}${path.basename(dirPath)}/ (does not exist)`);
+        return;
+      }
+      
+      try {
+        const stats = fs.statSync(dirPath);
+        if (!stats.isDirectory()) {
+          const size = stats.size;
+          const sizeStr = size > 1024 ? `${(size / 1024).toFixed(2)}KB` : `${size}B`;
+          console.log(`${prefix}${path.basename(dirPath)} (${sizeStr})`);
+          return;
+        }
+        
+        const dirName = path.basename(dirPath) || dirPath;
+        console.log(`${prefix}${dirName}/`);
+        
+        const entries = fs.readdirSync(dirPath, { withFileTypes: true });
+        entries.sort((a, b) => {
+          if (a.isDirectory() && !b.isDirectory()) return -1;
+          if (!a.isDirectory() && b.isDirectory()) return 1;
+          return a.name.localeCompare(b.name);
+        });
+        
+        entries.forEach((entry, index) => {
+          const isLast = index === entries.length - 1;
+          const newPrefix = prefix + (isLast ? '└── ' : '├── ');
+          const nextPrefix = prefix + (isLast ? '    ' : '│   ');
+          const fullPath = path.join(dirPath, entry.name);
+          
+          if (entry.isDirectory()) {
+            this.printDirectoryTree(fullPath, nextPrefix, maxDepth - 1);
+          } else {
+            try {
+              const fileStats = fs.statSync(fullPath);
+              const size = fileStats.size;
+              const sizeStr = size > 1024 * 1024 
+                ? `${(size / (1024 * 1024)).toFixed(2)}MB` 
+                : size > 1024 
+                  ? `${(size / 1024).toFixed(2)}KB` 
+                  : `${size}B`;
+              console.log(`${newPrefix}${entry.name} (${sizeStr})`);
+            } catch (err) {
+              console.log(`${newPrefix}${entry.name} (error reading)`);
+            }
+          }
+        });
+      } catch (error) {
+        console.log(`${prefix}${path.basename(dirPath)}/ (error: ${error.message})`);
+      }
+    }
+
     /**
      * Update shard information after detection (e.g., from GitHub API)
      * @param {number} shardIndex - The current shard index (1-based, shard 1 is aggregator)
@@ -570,12 +630,24 @@ class ResultsParser {
         }
     }
     async mergeReports() {
-      let breakMergeWaiter = false;
       try {
         const summariesDir = path.join('./', 'playwright-report');
         const blobReportsDir = path.join(summariesDir, 'blob-reports');
         const mergedBlobsDir = path.join(summariesDir, 'mergedBlobsDir');
         
+        console.log('\n=== Starting merge process ===');
+        console.log(`Summaries directory: ${path.resolve(summariesDir)}`);
+        console.log(`Blob reports directory: ${path.resolve(blobReportsDir)}`);
+        console.log(`Merged blobs directory: ${path.resolve(mergedBlobsDir)}`);
+        
+        // Show initial directory structure
+        console.log('\n📁 Initial playwright-report directory structure:');
+        if (fs.existsSync(summariesDir)) {
+          this.printDirectoryTree(summariesDir);
+        } else {
+          console.log('  (directory does not exist yet)');
+        }
+        
         // Create directory for extracted blob reports
         if (!fs.existsSync(blobReportsDir)) {
           fs.mkdirSync(blobReportsDir, { recursive: true });
@@ -587,7 +659,7 @@ class ResultsParser {
         }
         
         // Extract all blob zips to preserve traces
-        console.log('Extracting blob reports to preserve traces...');
+        console.log('\n📦 Extracting blob reports to preserve traces...');
         let hasBlobReports = false;
         for (let i = 1; i <= this.totalShardCount; i++) {
           const blobZipFile = path.join(summariesDir, `blob-report-node-${i}.zip`);
@@ -605,20 +677,70 @@ class ResultsParser {
           }
         }
         
+        // Show directory structure after extraction
+        if (hasBlobReports) {
+          console.log('\n📁 Directory structure after blob extraction:');
+          this.printDirectoryTree(summariesDir);
+        }
+        
         // Merge blob reports instead of HTML reports to preserve traces
         if (hasBlobReports && fs.existsSync(blobReportsDir) && fs.readdirSync(blobReportsDir).length > 0) {
-          console.log('Merging blob reports (preserves traces)...');
+          console.log('\n🔄 Merging blob reports (preserves traces)...');
+          let tempConfigPath = null;
           try {
-            // Merge blob reports - use relative path from summariesDir and output to mergedBlobsDir
+            // Merge blob reports - use config file to specify output directory
             // This prevents wiping the playwright-report directory
             const blobReportsRelativePath = path.relative(summariesDir, blobReportsDir);
-            const mergedBlobsRelativePath = path.relative(summariesDir, mergedBlobsDir);
-            await execAsync(`npx playwright merge-reports --reporter html --output-dir "${mergedBlobsRelativePath}" "${blobReportsRelativePath}"`, { cwd: summariesDir });
+            const mergedBlobsAbsolutePath = path.resolve(mergedBlobsDir);
+            console.log(`Merging blob reports from: ${blobReportsRelativePath}`);
+            console.log(`Output directory: ${mergedBlobsAbsolutePath}`);
+            
+            // Create temporary config file to specify output folder
+            tempConfigPath = path.join(summariesDir, 'merge-config.js');
+            // Escape backslashes and single quotes in path for JavaScript string
+            const escapedPath = mergedBlobsAbsolutePath.replace(/\\/g, '/').replace(/'/g, "\\'");
+            const configContent = `module.exports = {
+  reporter: [['html', { outputFolder: '${escapedPath}' }]]
+};`;
+            fs.writeFileSync(tempConfigPath, configContent);
+            console.log(`Created temporary config file: ${tempConfigPath}`);
+            console.log(`Config content: ${configContent}`);
+            
+            const mergeCommand = `npx playwright merge-reports --reporter html --config "${tempConfigPath}" "${blobReportsRelativePath}"`;
+            console.log(`Executing: ${mergeCommand}`);
+            const mergeResult = await execAsync(mergeCommand, { cwd: summariesDir });
+            console.log('Merge command output:', mergeResult.stdout);
+            if (mergeResult.stderr) {
+              console.warn('Merge command stderr:', mergeResult.stderr);
+            }
             console.log('Blob reports merged successfully with traces preserved.');
+            
+            // Show directory structure after merge
+            console.log('\n📁 Directory structure after merge:');
+            this.printDirectoryTree(summariesDir);
+            console.log('\n📁 Merged blobs directory contents:');
+            this.printDirectoryTree(mergedBlobsDir);
           } catch (error) {
             console.warn('Warning: Failed to merge blob reports. Falling back to HTML merge.', error.message);
+            if (error.stdout) console.log('Merge stdout:', error.stdout);
+            if (error.stderr) console.warn('Merge stderr:', error.stderr);
+            
+            // Show directory structure after failed merge
+            console.log('\n📁 Directory structure after failed merge:');
+            this.printDirectoryTree(summariesDir);
+            
             // Fallback to HTML merge
             execSync(`npx playwright merge-reports --reporter html -c ./playwright-report playwright-report`, { stdio: 'inherit' });
+          } finally {
+            // Clean up temporary config file
+            if (tempConfigPath && fs.existsSync(tempConfigPath)) {
+              try {
+                fs.unlinkSync(tempConfigPath);
+                console.log(`Cleaned up temporary config file: ${tempConfigPath}`);
+              } catch (cleanupError) {
+                console.warn(`Warning: Failed to clean up temp config file: ${cleanupError.message}`);
+              }
+            }
           }
         } else {
           // Fallback to HTML merge if no blob reports found
@@ -626,18 +748,40 @@ class ResultsParser {
           execSync(`npx playwright merge-reports --reporter html -c ./playwright-report playwright-report`, { stdio: 'inherit' });
         }
         
-        // Wait until index.html exists in mergedBlobsDir
-        while (!fs.existsSync(path.join(mergedBlobsDir, '/index.html'))) {
-          console.log('Waiting 2 seconds for merged html report to be generated...');
-          const currentTime = new Date().getTime();
-          breakMergeWaiter = false;
-          while (!breakMergeWaiter) {
-            if(new Date().getTime() - currentTime > 2000) {
-              breakMergeWaiter = true;
+        // Wait until index.html exists in mergedBlobsDir (with timeout)
+        const maxWaitTime = 60000; // 60 seconds max wait
+        const startWaitTime = new Date().getTime();
+        const indexHtmlPath = path.join(mergedBlobsDir, 'index.html');
+        
+        console.log('\n⏳ Waiting for merged HTML report to be generated...');
+        while (!fs.existsSync(indexHtmlPath)) {
+          const elapsed = new Date().getTime() - startWaitTime;
+          if (elapsed > maxWaitTime) {
+            console.warn(`\n⏱️  Timeout: index.html not found in ${mergedBlobsDir} after ${maxWaitTime}ms`);
+            console.log(`Checking if directory exists: ${fs.existsSync(mergedBlobsDir)}`);
+            if (fs.existsSync(mergedBlobsDir)) {
+              console.log('\n📁 Full directory tree at timeout:');
+              this.printDirectoryTree(summariesDir);
+              console.log('\n📁 Merged blobs directory detailed contents:');
+              this.printDirectoryTree(mergedBlobsDir);
+            } else {
+              console.log('Merged blobs directory does not exist!');
             }
+            break;
           }
+          console.log(`Waiting for merged html report to be generated... (${Math.round(elapsed/1000)}s elapsed)`);
+          await new Promise(resolve => setTimeout(resolve, 2000));
+        }
+        
+        if (fs.existsSync(indexHtmlPath)) {
+          console.log('\n✅ Reports merged successfully.');
+          console.log('\n📁 Final directory structure:');
+          this.printDirectoryTree(summariesDir);
+        } else {
+          console.warn('\n⚠️  Warning: Merged report index.html not found, but continuing...');
+          console.log('\n📁 Final directory structure:');
+          this.printDirectoryTree(summariesDir);
         }
-        console.log('Reports merged successfully.');
       } catch (error) {
           // Log a warning instead of throwing an error
           console.warn('Warning: Failed to merge reports. This may not affect the overall process.', error.message);

package/package.json

@@ -32,7 +32,7 @@
     "lint-fix": "npx eslint . --ext .ts --fix"
   },
   "name": "playwright-slack-report-burak",
-  "version": "3.0.28",
+  "version": "3.0.30",
   "main": "index.js",
   "types": "dist/index.d.ts",
   "author": "Burak B. <burak.boluk@hotmail.com>",