Stop timer logging

estherkim · Feb 15, 2019 · cdfd719 · cdfd719
1 parent 4b63cd7
commit cdfd719
Show file tree

Hide file tree

Showing 8 changed files with 11 additions and 10 deletions.
diff --git a/build-system/pr-check/build.js b/build-system/pr-check/build.js
@@ -102,7 +102,7 @@ function main() {
      'not affect the runtime, build system, or test files');
   }
 
-  stopTimer(FILENAME, startTime);
+  stopTimer(FILENAME, FILENAME, startTime);
   return 0;
 }
 

diff --git a/build-system/pr-check/checks.js b/build-system/pr-check/checks.js
@@ -74,7 +74,7 @@ function main() {
     }
   }
 
-  stopTimer(FILENAME, startTime);
+  stopTimer(FILENAME, FILENAME,  startTime);
   return 0;
 }
 

diff --git a/build-system/pr-check/dist-tests.js b/build-system/pr-check/dist-tests.js
@@ -75,7 +75,7 @@ function main() {
     }
   }
 
-  stopTimer(FILENAME, startTime);
+  stopTimer(FILENAME, FILENAME, startTime);
   return 0;
 }
 

diff --git a/build-system/pr-check/local-tests.js b/build-system/pr-check/local-tests.js
@@ -86,7 +86,7 @@ function main() {
     }
   }
 
-  stopTimer(FILENAME, startTime);
+  stopTimer(FILENAME, FILENAME, startTime);
   return 0;
 }
 

diff --git a/build-system/pr-check/remote-tests.js b/build-system/pr-check/remote-tests.js
@@ -71,7 +71,7 @@ function main() {
   }
 
   stopSauceConnect(FILENAME);
-  stopTimer(FILENAME, startTime);
+  stopTimer(FILENAME, FILENAME, startTime);
   return 0;
 }
 

diff --git a/build-system/pr-check/utils.js b/build-system/pr-check/utils.js
@@ -100,15 +100,16 @@ function startTimer(functionName, fileName) {
 /**
  * Stops the timer for the given function and prints the execution time.
  * @param {string} functionName
+ * @param {string} fileName
  * @param {DOMHighResTimeStamp} startTime
  * @return {number}
  */
-function stopTimer(functionName, startTime) {
+function stopTimer(functionName, fileName, startTime) {
   const endTime = Date.now();
   const executionTime = new Date(endTime - startTime);
   const mins = executionTime.getMinutes();
   const secs = executionTime.getSeconds();
-  const fileLogPrefix = colors.bold(colors.yellow(`${functionName}:`));
+  const fileLogPrefix = colors.bold(colors.yellow(`${fileName}:`));
   console.log(
       fileLogPrefix, 'Done running', colors.cyan(functionName),
       'Total time:', colors.green(mins + 'm ' + secs + 's'));
@@ -135,7 +136,7 @@ function timedExec(cmd) {
 function timedExecOrDie(cmd, fileName = 'utils.js') {
   const startTime = startTimer(cmd, fileName);
   execOrDie(cmd);
-  stopTimer(cmd, startTime);
+  stopTimer(cmd, fileName, startTime);
 }
 
 /**

diff --git a/build-system/pr-check/validator-tests.js b/build-system/pr-check/validator-tests.js
@@ -50,7 +50,7 @@ function main() {
       'not affect the validator or validator web UI.');
   }
 
-  stopTimer(FILENAME, startTime);
+  stopTimer(FILENAME, FILENAME, startTime);
   return 0;
 }
 

diff --git a/build-system/pr-check/visual-diff-tests.js b/build-system/pr-check/visual-diff-tests.js
@@ -62,7 +62,7 @@ function main() {
     }
   }
 
-  stopTimer(FILENAME, startTime);
+  stopTimer(FILENAME, FILENAME, startTime);
   return 0;
 }