From 4ebbafe4939f0e2186bc00a6fd300c5e2e76910a Mon Sep 17 00:00:00 2001 From: cristhianzl Date: Mon, 15 Jan 2024 22:31:46 -0300 Subject: [PATCH] fix(components): add data-testid to edit flow button for better testability and identification fix(dragAndDrop.spec): update locator for edit flow button to use data-testid for better stability fix(group.spec): update locator for edit flow button to use data-testid for better stability fix(saveComponents.spec): update locator for edit flow button to use data-testid for better stability --- .../src/pages/MainPage/components/components/index.tsx | 3 +++ src/frontend/tests/onlyFront/dragAndDrop.spec.ts | 4 +--- src/frontend/tests/onlyFront/group.spec.ts | 10 +++++----- src/frontend/tests/onlyFront/saveComponents.spec.ts | 5 +---- 4 files changed, 10 insertions(+), 12 deletions(-) diff --git a/src/frontend/src/pages/MainPage/components/components/index.tsx b/src/frontend/src/pages/MainPage/components/components/index.tsx index 6c48cc80e..130cb9eba 100644 --- a/src/frontend/src/pages/MainPage/components/components/index.tsx +++ b/src/frontend/src/pages/MainPage/components/components/index.tsx @@ -159,6 +159,9 @@ export default function ComponentsComponent({ variant="outline" size="sm" className="whitespace-nowrap " + data-testid={ + "edit-flow-button-" + item.id + "-" + idx + } > { ); await page - .locator( - '//*[@id="root"]/div/div[1]/div[2]/div[3]/div/div/div/div/div/div/div/div[2]/span[2]' - ) + .getByTestId("edit-flow-button-e9ac1bdc-429b-475d-ac03-d26f9a2a3210-0") .click(); await page.waitForTimeout(2000); diff --git a/src/frontend/tests/onlyFront/group.spec.ts b/src/frontend/tests/onlyFront/group.spec.ts index 8218e767b..f4cbe8458 100644 --- a/src/frontend/tests/onlyFront/group.spec.ts +++ b/src/frontend/tests/onlyFront/group.spec.ts @@ -33,7 +33,8 @@ test.describe("group node test", () => { return dt; }, jsonContent); - // Now dispatch + page.waitForTimeout(2000); + await page.dispatchEvent( '//*[@id="root"]/div/div[1]/div[2]/div[3]/div/div', "drop", @@ -43,9 +44,7 @@ test.describe("group node test", () => { ); await page - .locator( - '//*[@id="root"]/div/div[1]/div[2]/div[3]/div/div/div/div/div/div/div/div[2]/span[2]' - ) + .getByTestId("edit-flow-button-e9ac1bdc-429b-475d-ac03-d26f9a2a3210-0") .click(); await page.waitForTimeout(2000); @@ -67,7 +66,6 @@ test.describe("group node test", () => { }); await page.getByRole("button", { name: "Group" }).click(); - await page.locator("div").filter({ hasText: "Star13756" }).nth(3).click(); const textArea = page.getByTestId("div-textarea-2"); const elementCountText = await textArea.count(); @@ -80,5 +78,7 @@ test.describe("group node test", () => { if (elementGroup > 0) { expect(true).toBeTruthy(); } + + // Now dispatch }); }); diff --git a/src/frontend/tests/onlyFront/saveComponents.spec.ts b/src/frontend/tests/onlyFront/saveComponents.spec.ts index 07a074bad..a123a422e 100644 --- a/src/frontend/tests/onlyFront/saveComponents.spec.ts +++ b/src/frontend/tests/onlyFront/saveComponents.spec.ts @@ -51,9 +51,7 @@ test.describe("save component tests", () => { ); await page - .locator( - '//*[@id="root"]/div/div[1]/div[2]/div[3]/div/div/div/div/div/div/div/div[2]/span[2]' - ) + .getByTestId("edit-flow-button-e9ac1bdc-429b-475d-ac03-d26f9a2a3210-0") .click(); await page.waitForTimeout(2000); @@ -75,7 +73,6 @@ test.describe("save component tests", () => { }); await page.getByRole("button", { name: "Group" }).click(); - await page.locator("div").filter({ hasText: "Star13756" }).nth(3).click(); let textArea = page.getByTestId("div-textarea-2"); let elementCountText = await textArea.count();