skir 1.0.9 → 1.0.10
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/dist/error_renderer.js
CHANGED
|
@@ -224,11 +224,11 @@ export function getShortMessageForBreakingChange(breakingChange, before) {
|
|
|
224
224
|
switch (breakingChange.kind) {
|
|
225
225
|
case "illegal-type-change": {
|
|
226
226
|
const { type } = breakingChange;
|
|
227
|
-
return "
|
|
227
|
+
return "illegal type change; was: " + formatType(type.before, before);
|
|
228
228
|
}
|
|
229
229
|
case "missing-slots": {
|
|
230
230
|
const { missingRangeEnd } = breakingChange;
|
|
231
|
-
return `
|
|
231
|
+
return `missing slots; had ${missingRangeEnd}`;
|
|
232
232
|
}
|
|
233
233
|
case "variant-kind-change": {
|
|
234
234
|
const { record, variantName } = breakingChange;
|
|
@@ -236,18 +236,18 @@ export function getShortMessageForBreakingChange(breakingChange, before) {
|
|
|
236
236
|
const variantKind = caseMatches(variantName.before.text, "lower_underscore")
|
|
237
237
|
? "wrapper"
|
|
238
238
|
: "constant";
|
|
239
|
-
return `
|
|
239
|
+
return `was a ${variantKind} variant`;
|
|
240
240
|
}
|
|
241
241
|
case "missing-variant": {
|
|
242
242
|
const { number } = breakingChange;
|
|
243
|
-
return `
|
|
243
|
+
return `missing variant: ${number}`;
|
|
244
244
|
}
|
|
245
245
|
case "record-kind-change": {
|
|
246
246
|
const { recordType } = breakingChange;
|
|
247
|
-
return recordType.before === "struct" ? "
|
|
247
|
+
return recordType.before === "struct" ? "was a struct" : "was an enum";
|
|
248
248
|
}
|
|
249
249
|
case "removed-number-reintroduced": {
|
|
250
|
-
return "
|
|
250
|
+
return "number was marked as removed";
|
|
251
251
|
}
|
|
252
252
|
case "missing-record":
|
|
253
253
|
case "missing-method": {
|
|
@@ -15,7 +15,7 @@ describe("getShortMessageForBreakingChange", () => {
|
|
|
15
15
|
after: { kind: "primitive", primitive: "string" },
|
|
16
16
|
},
|
|
17
17
|
};
|
|
18
|
-
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("
|
|
18
|
+
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("illegal type change; was: int32");
|
|
19
19
|
});
|
|
20
20
|
it("missing-slots", () => {
|
|
21
21
|
const breakingChange = {
|
|
@@ -31,7 +31,7 @@ describe("getShortMessageForBreakingChange", () => {
|
|
|
31
31
|
missingRangeStart: 1,
|
|
32
32
|
missingRangeEnd: 2,
|
|
33
33
|
};
|
|
34
|
-
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("
|
|
34
|
+
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("missing slots; had 2");
|
|
35
35
|
});
|
|
36
36
|
it("variant-kind-change (constant to wrapper)", () => {
|
|
37
37
|
const breakingChange = {
|
|
@@ -54,7 +54,7 @@ describe("getShortMessageForBreakingChange", () => {
|
|
|
54
54
|
},
|
|
55
55
|
number: 1,
|
|
56
56
|
};
|
|
57
|
-
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("
|
|
57
|
+
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("was a constant variant");
|
|
58
58
|
});
|
|
59
59
|
it("variant-kind-change (wrapper to constant)", () => {
|
|
60
60
|
const breakingChange = {
|
|
@@ -77,7 +77,7 @@ describe("getShortMessageForBreakingChange", () => {
|
|
|
77
77
|
},
|
|
78
78
|
number: 1,
|
|
79
79
|
};
|
|
80
|
-
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("
|
|
80
|
+
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("was a wrapper variant");
|
|
81
81
|
});
|
|
82
82
|
it("missing-variant", () => {
|
|
83
83
|
const breakingChange = {
|
|
@@ -97,7 +97,7 @@ describe("getShortMessageForBreakingChange", () => {
|
|
|
97
97
|
variantName: { text: "BAR" },
|
|
98
98
|
number: 1,
|
|
99
99
|
};
|
|
100
|
-
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("
|
|
100
|
+
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("missing variant: 1");
|
|
101
101
|
});
|
|
102
102
|
it("record-kind-change (struct to enum)", () => {
|
|
103
103
|
const breakingChange = {
|
|
@@ -115,7 +115,7 @@ describe("getShortMessageForBreakingChange", () => {
|
|
|
115
115
|
after: "enum",
|
|
116
116
|
},
|
|
117
117
|
};
|
|
118
|
-
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("
|
|
118
|
+
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("was a struct");
|
|
119
119
|
});
|
|
120
120
|
it("record-kind-change (enum to struct)", () => {
|
|
121
121
|
const breakingChange = {
|
|
@@ -133,7 +133,7 @@ describe("getShortMessageForBreakingChange", () => {
|
|
|
133
133
|
after: "struct",
|
|
134
134
|
},
|
|
135
135
|
};
|
|
136
|
-
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("
|
|
136
|
+
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("was an enum");
|
|
137
137
|
});
|
|
138
138
|
it("removed-number-reintroduced", () => {
|
|
139
139
|
const breakingChange = {
|
|
@@ -149,7 +149,7 @@ describe("getShortMessageForBreakingChange", () => {
|
|
|
149
149
|
reintroducedAs: { text: "bar" },
|
|
150
150
|
removedNumber: 1,
|
|
151
151
|
};
|
|
152
|
-
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("
|
|
152
|
+
expect(getShortMessageForBreakingChange(breakingChange, mockModuleSet)).toBe("number was marked as removed");
|
|
153
153
|
});
|
|
154
154
|
it("missing-record", () => {
|
|
155
155
|
const breakingChange = {
|
package/package.json
CHANGED
package/src/error_renderer.ts
CHANGED
|
@@ -269,11 +269,11 @@ export function getShortMessageForBreakingChange(
|
|
|
269
269
|
switch (breakingChange.kind) {
|
|
270
270
|
case "illegal-type-change": {
|
|
271
271
|
const { type } = breakingChange;
|
|
272
|
-
return "
|
|
272
|
+
return "illegal type change; was: " + formatType(type.before, before);
|
|
273
273
|
}
|
|
274
274
|
case "missing-slots": {
|
|
275
275
|
const { missingRangeEnd } = breakingChange;
|
|
276
|
-
return `
|
|
276
|
+
return `missing slots; had ${missingRangeEnd}`;
|
|
277
277
|
}
|
|
278
278
|
case "variant-kind-change": {
|
|
279
279
|
const { record, variantName } = breakingChange;
|
|
@@ -284,18 +284,18 @@ export function getShortMessageForBreakingChange(
|
|
|
284
284
|
)
|
|
285
285
|
? "wrapper"
|
|
286
286
|
: "constant";
|
|
287
|
-
return `
|
|
287
|
+
return `was a ${variantKind} variant`;
|
|
288
288
|
}
|
|
289
289
|
case "missing-variant": {
|
|
290
290
|
const { number } = breakingChange;
|
|
291
|
-
return `
|
|
291
|
+
return `missing variant: ${number}`;
|
|
292
292
|
}
|
|
293
293
|
case "record-kind-change": {
|
|
294
294
|
const { recordType } = breakingChange;
|
|
295
|
-
return recordType.before === "struct" ? "
|
|
295
|
+
return recordType.before === "struct" ? "was a struct" : "was an enum";
|
|
296
296
|
}
|
|
297
297
|
case "removed-number-reintroduced": {
|
|
298
|
-
return "
|
|
298
|
+
return "number was marked as removed";
|
|
299
299
|
}
|
|
300
300
|
case "missing-record":
|
|
301
301
|
case "missing-method": {
|