refactor(ui): remove superseded component scaffolds
This commit is contained in:
@@ -38,7 +38,7 @@ test("generated component reference documents every bundled component and its pr
|
||||
readFileSync(join(uiComponentsDir(), "..", "component-catalog.json"), "utf8"),
|
||||
) as { components: Array<{ name: string }> };
|
||||
expect(reference.count).toBe(uiComponentNames().length);
|
||||
expect(reference.count).toBe(108);
|
||||
expect(reference.count).toBe(103);
|
||||
expect(reference.components.map((component) => component.name)).toEqual(
|
||||
expect.arrayContaining(uiComponentNames()),
|
||||
);
|
||||
@@ -81,7 +81,6 @@ test("interactive components expose their must-have public event contracts", ()
|
||||
Accordion: ["change", "open", "close"],
|
||||
Modal: ["open", "close", "cancel", "confirm"],
|
||||
DataTable: ["sort", "select", "change", "rowClick", "pageChange"],
|
||||
FileUpload: ["select", "upload", "progress", "success", "error", "cancel", "remove"],
|
||||
WysiwygEditor: ["input", "change", "focus", "blur"],
|
||||
};
|
||||
|
||||
@@ -140,7 +139,7 @@ test("removed duplicate aliases have valid canonical migration targets", () => {
|
||||
const names = new Set(uiComponentNames());
|
||||
|
||||
expect(Object.keys(migration.replacements)).toHaveLength(migration.removedCount);
|
||||
expect(migration.removedCount).toBe(0);
|
||||
expect(migration.removedCount).toBe(5);
|
||||
for (const [removed, replacement] of Object.entries(migration.replacements)) {
|
||||
expect(names.has(removed)).toBe(false);
|
||||
expect(names.has(replacement)).toBe(true);
|
||||
@@ -183,7 +182,6 @@ test("screenshot-directed canonical components are bundled", () => {
|
||||
"Tooltip",
|
||||
"DataTable",
|
||||
"Chart",
|
||||
"FileUpload",
|
||||
"Map",
|
||||
"WysiwygEditor",
|
||||
];
|
||||
@@ -3176,7 +3174,7 @@ test("no component gains an output that nothing ever emits", () => {
|
||||
* A ceiling, not a target. It only ever moves down: rebuilding one of these
|
||||
* components should tighten it.
|
||||
*/
|
||||
expect(offenders.length).toBeLessThanOrEqual(22);
|
||||
expect(offenders.length).toBeLessThanOrEqual(7);
|
||||
expect(offenders).not.toContain("LayoutSplitter.sizeChange");
|
||||
expect(offenders).not.toContain("CustomScrollbar.scroll");
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user