Skip to content

Commit a0c3cd6

Browse files
committed
test: cover npm pack override reference
1 parent e550e83 commit a0c3cd6

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

test/scripts/root-package-overrides.test.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ import path from "node:path";
33
import { describe, expect, it } from "vitest";
44

55
type RootPackageManifest = {
6+
dependencies?: Record<string, string>;
67
overrides?: Record<string, string>;
78
pnpm?: {
89
overrides?: Record<string, string>;
@@ -17,10 +18,13 @@ function readRootManifest(): RootPackageManifest {
1718
describe("root package override guardrails", () => {
1819
it("pins the Bedrock runtime below the Windows ARM Node 24 npm resolver failure", () => {
1920
const manifest = readRootManifest();
21+
const packageName = "@aws-sdk/client-bedrock-runtime";
22+
const npmOverride = manifest.overrides?.[packageName];
2023
const pnpmOverride = manifest.pnpm?.overrides?.["@aws-sdk/client-bedrock-runtime"];
2124

2225
expect(pnpmOverride).toBe("3.1024.0");
23-
expect(manifest.overrides?.["@aws-sdk/client-bedrock-runtime"]).toBe(pnpmOverride);
26+
expect(manifest.dependencies?.[packageName]).toBeDefined();
27+
expect(npmOverride).toBe(`$${packageName}`);
2428
});
2529

2630
it("pins the node-domexception alias exactly in npm and pnpm overrides", () => {

0 commit comments

Comments
 (0)