Skip to content

fix: use Vite mode to select the built package for tests - #2995

Merged
cexbrayat merged 1 commit into
vuejs:mainfrom
cexbrayat:fix-build-test-mode
Oct 30, 2026
Merged

cexbrayat merged 1 commit into
vuejs:mainfrom
cexbrayat:fix-build-test-mode

Conversation

@cexbrayat

Copy link
Copy Markdown
Member

The command meant to test the built package was accidentally testing the source code instead.

There were two different settings involved:

  • test:build set the test mode to test-build.
  • The configuration checked NODE_ENV, which still said test.

Because it checked the wrong setting, it loaded the source code. So a green “build test” run didn’t actually tell us whether the built package worked.

The command meant to test the **built package** was accidentally testing the **source code** instead.

There were two different settings involved:

- `test:build` set the test **mode** to `test-build`.
- The configuration checked **NODE_ENV**, which still said `test`.

Because it checked the wrong setting, it loaded the source code. So a green “build test” run didn’t actually tell us whether the built package worked.
@netlify

netlify Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for vue-test-utils-docs ready!

Name Link
🔨 Latest commit 628e615
🔍 Latest deploy log https://app.netlify.com/projects/vue-test-utils-docs/deploys/6ae44d2c8058e78616881a36
😎 Deploy Preview https://deploy-preview-2995--vue-test-utils-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@pkg-pr-new

pkg-pr-new Bot commented Oct 30, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@vue/test-utils@2995

commit: 628e615

@cexbrayat
cexbrayat merged commit 40af96f into vuejs:main Oct 30, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant