test(e2e): move plugins config into config object in test cases

e2e/cases/assets/inline-query-auto/index.test.ts

@@ -1,14 +1,10 @@
 import { expect, test } from '@e2e/helper';
-import { pluginReact } from '@rsbuild/plugin-react';
 
 test('should inline small assets automatically', async ({
   page,
   buildPreview,
 }) => {
-  await buildPreview({
-    plugins: [pluginReact()],
-  });
-
+  await buildPreview();
   await expect(
     page.evaluate(
       `document.getElementById('test-img').src.startsWith('data:image/png')`,

e2e/cases/assets/inline-query-auto/rsbuild.config.ts

@@ -0,0 +1,6 @@
+import { defineConfig } from '@rsbuild/core';
+import { pluginReact } from '@rsbuild/plugin-react';
+
+export default defineConfig({
+  plugins: [pluginReact()],
+});

e2e/cases/assets/inline-query-false/index.test.ts

@@ -1,14 +1,10 @@
 import { expect, test } from '@e2e/helper';
-import { pluginReact } from '@rsbuild/plugin-react';
 
 test('should disable asset inlining with `?__inline=false`', async ({
   page,
   buildPreview,
 }) => {
-  await buildPreview({
-    plugins: [pluginReact()],
-  });
-
+  await buildPreview();
   await expect(
     page.evaluate(
       `document.getElementById('test-img').src.includes('static/image/icon')`,

e2e/cases/assets/inline-query-false/rsbuild.config.ts

@@ -0,0 +1,6 @@
+import { defineConfig } from '@rsbuild/core';
+import { pluginReact } from '@rsbuild/plugin-react';
+
+export default defineConfig({
+  plugins: [pluginReact()],
+});

e2e/cases/assets/inline-query/index.test.ts

@@ -1,11 +1,7 @@
 import { expect, test } from '@e2e/helper';
-import { pluginReact } from '@rsbuild/plugin-react';
 
 test('should inline assets with `?inline`', async ({ page, buildPreview }) => {
-  await buildPreview({
-    plugins: [pluginReact()],
-  });
-
+  await buildPreview();
   await expect(
     page.evaluate(
       `document.getElementById('test-img').src.startsWith('data:image/png')`,

e2e/cases/assets/inline-query/rsbuild.config.ts

@@ -0,0 +1,6 @@
+import { defineConfig } from '@rsbuild/core';
+import { pluginReact } from '@rsbuild/plugin-react';
+
+export default defineConfig({
+  plugins: [pluginReact()],
+});

e2e/cases/css/css-modules-dom/index.test.ts

@@ -1,13 +1,10 @@
 import { expect, test } from '@e2e/helper';
-import { pluginReact } from '@rsbuild/plugin-react';
 
 test('should inject styles and not emit CSS files when output.injectStyles is true', async ({
   page,
   buildPreview,
 }) => {
-  const rsbuild = await buildPreview({
-    plugins: [pluginReact()],
-  });
+  const rsbuild = await buildPreview();
 
   // injectStyles worked
   const files = rsbuild.getDistFiles();

e2e/cases/css/css-modules-dom/rsbuild.config.ts

@@ -1,9 +1,10 @@
 import { defineConfig } from '@rsbuild/core';
 import { pluginLess } from '@rsbuild/plugin-less';
+import { pluginReact } from '@rsbuild/plugin-react';
 import { pluginSass } from '@rsbuild/plugin-sass';
 
 export default defineConfig({
-  plugins: [pluginLess(), pluginSass()],
+  plugins: [pluginLess(), pluginSass(), pluginReact()],
   output: {
     injectStyles: true,
   },

e2e/cases/html/inject/index.test.ts

@@ -6,12 +6,12 @@ test('should preserve the expected script injection order', async ({
   build,
 }) => {
   const rsbuild = await build({
-    plugins: [
-      pluginRem({
-        inlineRuntime: false,
-      }),
-    ],
     config: {
+      plugins: [
+        pluginRem({
+          inlineRuntime: false,
+        }),
+      ],
       html: {
         inject: false,
         template: './static/index.html',

e2e/cases/output/data-uri-limit/assets.test.ts

@@ -41,8 +41,10 @@ const cases = [
 for (const item of cases) {
   test(item.name, async ({ page, buildPreview }) => {
     await buildPreview({
-      plugins: [pluginReact()],
-      config: item.config || {},
+      config: {
+        ...(item.config || {}),
+        plugins: [pluginReact()],
+      },
     });
 
     if (item.expected === 'url') {

e2e/cases/output/externals/index.test.ts

@@ -6,8 +6,8 @@ test('should treat specified modules as externals', async ({
   buildPreview,
 }) => {
   await buildPreview({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       output: {
         externals: {
           './aaa': 'aa',
@@ -34,8 +34,8 @@ test('should not externalize dependencies when target is web worker', async ({
   build,
 }) => {
   const rsbuild = await build({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       output: {
         target: 'web-worker',
         externals: {

e2e/cases/output/legal-comments/index.test.ts

@@ -3,8 +3,8 @@ import { pluginReact } from '@rsbuild/plugin-react';
 
 rspackTest('legalComments linked (default)', async ({ page, buildPreview }) => {
   const rsbuild = await buildPreview({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       performance: {
         chunkSplit: {
           strategy: 'all-in-one',
@@ -44,8 +44,8 @@ test('should omit legal comments when legalComments is set to "none"', async ({
   buildPreview,
 }) => {
   const rsbuild = await buildPreview({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       performance: {
         chunkSplit: {
           strategy: 'all-in-one',
@@ -82,8 +82,8 @@ test('should inline legal comments when legalComments is set to "inline"', async
   buildPreview,
 }) => {
   const rsbuild = await buildPreview({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       performance: {
         chunkSplit: {
           strategy: 'all-in-one',

e2e/cases/performance/resource-hints-prefetch/index.test.ts

@@ -8,8 +8,8 @@ test('should generate prefetch link when prefetch is defined', async ({
   build,
 }) => {
   const rsbuild = await build({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       source: {
         entry: {
           main: join(fixtures, 'src/page1/index.ts'),
@@ -50,8 +50,8 @@ test('should generate prefetch link correctly when assetPrefix do not have a pro
   build,
 }) => {
   const rsbuild = await build({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       source: {
         entry: {
           main: join(fixtures, 'src/page1/index.ts'),
@@ -87,8 +87,8 @@ test('should generate prefetch link correctly when assetPrefix do not have a pro
 
 test('should generate prefetch link with include', async ({ build }) => {
   const rsbuild = await build({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       source: {
         entry: {
           main: join(fixtures, 'src/page1/index.ts'),
@@ -125,8 +125,8 @@ test('should generate prefetch link with include', async ({ build }) => {
 
 test('should generate prefetch link with include array', async ({ build }) => {
   const rsbuild = await build({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       source: {
         entry: {
           main: join(fixtures, 'src/page1/index.ts'),
@@ -163,8 +163,8 @@ test('should generate prefetch link with include array', async ({ build }) => {
 
 test('should generate prefetch link with exclude array', async ({ build }) => {
   const rsbuild = await build({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       source: {
         entry: {
           main: join(fixtures, 'src/page1/index.ts'),
@@ -203,8 +203,8 @@ test('should generate prefetch link by config (distinguish html)', async ({
   build,
 }) => {
   const rsbuild = await build({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       source: {
         entry: {
           page1: join(fixtures, 'src/page1/index.ts'),
@@ -252,8 +252,8 @@ rspackTest(
   'should not generate prefetch link for inlined assets',
   async ({ build }) => {
     const rsbuild = await build({
-      plugins: [pluginReact()],
       config: {
+        plugins: [pluginReact()],
         source: {
           entry: {
             main: join(fixtures, 'src/page1/index.ts'),
@@ -283,8 +283,8 @@ rspackTest(
   'should not generate prefetch link for inlined assets with test option',
   async ({ build }) => {
     const rsbuild = await build({
-      plugins: [pluginReact()],
       config: {
+        plugins: [pluginReact()],
         source: {
           entry: {
             main: join(fixtures, 'src/page1/index.ts'),

e2e/cases/performance/resource-hints-preload/index.test.ts

@@ -8,8 +8,8 @@ test('should generate preload link when preload is defined', async ({
   build,
 }) => {
   const rsbuild = await build({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       source: {
         entry: {
           main: join(fixtures, 'src/page1/index.ts'),
@@ -45,8 +45,8 @@ test('should generate preload link when preload is defined', async ({
 
 test('should generate preload link with duplicate', async ({ build }) => {
   const rsbuild = await build({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       source: {
         entry: {
           main: join(fixtures, 'src/page1/index.ts'),
@@ -84,8 +84,8 @@ test('should generate preload link with duplicate', async ({ build }) => {
 
 test('should generate preload link with crossOrigin', async ({ build }) => {
   const rsbuild = await build({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       source: {
         entry: {
           main: join(fixtures, 'src/page1/index.ts'),
@@ -129,8 +129,8 @@ test('should generate preload link without crossOrigin when same origin', async
   build,
 }) => {
   const rsbuild = await build({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       source: {
         entry: {
           main: join(fixtures, 'src/page1/index.ts'),
@@ -169,8 +169,8 @@ test('should generate preload link without crossOrigin when same origin', async
 
 test('should generate preload link with include', async ({ build }) => {
   const rsbuild = await build({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       source: {
         entry: {
           main: join(fixtures, 'src/page1/index.ts'),
@@ -207,8 +207,8 @@ test('should generate preload link with include', async ({ build }) => {
 
 test('should generate preload link with include array', async ({ build }) => {
   const rsbuild = await build({
-    plugins: [pluginReact()],
     config: {
+      plugins: [pluginReact()],
       source: {
         entry: {
           main: join(fixtures, 'src/page1/index.ts'),
@@ -247,8 +247,8 @@ rspackTest(
   'should not generate preload link for inlined assets',
   async ({ build }) => {
     const rsbuild = await build({
-      plugins: [pluginReact()],
       config: {
+        plugins: [pluginReact()],
         source: {
           entry: {
             main: join(fixtures, 'src/page1/index.ts'),
@@ -278,8 +278,8 @@ rspackTest(
   'should not generate preload link for inlined assets with test option',
   async ({ build }) => {
     const rsbuild = await build({
-      plugins: [pluginReact()],
       config: {
+        plugins: [pluginReact()],
         source: {
           entry: {
             main: join(fixtures, 'src/page1/index.ts'),

e2e/cases/performance/split-chunk-by-module/index.test.ts

@@ -1,13 +1,11 @@
 import { basename } from 'node:path';
-
 import { expect, test } from '@e2e/helper';
-import { pluginReact } from '@rsbuild/plugin-react';
 
 test('should generate module chunks when chunkSplit is "split-by-module"', async ({
   build,
 }) => {
   const rsbuild = await build({
-    plugins: [pluginReact()],
+    config: {},
   });
 
   const files = rsbuild.getDistFiles();

e2e/cases/performance/split-chunk-by-module/rsbuild.config.ts

@@ -1,6 +1,8 @@
 import { defineConfig } from '@rsbuild/core';
+import { pluginReact } from '@rsbuild/plugin-react';
 
 export default defineConfig({
+  plugins: [pluginReact()],
   output: {
     filenameHash: false,
   },

e2e/cases/plugin-api/async-plugin/index.test.ts

@@ -26,7 +26,9 @@ rspackTest(
   'should allow to register async plugin in plugins field',
   async ({ page, buildPreview }) => {
     await buildPreview({
-      plugins: [asyncPlugin()],
+      config: {
+        plugins: [asyncPlugin()],
+      },
     });
 
     const testEl = page.locator('#test-el');