Add missing await for tcpdump stop and move to stopping

sitespeedio · Feb 8, 2024 · 618f763 · 618f763
1 parent e823851
commit 618f763
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/lib/core/engine/command/measure.js b/lib/core/engine/command/measure.js
@@ -443,6 +443,11 @@ export class Measure {
       this.result[this.numberOfMeasuredPages],
       res
     );
+
+    if (this.options.tcpdump) {
+      await this.tcpDump.stop();
+    }
+
     return this.collect(url);
   }
 
@@ -486,9 +491,6 @@ export class Measure {
    * @private
    */
   async collect(url) {
-    if (this.options.tcpdump) {
-      await this.tcpDump.stop();
-    }
     // This stops collecting trace logs in Chrome etc
     // await this.engineDelegate.beforeCollect();
 
@@ -649,7 +651,7 @@ export class Measure {
     }
 
     if (this.options.tcpdump) {
-      this.tcpDump.mv(url, this.index);
+      await this.tcpDump.mv(url, this.index);
     }
 
     await this.engineDelegate.afterEachURL(