fix: track collected/discarded traces when recordSuccessfulVUs is set

artilleryio · May 10, 2024 · 533b301 · 533b301
1 parent 05c5a1c
commit 533b301
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/artillery-engine-playwright/index.js b/packages/artillery-engine-playwright/index.js
@@ -403,12 +403,10 @@ class PlaywrightEngine {
             await context.tracing.stop({
               path: initialContext.vars.__tracePath
             });
-            events.emit('counter', 'browser.traces_recorded', 1);
+            events.emit('counter', 'browser.traces_collected', 1);
             self.lastTraceRecordedTime = Date.now();
             self.tracesRecordedCount++;
             initialContext.vars.isRecording = false; // for finally{} block
-          } else {
-            events.emit('counter', 'browser.traces_discarded', 1);
           }
         }
 
@@ -426,8 +424,10 @@ class PlaywrightEngine {
             await context.tracing.stop({
               path: initialContext.vars.__tracePath
             });
+            events.emit('counter', 'browser.traces_collected', 1);
           } else {
             await context.tracing.stop();
+            events.emit('counter', 'browser.traces_discarded', 1);
           }
         }