[test-fix-2]

JewelleryManagement · Apr 14, 2024 · f706cc4 · f706cc4
1 parent 19ace7f
commit f706cc4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 13 deletions.
diff --git a/tests/Sales/create-sale.spec.js b/tests/Sales/create-sale.spec.js
@@ -39,7 +39,7 @@ test("Create sale", async ({ page }) => {
   await selectResource(page);
 
   await submitButton.click();
-  expect(page.getByText("Successfully sold the product!")).toBeVisible();
+  await expect(page.getByText("Successfully sold the product!")).toBeVisible();
 });
 
 test("Create sale with a discount", async ({ page }) => {
@@ -53,13 +53,12 @@ test("Create sale with a discount", async ({ page }) => {
   await selectDate(page, expect);
 
   await selectProduct(page);
-  await page.locator("#input-156").fill("1");
-
+  await page.getByLabel("Discount").first().fill("1");
   await selectResource(page);
-  await page.locator("#input-183").fill("1");
+  await page.getByLabel("Discount").nth(0).fill("1");
 
   await submitButton.click();
-  expect(page.getByText("Successfully sold the product!")).toBeVisible();
+  await expect(page.getByText("Successfully sold the product!")).toBeVisible();
 });
 
 test("Create sale - with empty inputs - fails", async ({ page }) => {

diff --git a/tests/utils/salesUtils.js b/tests/utils/salesUtils.js
@@ -62,14 +62,10 @@ export const selectResource = async (page) => {
 };
 
 export const checkErrorMessages = async (page, expect) => {
-  const errorMessages = [
-    { selector: "#input-37-messages", message: "Please select at least one" },
-    { selector: "#input-40-messages", message: "Please select at least one" },
-  ];
-
-  for (const { selector, message } of errorMessages) {
-    await expect(page.locator(selector).getByText(message)).toBeVisible();
-  }
+
+  await expect(page.getByText('Please select at least one author').first()).toBeVisible()
+  await expect(page.getByText('Please select at least one author').nth(1)).toBeVisible()
+
 };
 
 export const checkDisabledButtons = async (