vitest browser __vi_esm_0__.default is not a function

[NicoVogel]

Describe the bug

When executing the tests, each test fails because of TypeError: __vi_esm_0__.default is not a function.
After reducing it to the minimal reproducible example, I could isolate that the issue is the package pino.

You can find "working" tests here src/browser/working.spec.ts

The mocking does not work, but this is besides this bug

But the same code that uses pino is not able to load it: src/browser/logger.spec.ts

Reproduction

1. git clone https://github.com/NicoVogel/logger-wrapper
2. git checkout bug/vitest-browser-vi-esm-0-default-not-a-function
3. (optional) reopen in dev container
4. pnpm install
5. cd packages/logger
6. pnpm exec vitest -c vitest.browser.ts

System Info

System:
    OS: Linux 6.2 Ubuntu 22.04.3 LTS 22.04.3 LTS (Jammy Jellyfish)
    CPU: (16) x64 13th Gen Intel(R) Core(TM) i5-1340P
    Memory: 24.36 GB / 31.05 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 18.17.1 - ~/.nvm/versions/node/v18.17.1/bin/node
    npm: 9.6.7 - ~/.nvm/versions/node/v18.17.1/bin/npm
    pnpm: 8.6.12 - ~/.nvm/versions/node/v18.17.1/bin/pnpm
  Browsers:
    Chrome: 116.0.5845.140
  npmPackages:
    @vitest/browser: ^0.34.1 => 0.34.1

Used Package Manager

pnpm

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
• The provided reproduction is a minimal reproducible example of the bug.

[hi-ogawa]

I think this is also CJS import issue with browser.slowHijackESM: true and probably considered as a duplicate of #4264, #4505, etc...

As I commented in #4505 (comment), the default became browser.slowHijackESM: false in Vitest v1, so either you can upgrade it or you can manually set slowHijackESM: false in old version.

I tested your repro with slowHijackESM: false and default import works but import { pino } from "pino" is still undefined. It looks like that was issue on Pino side and it seems to be fixed recently in pinojs/pino#1803.
(Interestingly Pino now also exports default.default, so it actually allows it work with slowHijackESM: true as well)

[clocke3]

I am seeing something similar in an application, instead with this error: TypeError: vi_import_0.jsxDEVdescribe is not a function. It looks like it is thrown when trying to use vi.mock.

vi.mock('../../components/shared/table', () => { return { default: () => <div role='table'>table</div> }; });