chore: use isFirstCompile instead of isFirstRun (#506)

Author: 9aoy

e2e/watch/index.test.ts

@@ -28,7 +28,7 @@ describe('watch', () => {
     await cli.waitForStdout('Duration');
     expect(cli.stdout).toMatch('Tests 1 passed');
     expect(cli.stdout).not.toMatch('Test files to re-run:');
-    expect(cli.stdout).toMatch('Fully run test files for first run.');
+    expect(cli.stdout).toMatch('Run all tests.');
 
     // create
     cli.resetStd();

e2e/watch/shortcuts.test.ts

@@ -37,7 +37,7 @@ describe('CLI shortcuts', () => {
     await cli.waitForStdout('Duration');
     expect(cli.stdout).toMatch('Tests 1 failed | 1 passed');
     await cli.waitForStdout('press h to show help');
-    expect(cli.stdout).toMatch('Fully run test files for first run.');
+    expect(cli.stdout).toMatch('Run all tests.');
 
     cli.exec.process!.stdin!.write('h');
 
@@ -82,7 +82,7 @@ describe('CLI shortcuts', () => {
     await cli.waitForStdout('Duration');
     expect(cli.stdout).toMatch('Tests 1 failed | 1 passed');
     await cli.waitForStdout('press h to show help');
-    expect(cli.stdout).toMatch('Fully run test files for first run.');
+    expect(cli.stdout).toMatch('Run all tests.');
     cli.resetStd();
 
     // rerun failed tests

packages/core/src/core/rsbuild.ts

@@ -228,15 +228,13 @@ export const createRsbuildServer = async ({
     assetFiles: Record<string, string>;
     sourceMaps: Record<string, SourceMapInput>;
     getSourcemap: (sourcePath: string) => SourceMapInput | null;
-    isFirstRun: boolean;
     affectedEntries: EntryInfo[];
     deletedEntries: string[];
   }>;
   closeServer: () => Promise<void>;
 }> => {
   // Read files from memory via `rspackCompiler.outputFileSystem`
   let rspackCompiler: Rspack.Compiler | Rspack.MultiCompiler | undefined;
-  let isFirstCompile = false;
 
   const rstestCompilerPlugin: RsbuildPlugin = {
     name: 'rstest:compiler',
@@ -245,10 +243,6 @@ export const createRsbuildServer = async ({
         // outputFileSystem to be updated later by `rsbuild-dev-middleware`
         rspackCompiler = compiler;
       });
-
-      api.onAfterDevCompile(({ isFirstCompile: _isFirstCompile }) => {
-        isFirstCompile = _isFirstCompile;
-      });
     },
   };
 
@@ -410,7 +404,6 @@ export const createRsbuildServer = async ({
     return {
       affectedEntries,
       deletedEntries,
-      isFirstRun: isFirstCompile,
       hash,
       entries,
       setupEntries,

packages/core/src/core/runTests.ts

@@ -122,28 +122,23 @@ export async function runTests(context: Rstest): Promise<void> {
       getSourcemap,
       buildTime,
       hash,
-      isFirstRun,
       affectedEntries,
       deletedEntries,
     } = await getRsbuildStats({ fileFilters });
     const testStart = Date.now();
 
     let finalEntries: EntryInfo[] = entries;
     if (mode === 'on-demand') {
-      if (isFirstRun) {
-        logger.debug(color.yellow('Fully run test files for first run.\n'));
+      if (affectedEntries.length === 0) {
+        logger.debug(color.yellow('No test files are re-run.'));
       } else {
-        if (affectedEntries.length === 0) {
-          logger.debug(color.yellow('No test files are re-run.'));
-        } else {
-          logger.debug(
-            color.yellow('Test files to re-run:\n') +
-              affectedEntries.map((e) => e.testPath).join('\n') +
-              '\n',
-          );
-        }
-        finalEntries = affectedEntries;
+        logger.debug(
+          color.yellow('Test files to re-run:\n') +
+            affectedEntries.map((e) => e.testPath).join('\n') +
+            '\n',
+        );
       }
+      finalEntries = affectedEntries;
     } else {
       logger.debug(
         color.yellow(
@@ -228,7 +223,7 @@ export async function runTests(context: Rstest): Promise<void> {
 
     rsbuildInstance.onAfterDevCompile(async ({ isFirstCompile }) => {
       snapshotManager.clear();
-      await run({ mode: 'on-demand' });
+      await run({ mode: isFirstCompile ? 'all' : 'on-demand' });
 
       if (isFirstCompile && enableCliShortcuts) {
         const closeCliShortcuts = await setupCliShortcuts({