testlens-playwright-reporter 0.3.4 → 0.3.5

package/index.js

@@ -41,8 +41,15 @@ class TestLensReporter {
             for (const envVar of envVars) {
                 const value = process.env[envVar];
                 if (value) {
-                    customArgs[key] = value;
-                    console.log(`✓ Found ${envVar}=${value} (mapped to '${key}')`);
+                    // For testlensBuildTag, support comma-separated values
+                    if (key === 'testlensBuildTag' && value.includes(',')) {
+                        customArgs[key] = value.split(',').map(tag => tag.trim()).filter(tag => tag);
+                        console.log(`✓ Found ${envVar}=${value} (mapped to '${key}' as array of ${customArgs[key].length} tags)`);
+                    }
+                    else {
+                        customArgs[key] = value;
+                        console.log(`✓ Found ${envVar}=${value} (mapped to '${key}')`);
+                    }
                     break; // Use first match
                 }
             }
@@ -51,8 +58,10 @@ class TestLensReporter {
     }
     constructor(options) {
         this.runCreationFailed = false; // Track if run creation failed due to limits
+        this.cliArgs = {}; // Store CLI args separately
         // Parse custom CLI arguments
         const customArgs = TestLensReporter.parseCustomArgs();
+        this.cliArgs = customArgs; // Store CLI args separately for later use
         // Allow API key from environment variable if not provided in config
         // Check multiple environment variable names in priority order (uppercase and lowercase)
         const apiKey = options.apiKey
@@ -77,7 +86,7 @@ class TestLensReporter {
             flushInterval: options.flushInterval || 5000,
             retryAttempts: options.retryAttempts !== undefined ? options.retryAttempts : 0,
             timeout: options.timeout || 60000,
-            customMetadata: { ...customArgs, ...options.customMetadata } // CLI args + config metadata
+            customMetadata: { ...options.customMetadata, ...customArgs } // Config metadata first, then CLI args override
         };
         if (!this.config.apiKey) {
             throw new Error('API_KEY is required for TestLensReporter. Pass it as apiKey option in your playwright config or set one of these environment variables: TESTLENS_API_KEY, TESTLENS_KEY, PLAYWRIGHT_API_KEY, PW_API_KEY, API_KEY, or APIKEY.');
@@ -893,9 +902,13 @@ class TestLensReporter {
         if (tagMatches) {
             tags.push(...tagMatches);
         }
-        // Add testlensBuildTag from custom metadata if present
-        if (this.config.customMetadata?.testlensBuildTag) {
-            tags.push(`@${this.config.customMetadata.testlensBuildTag}`);
+        // Add testlensBuildTag: CLI args take precedence over config
+        const buildTagSource = this.cliArgs.testlensBuildTag || this.config.customMetadata?.testlensBuildTag;
+        if (buildTagSource) {
+            const buildTags = Array.isArray(buildTagSource)
+                ? buildTagSource
+                : [buildTagSource];
+            buildTags.forEach(tag => tags.push(`@${tag}`));
         }
         // Remove duplicates and return
         return [...new Set(tags)];

package/index.ts

@@ -183,13 +183,14 @@ export class TestLensReporter implements Reporter {
   private specMap: Map<string, SpecData>;
   private testMap: Map<string, TestData>;
   private runCreationFailed: boolean = false; // Track if run creation failed due to limits
+  private cliArgs: Record<string, any> = {}; // Store CLI args separately
 
   /**
    * Parse custom metadata from environment variables
    * Checks for common metadata environment variables
    */
-  private static parseCustomArgs(): Record<string, string> {
-    const customArgs: Record<string, string> = {};
+  private static parseCustomArgs(): Record<string, any> {
+    const customArgs: Record<string, any> = {};
     
     // Common environment variable names for build metadata
     const envVarMappings: Record<string, string[]> = {
@@ -207,8 +208,14 @@ export class TestLensReporter implements Reporter {
       for (const envVar of envVars) {
         const value = process.env[envVar];
         if (value) {
-          customArgs[key] = value;
-          console.log(`✓ Found ${envVar}=${value} (mapped to '${key}')`);
+          // For testlensBuildTag, support comma-separated values
+          if (key === 'testlensBuildTag' && value.includes(',')) {
+            customArgs[key] = value.split(',').map(tag => tag.trim()).filter(tag => tag);
+            console.log(`✓ Found ${envVar}=${value} (mapped to '${key}' as array of ${customArgs[key].length} tags)`);
+          } else {
+            customArgs[key] = value;
+            console.log(`✓ Found ${envVar}=${value} (mapped to '${key}')`);
+          }
           break; // Use first match
         }
       }
@@ -220,6 +227,7 @@ export class TestLensReporter implements Reporter {
   constructor(options: TestLensReporterOptions) {
     // Parse custom CLI arguments
     const customArgs = TestLensReporter.parseCustomArgs();
+    this.cliArgs = customArgs; // Store CLI args separately for later use
     
     // Allow API key from environment variable if not provided in config
     // Check multiple environment variable names in priority order (uppercase and lowercase)
@@ -246,7 +254,7 @@ export class TestLensReporter implements Reporter {
       flushInterval: options.flushInterval || 5000,
       retryAttempts: options.retryAttempts !== undefined ? options.retryAttempts : 0,
       timeout: options.timeout || 60000,
-      customMetadata: { ...customArgs, ...options.customMetadata } // CLI args + config metadata
+      customMetadata: { ...options.customMetadata, ...customArgs } // Config metadata first, then CLI args override
     } as Required<TestLensReporterConfig>;
 
     if (!this.config.apiKey) {
@@ -1152,9 +1160,13 @@ export class TestLensReporter implements Reporter {
       tags.push(...tagMatches);
     }
 
-    // Add testlensBuildTag from custom metadata if present
-    if (this.config.customMetadata?.testlensBuildTag) {
-      tags.push(`@${this.config.customMetadata.testlensBuildTag}`);
+    // Add testlensBuildTag: CLI args take precedence over config
+    const buildTagSource = this.cliArgs.testlensBuildTag || this.config.customMetadata?.testlensBuildTag;
+    if (buildTagSource) {
+      const buildTags = Array.isArray(buildTagSource)
+        ? buildTagSource
+        : [buildTagSource];
+      buildTags.forEach(tag => tags.push(`@${tag}`));
     }
     
     // Remove duplicates and return

package/package.json

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