fix(cli-utils): Add log entry for baseline writing

relative-ci · Jan 24, 2024 · 8b22b89 · 8b22b89
1 parent 98d0b18
commit 8b22b89
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/packages/cli-utils/src/reports.ts b/packages/cli-utils/src/reports.ts
@@ -115,10 +115,10 @@ export const generateReports = async (
       baselineStats = filter(baselineStatsData);
 
       if (!options.silent) {
-        logger.info(`${TEXT.BASELINE_READING} ${baselinePath}`);
+        logger.info(`${TEXT.BASELINE_READING} ${baselinePath}.`);
       }
     } catch (err) {
-      logger.warn(TEXT.BASELINE_MISSING);
+      logger.warn(`${TEXT.BASELINE_READING} ${baselinePath}. ${TEXT.BASELINE_MISSING}`);
     }
   }
 
@@ -150,6 +150,10 @@ export const generateReports = async (
       source: JSON.stringify(data),
       filepath: baselineAbsolutePath,
     };
+
+    if (!options.silent) {
+      logger.info(`${TEXT.BASELINE_WRITING} ${baselinePath}`);
+    }
   }
 
   const info = getReportInfo(report);

diff --git a/packages/cli-utils/src/text.ts b/packages/cli-utils/src/text.ts
@@ -1,5 +1,6 @@
 export const BASELINE_MISSING = 'Baseline stats file is missing, see "baseline" option.';
 export const BASELINE_READING = 'Reading baseline data from';
+export const BASELINE_WRITING = 'Writing baseline data to';
 
 export const CLI_NO_COMPARE_MODE = 'Not in compare mode (see --compare).';
 export const CLI_MULTIPLE_STATS = 'Multiple stat files already set.';

diff --git a/packages/webpack-plugin/src/__tests__/webpack-plugin.test.js b/packages/webpack-plugin/src/__tests__/webpack-plugin.test.js
@@ -116,7 +116,7 @@ describe('webpack plugin', () => {
         /* eslint-disable no-console */
         expect(console.warn).not.toBeCalledWith('Missing baseline stats, see "baseline" option.');
         expect(console.info).toBeCalledWith(
-          'Reading baseline data from ../node_modules/.cache/bundle-stats/baseline.json',
+          'Reading baseline data from ../node_modules/.cache/bundle-stats/baseline.json.',
         );
         expect(console.info).toBeCalledWith('Bundle Size — 27B (+35%).');
         /* eslint-enable no-console */
@@ -151,7 +151,7 @@ describe('webpack plugin', () => {
 
         /* eslint-disable no-console */
         expect(console.warn).not.toBeCalledWith('Missing baseline stats, see "baseline" option.');
-        expect(console.info).toBeCalledWith('Reading baseline data from custom-baseline.json');
+        expect(console.info).toBeCalledWith('Reading baseline data from custom-baseline.json.');
         expect(console.info).toBeCalledWith('Bundle Size — 27B (+8%).');
         /* eslint-enable no-console */