-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Expand file tree
/
Copy pathlegacy-parameters.test.ts
More file actions
137 lines (125 loc) · 6.09 KB
/
Copy pathlegacy-parameters.test.ts
File metadata and controls
137 lines (125 loc) · 6.09 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
import { describe, expect, it } from "vitest";
import {
isOmittableRequest,
readLegacyParameters,
validateLegacyDefinitions,
validateLegacyRequests,
} from "../../scripts/codegen/legacy-parameters.ts";
import { legacyRequestSchema } from "./legacy-parameters-fixture.ts";
describe("x-legacy-parameters", () => {
it("leaves unmarked requests to their existing generation", () => {
expect(readLegacyParameters(legacyRequestSchema(0).params, "sample.plan")).toBeUndefined();
});
it("keeps the annotation fixed across two successive optional additions", () => {
const one = readLegacyParameters(legacyRequestSchema(1).params, "sample.plan");
const two = readLegacyParameters(legacyRequestSchema(2).params, "sample.plan");
expect(one?.legacy).toEqual(["contract", "source", "scope"]);
expect(two?.legacy).toEqual(one?.legacy);
expect(one?.additions).toEqual(["policySessionId"]);
expect(two?.additions).toEqual(["policySessionId", "traceId"]);
expect([...(two?.required ?? [])]).toEqual(["contract", "source"]);
});
it("excludes implicit session properties from both sets", () => {
const parsed = readLegacyParameters(
legacyRequestSchema(1, "session").params,
"session.sample.plan",
{
implicit: ["sessionId"],
}
);
expect(parsed?.legacy).toEqual(["contract", "source", "scope"]);
expect(parsed?.additions).toEqual(["policySessionId"]);
expect(parsed?.required.has("sessionId")).toBe(false);
});
it.each([
["a non-array annotation", "contract", {}, "expected an array"],
["duplicate names", ["contract", "contract", "source"], {}, "duplicate"],
["an unknown property", ["contract", "source", "missing"], {}, "unknown property missing"],
["an omitted required input", ["contract", "scope"], {}, "required property source"],
[
"no added property",
["contract", "source", "scope", "policySessionId"],
{},
"at least one property",
],
["a nullable request", ["contract", "source", "scope"], { nullable: true }, "nullable"],
] as const)("rejects %s", (_name, legacy, options, message) => {
const { params } = legacyRequestSchema(1);
params["x-legacy-parameters"] = legacy;
expect(() => readLegacyParameters(params, "sample.plan", options)).toThrow(message);
});
it("rejects an implicit session property listed as a legacy parameter", () => {
const { params } = legacyRequestSchema(1, "session");
params["x-legacy-parameters"] = ["sessionId", "contract", "source"];
expect(() =>
readLegacyParameters(params, "session.sample.plan", { implicit: ["sessionId"] })
).toThrow("implicit property sessionId");
});
it("validates every section for generators with no extra projection", () => {
type Methods = Record<string, { rpcMethod: string; params: unknown }>;
const validate = (sections: Record<string, Methods>) =>
validateLegacyRequests(
sections,
(node) => Object.values(node),
(method) => method.params,
() => false
);
const session = legacyRequestSchema(1, "session").params;
expect(() =>
validate({ session: { plan: { rpcMethod: "session.sample.plan", params: session } } })
).not.toThrow();
const client = legacyRequestSchema(1).params;
expect(() =>
validate({ clientSession: { plan: { rpcMethod: "sample.plan", params: client } } })
).toThrow(
"Invalid x-legacy-parameters for sample.plan: only server and session requests are supported"
);
const invalid = legacyRequestSchema(1).params;
invalid["x-legacy-parameters"] = ["contract", "scope"];
expect(() =>
validate({ server: { plan: { rpcMethod: "sample.plan", params: invalid } } })
).toThrow("required property source must be a legacy parameter");
});
it("rejects optional requests before nullable ones", () => {
const { params } = legacyRequestSchema(1);
expect(() =>
readLegacyParameters(params, "sample.plan", { optional: true, nullable: true })
).toThrow(
"Invalid x-legacy-parameters for sample.plan: optional requests cannot declare legacy parameters"
);
expect(isOmittableRequest({ anyOf: [{ not: {} }, params] })).toBe(true);
expect(isOmittableRequest(params)).toBe(false);
});
it("requires schema property order when a positional record constructor is kept", () => {
const record = {
type: "object",
properties: { name: {}, owned: {}, status: {}, error: {} },
required: ["name", "status"],
"x-legacy-parameters": ["name", "status", "error"],
};
expect(readLegacyParameters(record, "Server", { ordered: true })?.additions).toEqual([
"owned",
]);
const misordered = { ...record, "x-legacy-parameters": ["status", "name", "error"] };
expect(readLegacyParameters(misordered, "Server")?.additions).toEqual(["owned"]);
expect(() => readLegacyParameters(misordered, "Server", { ordered: true })).toThrow(
"Invalid x-legacy-parameters for Server: legacy parameters must follow schema property order"
);
});
it("validates response definitions that no request path reads", () => {
const record = {
type: "object",
properties: { name: {}, status: {}, owned: {} },
required: ["name", "status"],
"x-legacy-parameters": ["name", "status"],
};
expect(() => validateLegacyDefinitions({ definitions: { Server: record } })).not.toThrow();
expect(() =>
validateLegacyDefinitions({
$defs: { Server: { ...record, "x-legacy-parameters": ["name"] } },
})
).toThrow(
"Invalid x-legacy-parameters for Server: required property status must be a legacy parameter"
);
});
});