test: fix flaky failure caused by concurrent resolveId calls (#8381)

sapphi-red · sapphi-red · commit ca7e58bf60a5 · 2026-02-19T13:39:08.000Z
diff --git a/packages/rolldown/tests/fixtures/tree-shake/module-side-effects-proxy/_config.ts b/packages/rolldown/tests/fixtures/tree-shake/module-side-effects-proxy/_config.ts
@@ -3,7 +3,6 @@ import { defineTest } from 'rolldown-tests';
 import { expect } from 'vitest';
 
 export default defineTest({
-  sequential: true,
   config: {
     plugins: [
       {
diff --git a/packages/rolldown/tests/fixtures/tree-shake/module-side-effects-proxy2/_config.ts b/packages/rolldown/tests/fixtures/tree-shake/module-side-effects-proxy2/_config.ts
@@ -3,14 +3,14 @@ import { defineTest } from 'rolldown-tests';
 import { expect } from 'vitest';
 
 export default defineTest({
-  sequential: true,
   config: {
     plugins: [
       {
         name: 'loader',
         async transform(code, id) {
           if (id.includes('bar.js')) {
             const resolved = await this.resolve('./foo.js', id);
+            await this.load({ id: resolved!.id }); // ensure moduleInfo for `foo.js` exists
             const moduleInfo = this.getModuleInfo(resolved!.id);
             moduleInfo!.moduleSideEffects = true;
 
diff --git a/packages/rolldown/tests/fixtures/tree-shake/module-side-effects-proxy3/_config.ts b/packages/rolldown/tests/fixtures/tree-shake/module-side-effects-proxy3/_config.ts
@@ -3,14 +3,14 @@ import { defineTest } from 'rolldown-tests';
 import { expect } from 'vitest';
 
 export default defineTest({
-  sequential: true,
   config: {
     plugins: [
       {
         name: 'loader',
         async transform(code, id) {
           if (id.includes('bar.js')) {
             const resolved = await this.resolve('./foo.js', id);
+            await this.load({ id: resolved!.id }); // ensure moduleInfo for `foo.js` exists
             const moduleInfo = this.getModuleInfo(resolved!.id);
             moduleInfo!.moduleSideEffects = false;
           }

Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,6 @@ import { defineTest } from 'rolldown-tests';`
`3`	`3`	`import { expect } from 'vitest';`
`4`	`4`
`5`	`5`	`export default defineTest({`
`6`		`- sequential: true,`
`7`	`6`	`config: {`
`8`	`7`	`plugins: [`
`9`	`8`	`{`