Skip to content

Commit a855414

Browse files
committed
test: fix bundled lint and override guard
1 parent 4cc7869 commit a855414

2 files changed

Lines changed: 11 additions & 3 deletions

File tree

extensions/oc-path/src/oc-path/jsonc/parse.ts

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import {
22
ParseErrorCode,
3-
type Node as JsoncParserNode,
43
type ParseError,
54
parseTree,
65
printParseErrorCode,
@@ -35,6 +34,14 @@ type LineMap = {
3534
lineForOffset(offset: number): number;
3635
};
3736

37+
type JsoncParserNode = {
38+
readonly type: "array" | "boolean" | "null" | "number" | "object" | "property" | "string";
39+
readonly offset: number;
40+
readonly length: number;
41+
readonly value?: unknown;
42+
readonly children?: readonly JsoncParserNode[];
43+
};
44+
3845
export function parseJsonc(raw: string): JsoncParseResult {
3946
if (raw.trim().length === 0) {
4047
return { ast: { kind: "jsonc", raw, root: null }, diagnostics: [] };
@@ -64,7 +71,7 @@ export function parseJsonc(raw: string): JsoncParseResult {
6471
allowTrailingComma: true,
6572
disallowComments: false,
6673
allowEmptyContent: true,
67-
});
74+
}) as JsoncParserNode | undefined;
6875
const lineMap = createLineMap(raw);
6976
const diagnostics = errors.map((error) => toDiagnostic(error, lineMap, tree));
7077
let root: JsoncValue | null = null;

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

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,12 @@ describe("root package override guardrails", () => {
1919
it("pins the Bedrock runtime below the Windows ARM Node 24 npm resolver failure", () => {
2020
const manifest = readRootManifest();
2121
const packageName = "@aws-sdk/client-bedrock-runtime";
22+
const dependencyVersion = manifest.dependencies?.[packageName];
2223
const npmOverride = manifest.overrides?.[packageName];
2324
const pnpmOverride = manifest.pnpm?.overrides?.["@aws-sdk/client-bedrock-runtime"];
2425

25-
expect(pnpmOverride).toBe("3.1024.0");
2626
expect(manifest.dependencies).toHaveProperty(packageName);
27+
expect(pnpmOverride).toBe(dependencyVersion);
2728
expect(npmOverride).toBe(`$${packageName}`);
2829
});
2930

0 commit comments

Comments
 (0)