🐛 Fix printed debug timestamps (#289)

percy · Apr 5, 2021 · 24edf25 · 24edf25
1 parent e657279
commit 24edf25
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/packages/logger/src/logger.js b/packages/logger/src/logger.js
@@ -174,7 +174,7 @@ export default class PercyLogger {
       let elapsed = timestamp - (this.lastlog || timestamp);
       if (isError && this.level !== 'debug') message = error.toString();
       this.write(level, this.format(message, debug, error ? 'error' : level, elapsed));
-      this.lastlog ||= timestamp;
+      this.lastlog = timestamp;
     }
   }
 

diff --git a/packages/logger/test/index.test.js b/packages/logger/test/index.test.js
@@ -257,6 +257,7 @@ describe('logger', () => {
       log.error('Error log');
       await new Promise(r => setTimeout(r, 100));
       log.debug('Debug log');
+      log.error('Final log');
 
       expect(helpers.stdout).toEqual([
         jasmine.stringMatching('Info log \\(\\dms\\)')
@@ -265,7 +266,8 @@ describe('logger', () => {
       expect(helpers.stderr).toEqual([
         jasmine.stringMatching('Warn log \\(\\dms\\)'),
         jasmine.stringMatching('Error log \\(\\dms\\)'),
-        jasmine.stringMatching('Debug log \\((9[0-9]|1[01][0-9])ms\\)')
+        jasmine.stringMatching('Debug log \\((9[0-9]|1[01][0-9])ms\\)'),
+        jasmine.stringMatching('Final log \\(\\dms\\)')
       ]);
     });
   });