From d3c9361159b8152a13eb5b3c2114c4ae84e391f1 Mon Sep 17 00:00:00 2001 From: Victor Giers Date: Fri, 3 Apr 2026 00:46:37 +0200 Subject: [PATCH] Refactor e2e test for importing Draco model asset --- tests/e2e/import-draco-model-asset.e2e.ts | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/tests/e2e/import-draco-model-asset.e2e.ts b/tests/e2e/import-draco-model-asset.e2e.ts index 9754814d..157416c0 100644 --- a/tests/e2e/import-draco-model-asset.e2e.ts +++ b/tests/e2e/import-draco-model-asset.e2e.ts @@ -33,8 +33,9 @@ test("imports a draco-compressed glb asset, places an instance, and survives rel await page.getByTestId("outliner-add-button").click(); await page.getByTestId("add-menu-assets").click(); await page.getByTestId("add-menu-assets-models").click(); - await expect(page.getByRole("button", { name: "tiny-triangle-draco.glb" })).toBeVisible(); - await page.getByRole("button", { name: "tiny-triangle-draco.glb" }).click(); + const addMenu = page.getByRole("menu", { name: "Add" }); + await expect(addMenu.getByRole("menuitem", { name: "tiny-triangle-draco.glb" })).toBeVisible(); + await addMenu.getByRole("menuitem", { name: "tiny-triangle-draco.glb" }).click(); const importedSnapshot = await getEditorStoreSnapshot(page); const importedModelAsset = Object.values(importedSnapshot.document.assets).find( (asset) => asset.kind === "model" && asset.sourceName === "tiny-triangle-draco.glb" @@ -77,7 +78,7 @@ test("imports a draco-compressed glb asset, places an instance, and survives rel await page.getByTestId("outliner-add-button").click(); await page.getByTestId("add-menu-assets").click(); await page.getByTestId("add-menu-assets-models").click(); - await expect(page.getByRole("button", { name: "tiny-triangle-draco.glb" })).toBeVisible(); + await expect(page.getByRole("menu", { name: "Add" }).getByRole("menuitem", { name: "tiny-triangle-draco.glb" })).toBeVisible(); await expect(page.getByTestId("outliner-model-instance-list").getByRole("button")).toHaveCount(2); await expect(page.getByTestId("asset-status-message")).toHaveCount(0);