mirror/pnpm
1
0
Fork 0
mirror of https://github.com/pnpm/pnpm.git synced 2026-07-23 06:02:50 -04:00
Code Issues Packages Projects Releases Wiki Activity
Files
node-runtime-fix
pnpm/installing/deps-installer/test/install/globalVirtualStore.ts
Zoltan Kochan 187049055f chore: upgrade @typescript/native-preview to 7.0.0-dev.20260421.2 (#11332) 
* chore: upgrade @typescript/native-preview to 7.0.0-dev.20260421.2

- Add explicit `types: ["node"]` to the shared tsconfig because tsgo
  20260421 no longer auto-acquires `@types/*` from `node_modules`.
- Refactor test files to explicitly import jest globals (`describe`,
  `it`, `test`, `expect`, `beforeEach`, etc.) from `@jest/globals`
  instead of relying on `@types/jest` ambient declarations. Under the
  new tsgo build, `import { jest } from '@jest/globals'` shadows the
  ambient `jest` namespace, breaking `@types/jest`'s `declare var
  describe: jest.Describe;` globals.
- Add `@jest/globals` to each package's devDependencies where tests
  now import from it, and add `@types/node` to packages that need it
  but were relying on hoisted resolution.
- Replace `fail()` calls with `throw new Error(...)` since `fail` is
  no longer globally available.

* chore: fix remaining tsgo type-strictness errors

- Strip `as <PnpmType>` casts on objects passed to toMatchObject /
  toStrictEqual / toEqual; @jest/globals rejects the typed objects
  (which include AsymmetricMatchers) vs. the repo-specific type.
- Type `jest.fn<...>()` explicitly where the mock's signature matters
  for toHaveBeenCalledWith.
- Replace `beforeEach(() => X)` with `beforeEach(() => { X })` so the
  return value is void, as the stricter jest typing requires.
- Use `expect.objectContaining({...})` in one place where the full
  expected object triggered stricter type resolution.
- Cast `prompt.mock.calls` arg through `as unknown as Record<...>[]`
  for patch.test.ts's nested-array matchers.
- Fix off-by-one `<reference path>` in pnpm/test/getConfig.test.ts
  that only surfaced now.
- Move `@jest/globals` from devDependencies to dependencies in the
  two `__utils__` packages that import it from `src/`.
- Clean up unused imports from the @jest/globals migration.

* chore: address Copilot review on #11332

- Move misplaced `@jest/globals` imports to the top import block in
  checkEngine, run.ts, and workspace/root-finder tests where the
  script dropped them below executable code.
- Replace `try { await x(); throw new Error('should have thrown') } catch`
  in bins/linker, lockfile/fs, and resolving/local-resolver tests with
  `await expect(x()).rejects.toMatchObject({...})`. The old pattern
  swallowed an unrelated `throw` if the under-test call silently
  succeeded, which would fail on the catch-block assertion with a
  misleading message.
2026-04-21 22:50:40 +02:00

729 lines
28 KiB
TypeScript
Raw Permalink Blame History

import fs from 'node:fs'
import path from 'node:path'
import { afterAll, expect, test } from '@jest/globals'
import { assertProject } from '@pnpm/assert-project'
import { install, type MutatedProject, mutateModules, type ProjectOptions } from '@pnpm/installing.deps-installer'
import { prepareEmpty, preparePackages } from '@pnpm/prepare'
import { addDistTag, getIntegrity } from '@pnpm/registry-mock'
import type { PackageFilesIndex } from '@pnpm/store.cafs'
import { StoreIndex, storeIndexKey } from '@pnpm/store.index'
import type { ProjectRootDir } from '@pnpm/types'
import { rimrafSync } from '@zkochan/rimraf'
import { testDefaults } from '../utils/index.js'
const storeIndexes: StoreIndex[] = []
afterAll(() => {
for (const si of storeIndexes) si.close()
})
test('using a global virtual store', async () => {
prepareEmpty()
const globalVirtualStoreDir = path.resolve('links')
const manifest = {
dependencies: {
'@pnpm.e2e/pkg-with-1-dep': '100.0.0',
},
}
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
hoistPattern: ['*'],
}))
{
expect(fs.existsSync(path.resolve('node_modules/.pnpm/node_modules/@pnpm.e2e/dep-of-pkg-with-1-dep/package.json'))).toBeTruthy()
expect(fs.existsSync(path.resolve('node_modules/.pnpm/lock.yaml'))).toBeTruthy()
const files = fs.readdirSync(path.join(globalVirtualStoreDir, '@pnpm.e2e/pkg-with-1-dep/100.0.0'))
expect(files).toHaveLength(1)
expect(fs.existsSync(path.join(globalVirtualStoreDir, '@pnpm.e2e/pkg-with-1-dep/100.0.0', files[0], 'node_modules/@pnpm.e2e/pkg-with-1-dep/package.json'))).toBeTruthy()
expect(fs.existsSync(path.join(globalVirtualStoreDir, '@pnpm.e2e/pkg-with-1-dep/100.0.0', files[0], 'node_modules/@pnpm.e2e/dep-of-pkg-with-1-dep/package.json'))).toBeTruthy()
}
rimrafSync('node_modules')
rimrafSync(globalVirtualStoreDir)
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
frozenLockfile: true,
hoistPattern: ['*'],
}))
{
expect(fs.existsSync(path.resolve('node_modules/.pnpm/node_modules/@pnpm.e2e/dep-of-pkg-with-1-dep/package.json'))).toBeTruthy()
expect(fs.existsSync(path.resolve('node_modules/.pnpm/lock.yaml'))).toBeTruthy()
const files = fs.readdirSync(path.join(globalVirtualStoreDir, '@pnpm.e2e/pkg-with-1-dep/100.0.0'))
expect(files).toHaveLength(1)
expect(fs.existsSync(path.join(globalVirtualStoreDir, '@pnpm.e2e/pkg-with-1-dep/100.0.0', files[0], 'node_modules/@pnpm.e2e/pkg-with-1-dep/package.json'))).toBeTruthy()
expect(fs.existsSync(path.join(globalVirtualStoreDir, '@pnpm.e2e/pkg-with-1-dep/100.0.0', files[0], 'node_modules/@pnpm.e2e/dep-of-pkg-with-1-dep/package.json'))).toBeTruthy()
}
})
test('reinstall from warm global virtual store after deleting node_modules', async () => {
prepareEmpty()
const globalVirtualStoreDir = path.resolve('links')
const manifest = {
dependencies: {
'@pnpm.e2e/pkg-with-1-dep': '100.0.0',
},
}
const opts = testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
hoistPattern: ['*'],
})
await install(manifest, opts)
// Delete only node_modules, keep the global virtual store warm
rimrafSync('node_modules')
expect(fs.existsSync(globalVirtualStoreDir)).toBeTruthy()
// Spy on fetchPackage to verify the fast-path skips fetching
const originalFetchPackage = opts.storeController.fetchPackage
let fetchPackageCalls = 0
opts.storeController.fetchPackage = ((fetchOpts) => {
fetchPackageCalls++
return originalFetchPackage(fetchOpts)
}) as typeof originalFetchPackage
// Reinstall with frozenLockfile — should reattach from the warm global store
await install(manifest, {
...opts,
frozenLockfile: true,
})
// fetchPackage should NOT be called — all packages reattached from warm GVS
expect(fetchPackageCalls).toBe(0)
expect(fs.existsSync(path.resolve('node_modules/.pnpm/node_modules/@pnpm.e2e/dep-of-pkg-with-1-dep/package.json'))).toBeTruthy()
expect(fs.existsSync(path.resolve('node_modules/.pnpm/lock.yaml'))).toBeTruthy()
const files = fs.readdirSync(path.join(globalVirtualStoreDir, '@pnpm.e2e/pkg-with-1-dep/100.0.0'))
expect(files).toHaveLength(1)
expect(fs.existsSync(path.join(globalVirtualStoreDir, '@pnpm.e2e/pkg-with-1-dep/100.0.0', files[0], 'node_modules/@pnpm.e2e/pkg-with-1-dep/package.json'))).toBeTruthy()
expect(fs.existsSync(path.join(globalVirtualStoreDir, '@pnpm.e2e/pkg-with-1-dep/100.0.0', files[0], 'node_modules/@pnpm.e2e/dep-of-pkg-with-1-dep/package.json'))).toBeTruthy()
})
test('modules are correctly updated when using a global virtual store', async () => {
prepareEmpty()
const globalVirtualStoreDir = path.resolve('links')
const manifest = {
dependencies: {
'@pnpm.e2e/pkg-with-1-dep': '100.0.0',
'@pnpm.e2e/peer-c': '1.0.0',
},
}
const opts = testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
})
await install(manifest, opts)
manifest.dependencies['@pnpm.e2e/peer-c'] = '2.0.0'
await install(manifest, opts)
{
expect(fs.existsSync(path.resolve('node_modules/.pnpm/lock.yaml'))).toBeTruthy()
const files = fs.readdirSync(path.join(globalVirtualStoreDir, '@pnpm.e2e/peer-c/2.0.0'))
expect(files).toHaveLength(1)
expect(fs.existsSync(path.join(globalVirtualStoreDir, '@pnpm.e2e/peer-c/2.0.0', files[0], 'node_modules/@pnpm.e2e/peer-c/package.json'))).toBeTruthy()
}
})
test('GVS hashes are engine-agnostic for packages not in allowBuilds', async () => {
prepareEmpty()
const manifest = {
dependencies: {
'@pnpm.e2e/pkg-with-1-dep': '100.0.0',
},
}
// Scenario 1: No packages allowed to build — all hashes should be engine-agnostic
const gvsDir1 = path.resolve('links1')
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: gvsDir1,
allowBuilds: {},
}))
rimrafSync('node_modules')
// Scenario 2: Dependency allowed to build — parent hash becomes engine-specific
// because it transitively depends on a package that is allowed to build
const gvsDir2 = path.resolve('links2')
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: gvsDir2,
frozenLockfile: true,
allowBuilds: { '@pnpm.e2e/dep-of-pkg-with-1-dep': true },
}))
// Read hash directories for the parent package from both scenarios
const hashNoBuilds = fs.readdirSync(path.join(gvsDir1, '@pnpm.e2e/pkg-with-1-dep/100.0.0'))[0]
const hashWithBuilds = fs.readdirSync(path.join(gvsDir2, '@pnpm.e2e/pkg-with-1-dep/100.0.0'))[0]
// Hashes must differ: scenario 1 omits ENGINE_NAME, scenario 2 includes it
// (because dep-of-pkg-with-1-dep is allowed to build)
expect(hashNoBuilds).not.toBe(hashWithBuilds)
// Both scenarios should still produce valid GVS layouts
expect(fs.existsSync(path.join(gvsDir1, '@pnpm.e2e/pkg-with-1-dep/100.0.0', hashNoBuilds, 'node_modules/@pnpm.e2e/pkg-with-1-dep/package.json'))).toBeTruthy()
expect(fs.existsSync(path.join(gvsDir2, '@pnpm.e2e/pkg-with-1-dep/100.0.0', hashWithBuilds, 'node_modules/@pnpm.e2e/pkg-with-1-dep/package.json'))).toBeTruthy()
})
test('GVS hashes are stable when allowBuilds targets an unrelated package', async () => {
prepareEmpty()
const manifest = {
dependencies: {
'@pnpm.e2e/pkg-with-1-dep': '100.0.0',
},
}
// Scenario 1: No packages allowed to build
const gvsDir1 = path.resolve('links1')
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: gvsDir1,
allowBuilds: {},
}))
rimrafSync('node_modules')
// Scenario 2: An unrelated package allowed to build
// This should NOT affect hashes of @pnpm.e2e/pkg-with-1-dep or its deps
const gvsDir2 = path.resolve('links2')
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: gvsDir2,
frozenLockfile: true,
allowBuilds: { 'some-unrelated-package': true },
}))
// Hashes should be identical since the allowBuilds target is not in the dep tree
const hash1 = fs.readdirSync(path.join(gvsDir1, '@pnpm.e2e/pkg-with-1-dep/100.0.0'))[0]
const hash2 = fs.readdirSync(path.join(gvsDir2, '@pnpm.e2e/pkg-with-1-dep/100.0.0'))[0]
expect(hash1).toBe(hash2)
})
test('GVS re-links when allowBuilds changes', async () => {
prepareEmpty()
const globalVirtualStoreDir = path.resolve('links')
const manifest = {
dependencies: {
'@pnpm.e2e/pkg-with-1-dep': '100.0.0',
},
}
// Step 1: Install with no packages allowed to build (engine-agnostic hashes)
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
allowBuilds: {},
}))
const hashBefore = fs.readdirSync(path.join(globalVirtualStoreDir, '@pnpm.e2e/pkg-with-1-dep/100.0.0'))[0]
// Verify allowBuilds is stored in modules.yaml
const rootModules = assertProject(process.cwd())
const modulesState = rootModules.readModulesManifest()
expect(modulesState?.allowBuilds).toEqual({})
// Step 2: Reinstall with dep allowed to build — hashes should change
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
allowBuilds: { '@pnpm.e2e/dep-of-pkg-with-1-dep': true },
}))
const hashAfter = fs.readdirSync(path.join(globalVirtualStoreDir, '@pnpm.e2e/pkg-with-1-dep/100.0.0'))
.find((h) => h !== hashBefore)
// A new hash directory should have been created
expect(hashAfter).toBeDefined()
expect(hashAfter).not.toBe(hashBefore)
// Verify the new GVS layout is valid
expect(fs.existsSync(path.join(globalVirtualStoreDir, '@pnpm.e2e/pkg-with-1-dep/100.0.0', hashAfter!, 'node_modules/@pnpm.e2e/pkg-with-1-dep/package.json'))).toBeTruthy()
// Verify modules.yaml is updated with new allowBuilds
const updatedState = rootModules.readModulesManifest()
expect(updatedState?.allowBuilds).toEqual({ '@pnpm.e2e/dep-of-pkg-with-1-dep': true })
})
test('GVS successful build creates package directory with build artifacts', async () => {
prepareEmpty()
const globalVirtualStoreDir = path.resolve('links')
const manifest = {
dependencies: {
'@pnpm.e2e/pre-and-postinstall-scripts-example': '1.0.0',
},
}
const opts = testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
fastUnpack: false,
allowBuilds: { '@pnpm.e2e/pre-and-postinstall-scripts-example': true },
})
await install(manifest, opts)
// The GVS directory should exist with build artifacts
const pkgDir = path.join(globalVirtualStoreDir, '@pnpm.e2e/pre-and-postinstall-scripts-example/1.0.0')
const hashes = fs.readdirSync(pkgDir)
expect(hashes).toHaveLength(1)
const pkgInGvs = path.join(pkgDir, hashes[0], 'node_modules/@pnpm.e2e/pre-and-postinstall-scripts-example')
expect(fs.existsSync(path.join(pkgInGvs, 'package.json'))).toBeTruthy()
// Build artifacts created by postinstall script should be present
expect(fs.existsSync(path.join(pkgInGvs, 'generated-by-postinstall.js'))).toBeTruthy()
expect(fs.existsSync(path.join(pkgInGvs, 'generated-by-preinstall.js'))).toBeTruthy()
// The .pnpm-needs-build marker should have been removed after successful build
expect(fs.existsSync(path.join(pkgInGvs, '.pnpm-needs-build'))).toBeFalsy()
// The .pnpm-needs-build marker must not be uploaded to the side effects cache
const filesIndexKey = storeIndexKey(getIntegrity('@pnpm.e2e/pre-and-postinstall-scripts-example', '1.0.0'), '@pnpm.e2e/pre-and-postinstall-scripts-example@1.0.0')
const storeIndex = new StoreIndex(opts.storeDir)
storeIndexes.push(storeIndex)
const filesIndex = storeIndex.get(filesIndexKey) as PackageFilesIndex
if (filesIndex.sideEffects) {
for (const [, diff] of filesIndex.sideEffects) {
expect(diff.added?.has('.pnpm-needs-build')).toBeFalsy()
}
}
})
test('GVS: approve-builds scenario — install with no builds, then reinstall with allowBuilds', async () => {
prepareEmpty()
const globalVirtualStoreDir = path.resolve('links')
const manifest = {
dependencies: {
'@pnpm.e2e/pre-and-postinstall-scripts-example': '1.0.0',
},
}
// Step 1: Install with builds NOT approved (simulating first `pnpm install`)
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
fastUnpack: false,
allowBuilds: {},
}))
const pkgVersionDir = path.join(globalVirtualStoreDir, '@pnpm.e2e/pre-and-postinstall-scripts-example/1.0.0')
const hashBefore = fs.readdirSync(pkgVersionDir)
expect(hashBefore).toHaveLength(1)
// Build artifacts should NOT be present
expect(fs.existsSync(path.resolve('node_modules/@pnpm.e2e/pre-and-postinstall-scripts-example/generated-by-postinstall.js'))).toBeFalsy()
// Step 2: Reinstall with allowBuilds changed (simulating what approve-builds does)
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
fastUnpack: false,
allowBuilds: { '@pnpm.e2e/pre-and-postinstall-scripts-example': true },
}))
// Step 3: Verify the hash changed and build artifacts are in the new directory
const hashesAfter = fs.readdirSync(pkgVersionDir)
const newHash = hashesAfter.find((h) => h !== hashBefore[0])
expect(newHash).toBeDefined()
expect(newHash).not.toBe(hashBefore[0])
// Build artifacts in new hash directory
const newPkgDir = path.join(pkgVersionDir, newHash!, 'node_modules/@pnpm.e2e/pre-and-postinstall-scripts-example')
expect(fs.existsSync(path.join(newPkgDir, 'generated-by-postinstall.js'))).toBeTruthy()
expect(fs.existsSync(path.join(newPkgDir, 'generated-by-preinstall.js'))).toBeTruthy()
// Build artifacts accessible via node_modules
expect(fs.existsSync(path.resolve('node_modules/@pnpm.e2e/pre-and-postinstall-scripts-example/generated-by-postinstall.js'))).toBeTruthy()
expect(fs.existsSync(path.resolve('node_modules/@pnpm.e2e/pre-and-postinstall-scripts-example/generated-by-preinstall.js'))).toBeTruthy()
})
test('GVS build failure cleans up broken package directory', async () => {
prepareEmpty()
const globalVirtualStoreDir = path.resolve('links')
const manifest = {
dependencies: {
'@pnpm.e2e/failing-postinstall': '1.0.0',
},
}
await expect(
install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
fastUnpack: false,
allowBuilds: { '@pnpm.e2e/failing-postinstall': true },
}))
).rejects.toThrow()
// The GVS hash directory for the failed package should have been removed
// on build failure so the next install can re-fetch and re-build.
const pkgVersionDir = path.join(globalVirtualStoreDir, '@pnpm.e2e/failing-postinstall/1.0.0')
if (fs.existsSync(pkgVersionDir)) {
const hashes = fs.readdirSync(pkgVersionDir)
for (const hash of hashes) {
const pkgInGvs = path.join(pkgVersionDir, hash, 'node_modules/@pnpm.e2e/failing-postinstall')
expect(fs.existsSync(pkgInGvs)).toBeFalsy()
}
}
})
test('GVS rebuilds successfully after simulated build failure cleanup', async () => {
prepareEmpty()
const globalVirtualStoreDir = path.resolve('links')
const manifest = {
dependencies: {
'@pnpm.e2e/pre-and-postinstall-scripts-example': '1.0.0',
},
}
// Step 1: Successful install with build
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
fastUnpack: false,
allowBuilds: { '@pnpm.e2e/pre-and-postinstall-scripts-example': true },
}))
const pkgDir = path.join(globalVirtualStoreDir, '@pnpm.e2e/pre-and-postinstall-scripts-example/1.0.0')
const hashes = fs.readdirSync(pkgDir)
expect(hashes).toHaveLength(1)
const hashDir = path.join(pkgDir, hashes[0])
expect(fs.existsSync(path.join(hashDir, 'node_modules/@pnpm.e2e/pre-and-postinstall-scripts-example/generated-by-postinstall.js'))).toBeTruthy()
// Step 2: Simulate a previous build failure by removing the GVS hash directory
rimrafSync(hashDir)
expect(fs.existsSync(hashDir)).toBeFalsy()
// Step 3: Remove node_modules and reinstall with frozenLockfile
// The GVS fast path should NOT kick in because the hash dir is gone
rimrafSync('node_modules')
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
frozenLockfile: true,
fastUnpack: false,
allowBuilds: { '@pnpm.e2e/pre-and-postinstall-scripts-example': true },
}))
// The GVS directory should be recreated with build artifacts
const hashesAfter = fs.readdirSync(pkgDir)
expect(hashesAfter).toHaveLength(1)
expect(fs.existsSync(path.join(pkgDir, hashesAfter[0], 'node_modules/@pnpm.e2e/pre-and-postinstall-scripts-example/generated-by-postinstall.js'))).toBeTruthy()
})
test('GVS .pnpm-needs-build marker triggers re-import on next install', async () => {
prepareEmpty()
const globalVirtualStoreDir = path.resolve('links')
const manifest = {
dependencies: {
'@pnpm.e2e/pre-and-postinstall-scripts-example': '1.0.0',
},
}
// Step 1: Install with build
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
fastUnpack: false,
allowBuilds: { '@pnpm.e2e/pre-and-postinstall-scripts-example': true },
}))
const pkgDir = path.join(globalVirtualStoreDir, '@pnpm.e2e/pre-and-postinstall-scripts-example/1.0.0')
const hashes = fs.readdirSync(pkgDir)
expect(hashes).toHaveLength(1)
const hashDir = path.join(pkgDir, hashes[0])
const pkgInGvs = path.join(hashDir, 'node_modules/@pnpm.e2e/pre-and-postinstall-scripts-example')
expect(fs.existsSync(path.join(pkgInGvs, 'generated-by-postinstall.js'))).toBeTruthy()
// Marker should not be present after successful build
expect(fs.existsSync(path.join(pkgInGvs, '.pnpm-needs-build'))).toBeFalsy()
// Step 2: Simulate a crash between import and build — write a .pnpm-needs-build
// marker and remove build artifacts (as if the build never completed)
fs.writeFileSync(path.join(pkgInGvs, '.pnpm-needs-build'), '')
fs.unlinkSync(path.join(pkgInGvs, 'generated-by-postinstall.js'))
expect(fs.existsSync(path.join(pkgInGvs, '.pnpm-needs-build'))).toBeTruthy()
// Remove node_modules to force a re-install
rimrafSync('node_modules')
// Step 3: Reinstall — the GVS fast path should detect the .pnpm-needs-build
// marker and force a re-fetch, re-import, and re-build.
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
frozenLockfile: true,
fastUnpack: false,
allowBuilds: { '@pnpm.e2e/pre-and-postinstall-scripts-example': true },
}))
// The marker should be gone and the package rebuilt with artifacts
expect(fs.existsSync(path.join(pkgInGvs, '.pnpm-needs-build'))).toBeFalsy()
expect(fs.existsSync(path.join(pkgInGvs, 'generated-by-postinstall.js'))).toBeTruthy()
})
test('injected local packages work with global virtual store', async () => {
const project1Manifest = {
name: 'project-1',
version: '1.0.0',
dependencies: {
'is-positive': '1.0.0',
},
}
const project2Manifest = {
name: 'project-2',
version: '1.0.0',
dependencies: {
'project-1': 'workspace:1.0.0',
},
dependenciesMeta: {
'project-1': {
injected: true,
},
},
}
preparePackages([
{
location: 'project-1',
package: project1Manifest,
},
{
location: 'project-2',
package: project2Manifest,
},
])
fs.writeFileSync('project-1/foo.js', '', 'utf8')
const globalVirtualStoreDir = path.resolve('links')
const importers: MutatedProject[] = [
{
mutation: 'install',
rootDir: path.resolve('project-1') as ProjectRootDir,
},
{
mutation: 'install',
rootDir: path.resolve('project-2') as ProjectRootDir,
},
]
const allProjects: ProjectOptions[] = [
{
buildIndex: 0,
manifest: project1Manifest,
rootDir: path.resolve('project-1') as ProjectRootDir,
},
{
buildIndex: 0,
manifest: project2Manifest,
rootDir: path.resolve('project-2') as ProjectRootDir,
},
]
await mutateModules(importers, testDefaults({
autoInstallPeers: false,
allProjects,
enableGlobalVirtualStore: true,
dedupeInjectedDeps: false,
virtualStoreDir: globalVirtualStoreDir,
}))
// Verify project-2 has project-1 installed
expect(fs.existsSync(path.resolve('project-2/node_modules/project-1'))).toBeTruthy()
// Verify the modules manifest has injectedDeps pointing to global virtual store
const rootModules = assertProject(process.cwd())
const modulesState = rootModules.readModulesManifest()
expect(modulesState?.injectedDeps?.['project-1']).toBeDefined()
expect(modulesState?.injectedDeps?.['project-1'].length).toBeGreaterThan(0)
// Injected deps should be in the global virtual store (links directory)
const injectedDepLocation = modulesState?.injectedDeps?.['project-1'][0]
expect(injectedDepLocation).toContain('links')
expect(fs.existsSync(path.join(injectedDepLocation!, 'foo.js'))).toBeTruthy()
})
test('virtualStoreOnly populates standard virtual store without importer symlinks', async () => {
await addDistTag({ package: '@pnpm.e2e/dep-of-pkg-with-1-dep', version: '100.1.0', distTag: 'latest' })
prepareEmpty()
const manifest = {
dependencies: {
'@pnpm.e2e/pkg-with-1-dep': '100.0.0',
},
}
await install(manifest, testDefaults({
virtualStoreOnly: true,
}))
// Standard virtual store should be populated
expect(fs.existsSync(path.resolve('node_modules/.pnpm/@pnpm.e2e+pkg-with-1-dep@100.0.0/node_modules/@pnpm.e2e/pkg-with-1-dep/package.json'))).toBeTruthy()
expect(fs.existsSync(path.resolve('node_modules/.pnpm/@pnpm.e2e+dep-of-pkg-with-1-dep@100.1.0/node_modules/@pnpm.e2e/dep-of-pkg-with-1-dep/package.json'))).toBeTruthy()
// Importer-level symlinks should NOT exist
expect(fs.existsSync(path.resolve('node_modules/@pnpm.e2e/pkg-with-1-dep'))).toBeFalsy()
})
test('virtualStoreOnly with enableModulesDir=false throws config error (standard virtual store)', async () => {
prepareEmpty()
await expect(
install({}, testDefaults({
virtualStoreOnly: true,
enableModulesDir: false,
}))
).rejects.toMatchObject({
code: 'ERR_PNPM_CONFIG_CONFLICT_VIRTUAL_STORE_ONLY_WITH_NO_MODULES_DIR',
})
})
test('virtualStoreOnly with enableModulesDir=false works when GVS is enabled', async () => {
prepareEmpty()
const globalVirtualStoreDir = path.resolve('gvs-no-modules')
const manifest = {
dependencies: {
'@pnpm.e2e/pkg-with-1-dep': '100.0.0',
},
}
// First install to generate lockfile (with modules dir enabled)
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
}))
rimrafSync('node_modules')
rimrafSync(globalVirtualStoreDir)
// Now install with virtualStoreOnly + enableModulesDir=false + GVS — should NOT throw
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
virtualStoreOnly: true,
enableModulesDir: false,
frozenLockfile: true,
}))
// GVS should be populated
const pkgDir = path.join(globalVirtualStoreDir, '@pnpm.e2e/pkg-with-1-dep/100.0.0')
expect(fs.existsSync(pkgDir)).toBeTruthy()
const hashes = fs.readdirSync(pkgDir)
expect(hashes).toHaveLength(1)
expect(fs.existsSync(path.join(pkgDir, hashes[0], 'node_modules/@pnpm.e2e/pkg-with-1-dep/package.json'))).toBeTruthy()
})
test('virtualStoreOnly with GVS populates global virtual store without importer links', async () => {
prepareEmpty()
const globalVirtualStoreDir = path.resolve('gvs')
const manifest = {
dependencies: {
'@pnpm.e2e/pkg-with-1-dep': '100.0.0',
},
}
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
virtualStoreOnly: true,
}))
// GVS should be populated
const pkgDir = path.join(globalVirtualStoreDir, '@pnpm.e2e/pkg-with-1-dep/100.0.0')
expect(fs.existsSync(pkgDir)).toBeTruthy()
const hashes = fs.readdirSync(pkgDir)
expect(hashes).toHaveLength(1)
expect(fs.existsSync(path.join(pkgDir, hashes[0], 'node_modules/@pnpm.e2e/pkg-with-1-dep/package.json'))).toBeTruthy()
expect(fs.existsSync(path.join(pkgDir, hashes[0], 'node_modules/@pnpm.e2e/dep-of-pkg-with-1-dep/package.json'))).toBeTruthy()
// Importer-level links should NOT exist
expect(fs.existsSync(path.resolve('node_modules/@pnpm.e2e/pkg-with-1-dep'))).toBeFalsy()
// No hoisted deps
expect(fs.existsSync(path.resolve('node_modules/.pnpm/node_modules/@pnpm.e2e/dep-of-pkg-with-1-dep'))).toBeFalsy()
// No bin links
expect(fs.existsSync(path.resolve('node_modules/.bin'))).toBeFalsy()
})
test('virtualStoreOnly with frozenLockfile populates virtual store without importer symlinks', async () => {
prepareEmpty()
const globalVirtualStoreDir = path.resolve('gvs-frozen')
const manifest = {
dependencies: {
'@pnpm.e2e/pkg-with-1-dep': '100.0.0',
},
}
// First install to generate lockfile
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
}))
// Remove node_modules and GVS, then reinstall with frozenLockfile + virtualStoreOnly
rimrafSync('node_modules')
rimrafSync(globalVirtualStoreDir)
await install(manifest, testDefaults({
enableGlobalVirtualStore: true,
virtualStoreDir: globalVirtualStoreDir,
virtualStoreOnly: true,
frozenLockfile: true,
}))
// GVS should be populated
const pkgDir = path.join(globalVirtualStoreDir, '@pnpm.e2e/pkg-with-1-dep/100.0.0')
expect(fs.existsSync(pkgDir)).toBeTruthy()
const hashes = fs.readdirSync(pkgDir)
expect(hashes).toHaveLength(1)
expect(fs.existsSync(path.join(pkgDir, hashes[0], 'node_modules/@pnpm.e2e/pkg-with-1-dep/package.json'))).toBeTruthy()
// Transitive dependency should also be in GVS
expect(fs.existsSync(path.join(pkgDir, hashes[0], 'node_modules/@pnpm.e2e/dep-of-pkg-with-1-dep/package.json'))).toBeTruthy()
// Importer-level symlinks should NOT exist
expect(fs.existsSync(path.resolve('node_modules/@pnpm.e2e/pkg-with-1-dep'))).toBeFalsy()
// No hoisted deps
expect(fs.existsSync(path.resolve('node_modules/.pnpm/node_modules/@pnpm.e2e/dep-of-pkg-with-1-dep'))).toBeFalsy()
// No bin links
expect(fs.existsSync(path.resolve('node_modules/.bin'))).toBeFalsy()
})
test('virtualStoreOnly with frozenLockfile populates standard virtual store without importer symlinks', async () => {
await addDistTag({ package: '@pnpm.e2e/dep-of-pkg-with-1-dep', version: '100.1.0', distTag: 'latest' })
prepareEmpty()
const manifest = {
dependencies: {
'@pnpm.e2e/pkg-with-1-dep': '100.0.0',
},
}
// First install to generate lockfile
await install(manifest, testDefaults())
// Remove node_modules, then reinstall with frozenLockfile + virtualStoreOnly
rimrafSync('node_modules')
await install(manifest, testDefaults({
virtualStoreOnly: true,
frozenLockfile: true,
}))
// Standard virtual store should be populated
expect(fs.existsSync(path.resolve('node_modules/.pnpm/@pnpm.e2e+pkg-with-1-dep@100.0.0/node_modules/@pnpm.e2e/pkg-with-1-dep/package.json'))).toBeTruthy()
expect(fs.existsSync(path.resolve('node_modules/.pnpm/@pnpm.e2e+dep-of-pkg-with-1-dep@100.1.0/node_modules/@pnpm.e2e/dep-of-pkg-with-1-dep/package.json'))).toBeTruthy()
// Importer-level symlinks should NOT exist
expect(fs.existsSync(path.resolve('node_modules/@pnpm.e2e/pkg-with-1-dep'))).toBeFalsy()
// No hoisted deps
expect(fs.existsSync(path.resolve('node_modules/.pnpm/node_modules/@pnpm.e2e/dep-of-pkg-with-1-dep'))).toBeFalsy()
// No bin links
expect(fs.existsSync(path.resolve('node_modules/.bin'))).toBeFalsy()
})
test('virtualStoreOnly suppresses hoisting even with explicit hoistPattern', async () => {
prepareEmpty()
const manifest = {
dependencies: {
'@pnpm.e2e/pkg-with-1-dep': '100.0.0',
},
}
await install(manifest, testDefaults({
virtualStoreOnly: true,
hoistPattern: ['*'],
publicHoistPattern: ['*'],
}))
// Virtual store should be populated
expect(fs.existsSync(path.resolve('node_modules/.pnpm/@pnpm.e2e+pkg-with-1-dep@100.0.0/node_modules/@pnpm.e2e/pkg-with-1-dep/package.json'))).toBeTruthy()
// No hoisted packages (despite hoistPattern: ['*'])
expect(fs.existsSync(path.resolve('node_modules/.pnpm/node_modules/@pnpm.e2e/dep-of-pkg-with-1-dep'))).toBeFalsy()
// No importer-level symlinks
expect(fs.existsSync(path.resolve('node_modules/@pnpm.e2e/pkg-with-1-dep'))).toBeFalsy()
})
Reference in New Issue View Git Blame Copy Permalink