Files
pnpm/pnpm11/object/property-path/test/delete.test.ts
Zoltan Kochan fc2f33912e refactor: move the TypeScript pnpm CLI into a pnpm11/ directory (#12537)
The TypeScript pnpm CLI freezes at v11; pnpm 12 will be the Rust pacquet
port. To make that split legible, all TypeScript source, test, and build
directories move under a new top-level pnpm11/ directory. The name states
the version boundary rather than implying a behavioral fork, since the two
stacks are meant to behave identically.

Scope is source-only: the shared workspace root stays at the repo root.
pnpm-workspace.yaml, package.json, pnpm-lock.yaml, .pnpmfile.cjs,
.meta-updater, __patches__, .changeset, .husky, and the lint/spell configs
remain in place, so one pnpm workspace and one Cargo workspace still span
all three products. pnpr/client and pacquet/tasks/registry-mock stay as
cross-product workspace members.

Rewiring the move required:
- pnpm-workspace.yaml globs prefixed with pnpm11/
- root package.json script paths, eslint.config.mjs, tsconfig.lint.json,
  .gitignore, and CODEOWNERS updated
- .meta-updater/src/index.ts literals repointed (pnpm11/pnpm/package.json,
  pnpm11/__utils__, pnpm11/__typings__, and the main package directory)
- regenerated every moved package's repository/homepage URL via meta-updater
- pnpm11/pnpm/bundle-deps.ts and __utils__/scripts/src/typecheck-only.ts
  climb one more level to reach the repo root

.meta-updater stays at the repo root because @pnpm/meta-updater resolves
its config at <cwd>/.meta-updater/main.mjs.

TS CI (.github/workflows/ci.yml) now only runs when pnpm11/-relevant paths
change, via a dorny/paths-filter changes job plus a TS CI / Success
aggregate gate; branch protection should require only that gate.
2026-06-20 14:36:25 +02:00

65 lines
2.3 KiB
TypeScript

import { expect, test } from '@jest/globals'
import {
deleteObjectValueByPropertyPathString,
type UnsafePropertyPathKeyError,
} from '../src/index.js'
test('deletes a top-level key', () => {
const obj: Record<string, unknown> = { name: 'foo', version: '1.0.0' }
deleteObjectValueByPropertyPathString(obj, 'version')
expect(obj).toEqual({ name: 'foo' })
})
test('deletes a nested key', () => {
const obj: Record<string, unknown> = { scripts: { build: 'tsc', test: 'jest' } }
deleteObjectValueByPropertyPathString(obj, 'scripts.test')
expect(obj).toEqual({ scripts: { build: 'tsc' } })
})
test('removes an array element without leaving a hole', () => {
const obj: Record<string, unknown> = { contributors: [{ name: 'Alice' }, { name: 'Bob' }] }
deleteObjectValueByPropertyPathString(obj, 'contributors[0]')
expect(obj).toEqual({ contributors: [{ name: 'Bob' }] })
})
test('removes an array element by string index without leaving a hole', () => {
const obj: Record<string, unknown> = { contributors: [{ name: 'Alice' }, { name: 'Bob' }] }
deleteObjectValueByPropertyPathString(obj, 'contributors["0"]')
expect(obj).toEqual({ contributors: [{ name: 'Bob' }] })
})
test('no-op on a missing path', () => {
const obj: Record<string, unknown> = { name: 'foo' }
deleteObjectValueByPropertyPathString(obj, 'scripts.test')
expect(obj).toEqual({ name: 'foo' })
})
test('no-op when an intermediate value is null', () => {
const obj: Record<string, unknown> = { a: null }
deleteObjectValueByPropertyPathString(obj, 'a.b')
expect(obj).toEqual({ a: null })
})
test('no-op when an intermediate value is a scalar', () => {
const obj: Record<string, unknown> = { a: 'scalar' }
deleteObjectValueByPropertyPathString(obj, 'a.b')
expect(obj).toEqual({ a: 'scalar' })
})
test('no-op on an empty property path', () => {
const obj: Record<string, unknown> = { name: 'foo' }
deleteObjectValueByPropertyPathString(obj, '')
expect(obj).toEqual({ name: 'foo' })
})
test('rejects __proto__, constructor and prototype keys', () => {
for (const unsafe of ['__proto__', 'constructor', 'prototype']) {
expect(() => deleteObjectValueByPropertyPathString({}, `${unsafe}.foo`))
.toThrow(expect.objectContaining({
code: 'ERR_PNPM_UNSAFE_PROPERTY_PATH_KEY',
key: unsafe,
} as Partial<UnsafePropertyPathKeyError>))
}
})