test: remove failure expected cases

packages/vite/src/node/build.ts

@@ -70,8 +70,6 @@ import {
   resolveChokidarOptions,
   resolveEmptyOutDir,
 } from './watch'
-import { completeAmdWrapPlugin } from './plugins/completeAmdWrap'
-import { completeSystemWrapPlugin } from './plugins/completeSystemWrap'
 import { webWorkerPostPlugin } from './plugins/worker'
 import { getHookHandler } from './plugins'
 import { BaseEnvironment } from './baseEnvironment'
@@ -505,8 +503,6 @@ export async function resolveBuildPlugins(config: ResolvedConfig): Promise<{
 }> {
   return {
     pre: [
-      completeAmdWrapPlugin(),
-      completeSystemWrapPlugin(),
       ...(!config.isWorker ? [prepareOutDirPlugin()] : []),
       perEnvironmentPlugin(
         'vite:rollup-options-plugins',

playground/css/__tests__/tests.ts

@@ -516,7 +516,7 @@ export const tests = (isLightningCSS: boolean) => {
     expect(await getColor('.at-import-scss')).toBe('red')
   })
 
-  // TODO: skipped because of https://github.com/rolldown/rolldown/issues/4790
+  // TODO: skipped because of https://github.com/rolldown/rolldown/issues/7315
   test.runIf(isBuild).skip('manual chunk path', async () => {
     // assert that the manual-chunk css is output in the directory specified in manualChunk (#12072)
     expect(

playground/resolve/__tests__/resolve.spec.ts

@@ -139,16 +139,6 @@ test('Resolve browser field even if module field exists', async () => {
   expect(await page.textContent('.browser-module1')).toMatch('[success]')
 })
 
-// should not fallback
-test.skip('Resolve module field if browser field is likely UMD or CJS', async () => {
-  expect(await page.textContent('.browser-module2')).toMatch('[success]')
-})
-
-// should not fallback
-test.skip('Resolve module field if browser field is likely IIFE', async () => {
-  expect(await page.textContent('.browser-module3')).toMatch('[success]')
-})
-
 test('css entry', async () => {
   expect(await page.textContent('.css')).toMatch('[success]')
 })

playground/resolve/index.html

@@ -137,12 +137,6 @@ <h2>Browser Field</h2>
 <h2>Resolve browser field even if module field exists</h2>
 <p class="browser-module1">fail</p>
 
-<h2>Resolve module field if browser field is likely UMD or CJS</h2>
-<p class="browser-module2">fail</p>
-
-<h2>Resolve module field if browser field is likely IIFE</h2>
-<p class="browser-module3">fail</p>
-
 <h2>Don't resolve to the `module` field if the importer is a `require` call</h2>
 <p class="require-pkg-with-module-field">fail</p>
 
@@ -359,14 +353,6 @@ <h2>utf8-bom-package</h2>
   import browserModule1 from '@vitejs/test-resolve-browser-module-field1'
   text('.browser-module1', browserModule1)
 
-  // should not fallback
-  // import browserModule2 from '@vitejs/test-resolve-browser-module-field2'
-  // text('.browser-module2', browserModule2)
-
-  // should not fallback
-  // import browserModule3 from '@vitejs/test-resolve-browser-module-field3'
-  // text('.browser-module3', browserModule3)
-
   import { msg as requireButWithModuleFieldMsg } from '@vitejs/test-require-pkg-with-module-field'
   text('.require-pkg-with-module-field', requireButWithModuleFieldMsg)
 

playground/resolve/package.json

@@ -26,8 +26,6 @@
     "@vitejs/test-require-pkg-with-module-field": "link:./require-pkg-with-module-field",
     "@vitejs/test-resolve-browser-field": "link:./browser-field",
     "@vitejs/test-resolve-browser-module-field1": "link:./browser-module-field1",
-    "@vitejs/test-resolve-browser-module-field2": "link:./browser-module-field2",
-    "@vitejs/test-resolve-browser-module-field3": "link:./browser-module-field3",
     "@vitejs/test-resolve-custom-condition": "link:./custom-condition",
     "@vitejs/test-resolve-custom-main-field": "link:./custom-main-field",
     "@vitejs/test-resolve-custom-browser-main-field": "link:./custom-browser-main-field",