Skip to content

Commit 835299e

Browse files
leonardomendixgjulivan
authored andcommitted
test: fix phone leaf node e2e
1 parent b4a4ada commit 835299e

1 file changed

Lines changed: 11 additions & 8 deletions

File tree

packages/pluggableWidgets/tree-node-web/e2e/TreeNode.spec.js

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -146,14 +146,17 @@ test.describe("v2: startExpanded", () => {
146146
const widget = page.locator(".mx-name-treeNodeV2_2");
147147
// All levels should be present with aria-expanded="true"
148148
await expect(widget.getByRole("treeitem", { name: "Electronics" }).first()).toBeVisible();
149-
await expect(widget.getByRole("treeitem", { name: "Phones" }).first()).toBeVisible();
150-
// Deep nodes load via network — allow extra time
151-
await expect(widget.getByRole("treeitem", { name: "Android" })).toHaveAttribute("aria-expanded", "true", {
152-
timeout: 8000
153-
});
154-
await expect(widget.getByRole("treeitem", { name: "iOS" })).toHaveAttribute("aria-expanded", "true", {
155-
timeout: 8000
156-
});
149+
// Phones has children — assert it is expanded
150+
await expect(widget.getByRole("treeitem", { name: "Phones" }).first()).toHaveAttribute(
151+
"aria-expanded",
152+
"true",
153+
{
154+
timeout: 8000
155+
}
156+
);
157+
// Android and iOS are leaf nodes — aria-expanded is not set on them; just verify they are visible
158+
await expect(widget.getByRole("treeitem", { name: "Android" })).toBeVisible({ timeout: 8000 });
159+
await expect(widget.getByRole("treeitem", { name: "iOS" })).toBeVisible({ timeout: 8000 });
157160
await expect(widget.getByRole("treeitem", { name: "Men", exact: true })).toBeVisible();
158161
});
159162

0 commit comments

Comments
 (0)