Final version without page reload

Signed-off-by: mrrajan <86094767+mrrajan@users.noreply.github.com.>

Author: mrrajan

tests/ui/features/@sbom-explorer/sbom-explorer.feature

@@ -126,7 +126,6 @@ Feature: SBOM Explorer - View SBOM details
         | sbomName    |
         | quarkus-bom |
 
-    @slow
     Scenario Outline: Add Labels to SBOM from SBOM List Page
         Given An ingested SBOM "<sbomName>" is available
         When User Adds Labels "<Labels>" to "<sbomName>" SBOM from List Page
@@ -135,7 +134,6 @@ Feature: SBOM Explorer - View SBOM details
         | sbomName    |     Labels    |
         | quarkus-bom | RANDOM_LABELS |
 
-    @slow
     Scenario Outline: Add Labels to SBOM from SBOM Explorer Page
         Given An ingested SBOM "<sbomName>" is available
         When User visits SBOM details Page of "<sbomName>"

tests/ui/features/@sbom-explorer/sbom-explorer.step.ts

@@ -202,7 +202,6 @@ When(
     // Generate random labels if placeholder is used
     const labelsToAdd =
       labelList === "RANDOM_LABELS" ? detailsPage.generateLabels() : labelList;
-    console.log(`List page - Labels added ${labelList}`);
     await detailsPage.addLabels(labelsToAdd);
 
     // Store generated labels for verification
@@ -223,7 +222,6 @@ Then(
       labelList === "RANDOM_LABELS"
         ? (page as any).testContext?.generatedLabels || labelList
         : labelList;
-    console.log(`List page - Expected labels ${labelsToVerify}`);
     await detailsPage.verifyLabels(labelsToVerify, sbomName);
   }
 );
@@ -235,7 +233,6 @@ When(
     await detailsPage.editLabelsDetailsPage();
     const labelsToAdd =
       labelList === "RANDOM_LABELS" ? detailsPage.generateLabels() : labelList;
-    console.log(`Explorer page - Labels added ${labelList}`);
     await detailsPage.addLabels(labelsToAdd);
     (page as any).testContext = {
       ...(page as any).testContext,
@@ -256,7 +253,6 @@ Then(
       labelList === "RANDOM_LABELS"
         ? (page as any).testContext?.generatedLabels || labelList
         : labelList;
-    console.log(`Explorer page - Expected labels ${labelsToVerify}`);
     await detailsPage.verifyLabels(labelsToVerify, sbomName, infoSection);
   }
 );

tests/ui/helpers/DetailsPage.ts

@@ -186,7 +186,6 @@ export class DetailsPage {
    */
   async addLabels(labelList: string) {
     let labels = labelList.split(",").map((label) => label.trim());
-    console.log(`Labels added to the list: ${labelList}`);
     await this.page.getByText("Edit labels").isVisible();
     for (let label of labels) {
       await this.page.getByPlaceholder("Add label").fill(label);
@@ -227,31 +226,38 @@ export class DetailsPage {
     );
     await expect(labelContainer).toBeVisible({ timeout: 5000 });
     let max_attempts = 0;
-    while (max_attempts < 3) {
+    let labelFound = true;
+    let labelExpected = [...labels];
+    while (max_attempts < 3 && labelFound) {
       max_attempts++;
-      const labelElements = await parentElem
-        .locator(`xpath=//ul[@aria-label='Label group category']/li`)
-        .all();
-
-      for (const elem of labelElements) {
-        if (labels.length > 0) {
-          const labelText = await elem.innerText({ timeout: 2000 });
-          labels = labels.filter((label) => label !== labelText);
-          console.log(`Verification loop: Labels from UI - ${labelText}, LabelListRemaining - ${labels}`);
-        } else {
-          break;
+      for (const label of labels) {
+        try {
+          const labelUI = await parentElem.locator(
+            `xpath=//ul[@aria-label='Label group category']/li[.='${label}']`
+          );
+          await expect(labelUI).toBeVisible({ timeout: 2000 });
+          labelExpected = labelExpected.filter(
+            (labelTemp) => labelTemp !== label
+          );
+        } catch {
+          await this.page.waitForTimeout(500);
         }
       }
-      if (labels.length === 0) {
-        break;
-      } else {
-        console.log(`Reload`);
-        await this.page.reload({ waitUntil: 'networkidle' }); 
+      // for (const elem of labelElements) {
+      //   if (labels.length > 0) {
+      //     const labelText = await elem.innerText({ timeout: 2000 });
+      //     labels = labels.filter((label) => label !== labelText);
+      //   } else {
+      //     break;
+      //   }
+      // }
+      if (labelExpected.length === 0) {
+        labelFound = false;
       }
     }
     expect(
-      labels.length,
-      `Labels missing from the given list: ${labels.join(", ")}`
+      labelExpected.length,
+      `Labels missing from the given list: ${labelExpected.join(", ")}`
     ).toBe(0);
   }
 }