fix: revert vitejs#9459

sapphi-red · Aug 12, 2022 · 502fadd · 502fadd
1 parent 7121ee0
commit 502fadd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/packages/vite/src/node/plugins/resolve.ts b/packages/vite/src/node/plugins/resolve.ts
@@ -867,8 +867,6 @@ export function resolvePackageEntry(
             ) {
               // likely UMD or CJS(!!! e.g. firebase 7.x), prefer module
               entryPoint = data.module
-            } else {
-              entryPoint = browserEntry
             }
           }
         } else {

diff --git a/playground/resolve/__tests__/resolve.spec.ts b/playground/resolve/__tests__/resolve.spec.ts
@@ -88,7 +88,8 @@ test('browser field', async () => {
   expect(await page.textContent('.browser')).toMatch('[success]')
 })
 
-test('Resolve browser field even if module field exists', async () => {
+// TODO: skip because #9459 is reverted
+test.skip('Resolve browser field even if module field exists', async () => {
   expect(await page.textContent('.browser-module1')).toMatch('[success]')
 })