Zoltan Kochan
fc2f33912e
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.
82 lines
2.7 KiB
TypeScript
82 lines
2.7 KiB
TypeScript
import { describe, expect, it } from '@jest/globals'
|
|
|
|
import { removePort } from '../src/helpers/removePort.js'
|
|
|
|
describe('removePort()', () => {
|
|
it('does not mutate the url if no port is found', () => {
|
|
const urlString = 'https://custom.domain.com/npm/-/foo-1.0.0.tgz'
|
|
expect(removePort(new URL(urlString))).toEqual(urlString)
|
|
|
|
const urlStringWithTrailingSlash = 'https://custom.domain.com/npm/'
|
|
expect(removePort(new URL(urlStringWithTrailingSlash))).toEqual(
|
|
urlStringWithTrailingSlash
|
|
)
|
|
})
|
|
|
|
it('removes ports from urls with https | https | ws | wss protocols', () => {
|
|
const portsToTest = [1, 8888, 8080, 8081, 65535]
|
|
const protocols = ['http', 'https', 'ws', 'wss']
|
|
|
|
const getUrl = (port: number, protocol: string) =>
|
|
new URL(`${protocol}://custom.domain.com:${port}/artifactory/api/npm/npm-virtual/-/foo-1.0.0.tgz`)
|
|
|
|
const expectedOutput = (protocol: string) =>
|
|
`${protocol}://custom.domain.com/artifactory/api/npm/npm-virtual/-/foo-1.0.0.tgz`
|
|
|
|
for (const port of portsToTest) {
|
|
for (const protocol of protocols) {
|
|
expect(removePort(getUrl(port, protocol))).toEqual(
|
|
expectedOutput(protocol)
|
|
)
|
|
}
|
|
}
|
|
})
|
|
|
|
it('removes ports from valid urls with http, https, ws, wss protocols', () => {
|
|
const portsWithEmptyReturns = new Map([
|
|
['http', 80],
|
|
['https', 443],
|
|
['ws', 80],
|
|
['wss', 443],
|
|
])
|
|
|
|
const getUrl = (port: number, protocol: string) =>
|
|
new URL(`${protocol}://custom.domain.com:${port}/artifactory/api/npm/npm-virtual/-/foo-1.0.0.tgz`)
|
|
|
|
const expectedOutput = (protocol: string) =>
|
|
`${protocol}://custom.domain.com/artifactory/api/npm/npm-virtual/-/foo-1.0.0.tgz`
|
|
|
|
portsWithEmptyReturns.forEach((value: number, protocol) => {
|
|
expect(removePort(getUrl(value, protocol))).toEqual(
|
|
expectedOutput(protocol)
|
|
)
|
|
})
|
|
})
|
|
|
|
/**
|
|
* @description intentionally mismatch the port
|
|
* https|wss set to 443
|
|
* http|ws set to 80
|
|
*
|
|
* @tests regexp loopholes of (80:443)
|
|
*/
|
|
it('removes the ports from urls with protocol port mismatches', () => {
|
|
const mismatchProtocolPorts = new Map([
|
|
['http', 443],
|
|
['ws', 443],
|
|
['https', 80],
|
|
['wss', 80],
|
|
])
|
|
|
|
const getUrl = (port: number, protocol: string) =>
|
|
new URL(`${protocol}://custom.domain.com:${port}/artifactory/api/npm/npm-virtual/-/foo-1.0.0.tgz`)
|
|
const expectedOutput = (protocol: string) =>
|
|
`${protocol}://custom.domain.com/artifactory/api/npm/npm-virtual/-/foo-1.0.0.tgz`
|
|
mismatchProtocolPorts.forEach((value: number, protocol) => {
|
|
expect(removePort(getUrl(value, protocol))).toEqual(
|
|
expectedOutput(protocol)
|
|
)
|
|
})
|
|
})
|
|
})
|