github/penpot
1
0
Fork 0
mirror of https://github.com/penpot/penpot.git synced 2026-10-29 22:10:34 +00:00
Code Issues Packages Projects Releases Wiki Activity

✨ Test the detach option on nested broken copies

Browse Source 
Design confirmed the expected behavior: a broken copy nested in a main
offers "Detach instance", and one nested in a copy does not. The fixture
already had both cases ("Banner" with "Nested in main" and its copy with
"Nested in copy"); the tests delete the "Icon" main to break them, check
the broken indicator, and check the options menu: present with Detach
for the first, absent for the second.

AI-assisted-by: claude-opus-5-5
This commit is contained in:
Eva Marco 2026-10-29 13:41:40 +01:00
parent 32d47c3b14
commit 1703cf46b4
1 changed files with 44 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

44
frontend/playwright/ui/specs/broken-copy-indicator.spec.js
View File

@ -75,3 +75,47 @@ test("Shows the broken indicator when every selected copy is broken", async ({
await expect(brokenIndicator(workspacePage)).toBeVisible();
});
// "Banner" is a main with a copy of "Icon" inside ("Nested in main"), and
// "Banner copy" is a copy of it ("Nested in copy"). Deleting the "Icon" main
// breaks both nested copies.
const breakNestedCopies = async (workspacePage) => {
await workspacePage.clickLeafLayer("Icon");
await workspacePage.page.keyboard.press("Delete");
for (const parent of ["Banner", "Banner copy"]) {
await workspacePage.layers
.getByTestId("layer-row")
.filter({ has: workspacePage.page.getByText(parent, { exact: true }) })
.getByTestId("toggle-content")
.click();
}
};
const openComponentMenu = async (workspacePage) => {
await workspacePage.rightSidebar
.getByRole("button", { name: "Options" })
.click();
};
test("Offers detach on a broken copy nested in a main", async ({ page }) => {
const workspacePage = new WorkspacePage(page);
await breakNestedCopies(workspacePage);
await workspacePage.clickLeafLayer("Nested in main");
await expect(brokenIndicator(workspacePage)).toBeVisible();
await openComponentMenu(workspacePage);
await expect(page.getByText("Detach instance")).toBeVisible();
});
test("Offers no detach on a broken copy nested in a copy", async ({ page }) => {
const workspacePage = new WorkspacePage(page);
await breakNestedCopies(workspacePage);
await workspacePage.clickLeafLayer("Nested in copy");
await expect(brokenIndicator(workspacePage)).toBeVisible();
// A copy inside another copy has no menu entries, so no options button.
await expect(
workspacePage.rightSidebar.getByRole("button", { name: "Options" }),
).toHaveCount(0);
});