Skip to content

Commit 28c1b61

Browse files
mikeland73claude
andcommitted
Fix test configs: use corepack instead of pnpm nix package
Enable DEVBOX_COREPACK_ENABLED in test devbox.json so pnpm is available via corepack (bundled with nodejs) rather than as a separate nix package. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 56e104f commit 28c1b61

2 files changed

Lines changed: 10 additions & 9 deletions

File tree

testscripts/add/add_jspm.test.txt

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,7 @@
11
# Test adding and removing JS package manager packages
22

3-
exec devbox init
4-
5-
# Add nodejs and pnpm first so the JS package manager is available
6-
exec devbox add nodejs_22 pnpm
3+
# Add nodejs first so the JS package manager is available
4+
exec devbox add nodejs_22
75

86
# Add a pnpm-managed JS package
97
exec devbox add pnpm:vercel
@@ -20,22 +18,23 @@ json.superset devbox.json expected_devbox_after_rm.json
2018

2119
-- devbox.json --
2220
{
23-
"packages": []
21+
"packages": [],
22+
"env": {
23+
"DEVBOX_COREPACK_ENABLED": "true"
24+
}
2425
}
2526

2627
-- expected_devbox_after_add.json --
2728
{
2829
"packages": [
2930
"nodejs_22@latest",
30-
"pnpm@latest",
3131
"pnpm:vercel@latest"
3232
]
3333
}
3434

3535
-- expected_devbox_after_rm.json --
3636
{
3737
"packages": [
38-
"nodejs_22@latest",
39-
"pnpm@latest"
38+
"nodejs_22@latest"
4039
]
4140
}

testscripts/jspm/devbox.json

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,11 @@
11
{
22
"packages": [
33
"nodejs_22@latest",
4-
"pnpm@latest",
54
"pnpm:vercel@latest"
65
],
6+
"env": {
7+
"DEVBOX_COREPACK_ENABLED": "true"
8+
},
79
"shell": {
810
"scripts": {
911
"run_test": "vercel --version && echo 'JSPM test passed'"

0 commit comments

Comments
 (0)