revert!: don't rely on environment for interopDefault (#2682)

Closes #2610

Author: sheremet-va

packages/vitest/src/runtime/execute.ts

@@ -71,10 +71,4 @@ export class VitestRunner extends ViteNodeRunner {
       __vitest_mocker__: this.mocker,
     })
   }
-
-  shouldInterop(path: string, mod: any) {
-    if (this.options.interopDefault === false)
-      return false
-    return (this.options.interopDefault || getCurrentEnvironment() !== 'node') && super.shouldInterop(path, mod)
-  }
 }

test/core/test/module-dom.test.ts

@@ -30,17 +30,21 @@ describe('validating nested defaults in isolation', () => {
   it.each([
     nestedDefaultCjs,
     nestedDefaultExternalCjs,
-  ])('nested default should stay, because environment is node', (mod) => {
+  ])('nested default should be resolved', (mod) => {
     expect(mod).toHaveProperty('default')
-    expect(mod.default).toHaveProperty('default')
-    expect(mod.default.default.a).toBe('a')
-    expect(mod.default.default.b).toBe('b')
+    expect(mod.default).not.toHaveProperty('default')
+    expect(mod.default.a).toBe('a')
+    expect(mod.default.b).toBe('b')
+    expect(mod.a).toBe('a')
+    expect(mod.b).toBe('b')
   })
 
-  it('don\'t interop external module.exports, because environment is node', () => {
+  it('externalized "module.exports" CJS module interops default', () => {
     expect(moduleDefaultCjs).toHaveProperty('default')
     expect(moduleDefaultCjs.default).toHaveProperty('a')
-    expect(moduleDefaultCjs).not.toHaveProperty('a')
+    expect(moduleDefaultCjs.default.a).toBe('a')
+    expect(moduleDefaultCjs).toHaveProperty('a')
+    expect(moduleDefaultCjs.a).toBe('a')
   })
 })