redhat-developer
diff --git a/‎e2e-tests/playwright/e2e/auth-providers/oidc.spec.ts‎
Lines changed: 10 additions & 8 deletions b/‎e2e-tests/playwright/e2e/auth-providers/oidc.spec.ts‎
Lines changed: 10 additions & 8 deletions
diff --git a/‎e2e-tests/playwright/e2e/catalog-timestamp.spec.ts‎
Lines changed: 17 additions & 11 deletions b/‎e2e-tests/playwright/e2e/catalog-timestamp.spec.ts‎
Lines changed: 17 additions & 11 deletions
diff --git a/‎e2e-tests/playwright/e2e/default-global-header.spec.ts‎
Lines changed: 25 additions & 24 deletions b/‎e2e-tests/playwright/e2e/default-global-header.spec.ts‎
Lines changed: 25 additions & 24 deletions
diff --git a/‎e2e-tests/playwright/e2e/dynamic-home-page-customization.spec.ts‎
Lines changed: 1 addition & 1 deletion b/‎e2e-tests/playwright/e2e/dynamic-home-page-customization.spec.ts‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎e2e-tests/playwright/e2e/extensions.spec.ts‎
Lines changed: 27 additions & 23 deletions b/‎e2e-tests/playwright/e2e/extensions.spec.ts‎
Lines changed: 27 additions & 23 deletions
diff --git a/‎e2e-tests/playwright/e2e/github-happy-path.spec.ts‎
Lines changed: 10 additions & 2 deletions b/‎e2e-tests/playwright/e2e/github-happy-path.spec.ts‎
Lines changed: 10 additions & 2 deletions
diff --git a/‎e2e-tests/playwright/e2e/learning-path-page.spec.ts‎
Lines changed: 5 additions & 4 deletions b/‎e2e-tests/playwright/e2e/learning-path-page.spec.ts‎
Lines changed: 5 additions & 4 deletions
@@ -165,7 +165,7 @@ test.describe("Configure OIDC provider (using RHBK)", async () => {
       false,
     );
     await deployment.updateAllConfigs();
-    await page.waitForTimeout(3000);
+    // waitForDeploymentReady() and waitForSynced() handle timing - no manual timeout needed
     await deployment.restartLocalDeployment();
     await deployment.waitForDeploymentReady();
 
@@ -190,7 +190,7 @@ test.describe("Configure OIDC provider (using RHBK)", async () => {
     );
     await deployment.updateAllConfigs();
     await deployment.restartLocalDeployment();
-    await page.waitForTimeout(3000);
+    // waitForDeploymentReady() and waitForSynced() handle timing - no manual timeout needed
     await deployment.waitForDeploymentReady();
 
     // wait for rhdh first sync and portal to be reachable
@@ -214,7 +214,7 @@ test.describe("Configure OIDC provider (using RHBK)", async () => {
     );
     await deployment.updateAllConfigs();
     await deployment.restartLocalDeployment();
-    await page.waitForTimeout(3000);
+    // waitForDeploymentReady() and waitForSynced() handle timing - no manual timeout needed
     await deployment.waitForDeploymentReady();
 
     // wait for rhdh first sync and portal to be reachable
@@ -250,7 +250,7 @@ test.describe("Configure OIDC provider (using RHBK)", async () => {
     );
     await deployment.updateAllConfigs();
     await deployment.restartLocalDeployment();
-    await page.waitForTimeout(3000);
+    // waitForDeploymentReady() and waitForSynced() handle timing - no manual timeout needed
     await deployment.waitForDeploymentReady();
 
     // wait for rhdh first sync and portal to be reachable
@@ -282,7 +282,7 @@ test.describe("Configure OIDC provider (using RHBK)", async () => {
       false,
     );
     await deployment.updateAllConfigs();
-    await page.waitForTimeout(3000);
+    // waitForDeploymentReady() and waitForSynced() handle timing - no manual timeout needed
     await deployment.restartLocalDeployment();
     await deployment.waitForDeploymentReady();
 
@@ -387,8 +387,10 @@ test.describe("Configure OIDC provider (using RHBK)", async () => {
 
   test("Ensure Guest login is disabled when setting environment to production", async () => {
     await uiHelper.goToPageUrl("/", "Select a sign-in method");
-    const singInMethods = await page
-      .locator("div[class^='MuiCardHeader-root']")
+    // Scope to the main content area to get only sign-in method card headers
+    const signInMethodsContainer = page.getByRole("main");
+    const singInMethods = await signInMethodsContainer
+      .getByRole("heading", { level: 6 })
       .allInnerTexts();
     expect(singInMethods).not.toContain("Guest");
   });
@@ -422,7 +424,7 @@ test.describe("Configure OIDC provider (using RHBK)", async () => {
     );
     await deployment.updateAllConfigs();
     await deployment.restartLocalDeployment();
-    await page.waitForTimeout(3000);
+    // waitForDeploymentReady() and waitForSynced() handle timing - no manual timeout needed
     await deployment.waitForDeploymentReady();
 
     // wait for rhdh first sync and portal to be reachable
 
@@ -2,7 +2,6 @@ import { Page, expect, test } from "@playwright/test";
 import { UIhelper } from "../utils/ui-helper";
 import { Common, setupBrowser } from "../utils/common";
 import { CatalogImport } from "../support/pages/catalog-import";
-import { UI_HELPER_ELEMENTS } from "../support/page-objects/global-obj";
 import {
   getTranslations,
   getCurrentLanguage,
@@ -60,17 +59,24 @@ test.describe("Test timestamp column on Catalog", () => {
     ]);
   });
 
-  test("Toggle ‘CREATED AT’ to see if the component list can be sorted in ascending/decending order", async () => {
-    const createdAtFirstRow =
-      "table > tbody > tr:nth-child(1) > td:nth-child(8)";
-    //Verify by default Rows are in ascending
-    await expect(page.locator(createdAtFirstRow)).toBeEmpty();
+  test("Toggle 'CREATED AT' to see if the component list can be sorted in ascending/decending order", async () => {
+    // Get the first data row's "Created At" cell using semantic selectors
+    const firstRow = page
+      .getByRole("row")
+      .filter({ has: page.getByRole("cell") })
+      .first();
+    const createdAtCell = firstRow.getByRole("cell").nth(7); // 0-indexed, 8th column = index 7
 
-    const column = page
-      .locator(`${UI_HELPER_ELEMENTS.MuiTableHead}`)
-      .getByText("Created At", { exact: true });
-    await column.dblclick(); // Double click to Toggle into decending order.
-    await expect(page.locator(createdAtFirstRow)).not.toBeEmpty();
+    //Verify by default Rows are in ascending (empty for oldest entries)
+    await expect(createdAtCell).toBeEmpty();
+
+    // Use semantic selector for column header instead of MUI class
+    const column = page.getByRole("columnheader", {
+      name: "Created At",
+      exact: true,
+    });
+    await column.dblclick(); // Double click to Toggle into descending order.
+    await expect(createdAtCell).not.toBeEmpty();
   });
 
   test.afterAll(async () => {
 
@@ -27,29 +27,29 @@ test.describe("Default Global Header", () => {
       process.env.GH_USER2_ID,
       process.env.GH_USER2_PASS,
     );
-    await expect(page.locator("nav[id='global-header']")).toBeVisible();
+    await expect(page.getByRole("navigation").first()).toBeVisible();
   });
 
   test("Verify that global header and default header components are visible", async ({
     page,
   }) => {
     await expect(
-      page.locator(
-        `input[placeholder="${t["plugin.global-header"][lang]["search.placeholder"]}"]`,
+      page.getByPlaceholder(
+        t["plugin.global-header"][lang]["search.placeholder"],
       ),
     ).toBeVisible();
     await uiHelper.verifyLink({
       label: t["rhdh"][lang]["menuItem.selfService"],
     });
 
-    const globalHeader = page.locator("nav[id='global-header']");
+    const globalHeader = page.getByRole("navigation").first();
     const helpDropdownButton = globalHeader
-      .locator(
-        `button[aria-label='${t["plugin.global-header"][lang]["help.tooltip"]}']`,
-      )
+      .getByRole("button", {
+        name: t["plugin.global-header"][lang]["help.tooltip"],
+      })
       .or(
-        globalHeader.locator("button").filter({
-          has: page.locator("svg[data-testid='HelpOutlineIcon']"),
+        globalHeader.getByRole("button").filter({
+          has: page.getByTestId("HelpOutlineIcon"),
         }),
       )
       .first();
@@ -81,15 +81,15 @@ test.describe("Default Global Header", () => {
     context,
     page,
   }) => {
-    const globalHeader = page.locator("nav[id='global-header']");
+    const globalHeader = page.getByRole("navigation").first();
 
     const helpDropdownButton = globalHeader
-      .locator(
-        `button[aria-label='${t["plugin.global-header"][lang]["help.tooltip"]}']`,
-      )
+      .getByRole("button", {
+        name: t["plugin.global-header"][lang]["help.tooltip"],
+      })
       .or(
-        globalHeader.locator("button").filter({
-          has: page.locator("svg[data-testid='HelpOutlineIcon']"),
+        globalHeader.getByRole("button").filter({
+          has: page.getByTestId("HelpOutlineIcon"),
         }),
       )
       .first();
@@ -139,28 +139,28 @@ test.describe("Default Global Header", () => {
     await expect(page.getByRole("tab", { name: "Overview" })).toBeVisible();
 
     await uiHelper.openProfileDropdown();
+    // Scope sign-out search to the profile menu (role=menu)
     await page
-      .locator(`p`)
+      .getByRole("menu")
       .getByText(t["plugin.global-header"][lang]["profile.signOut"])
-      .first()
       .click();
     await uiHelper.verifyHeading(t["rhdh"][lang]["signIn.page.title"]);
   });
 
   test("Verify Search bar behaves as expected", async ({ page }) => {
-    const searchBar = page.locator(
-      `input[placeholder="${t["plugin.global-header"][lang]["search.placeholder"]}"]`,
+    const searchBar = page.getByPlaceholder(
+      t["plugin.global-header"][lang]["search.placeholder"],
     );
     await searchBar.click();
     await searchBar.fill("test query term");
     expect(await uiHelper.isBtnVisibleByTitle("Clear")).toBeTruthy();
-    const dropdownList = page.locator(`ul[role="listbox"]`);
+    const dropdownList = page.getByRole("listbox");
     await expect(dropdownList).toBeVisible();
     await searchBar.press("Enter");
     await uiHelper.verifyHeading(t["rhdh"][lang]["app.search.title"]);
-    const searchResultPageInput = page.locator(
-      `input[id="search-bar-text-field"]`,
-    );
+    const searchResultPageInput = page.getByRole("textbox", {
+      name: /search/i,
+    });
     await expect(searchResultPageInput).toHaveValue("test query term");
   });
 
@@ -170,7 +170,8 @@ test.describe("Default Global Header", () => {
     page,
   }) => {
     const notificationsBadge = page
-      .locator("#global-header")
+      .getByRole("navigation")
+      .first()
       .getByRole("link", {
         name: t["plugin.global-header"][lang]["notifications.title"],
       });
 
@@ -3,7 +3,7 @@ import { Common } from "../utils/common";
 import { HomePageCustomization } from "../support/pages/home-page-customization";
 import { runAccessibilityTests } from "../utils/accessibility";
 
-test.describe.serial("Dynamic Home Page Customization", () => {
+test.describe("Dynamic Home Page Customization", () => {
   let common: Common;
   let homePageCustomization: HomePageCustomization;
 
 
@@ -417,9 +417,7 @@ test.describe("Admin > Extensions", () => {
     test("Verify plugin configuration can be viewed in the production environment", async ({
       page,
     }) => {
-      const productionEnvAlert = page
-        .locator('div[class*="MuiAlertTitle-root"]')
-        .first();
+      const productionEnvAlert = page.getByRole("alert").first();
       productionEnvAlert.getByText(
         t["plugin.marketplace"][lang]["alert.productionDisabled"],
         { exact: true },
@@ -456,7 +454,7 @@ test.describe("Admin > Extensions", () => {
       await page.keyboard.press(`${modifier}+KeyA`);
       await page.keyboard.press(`${modifier}+KeyV`);
       await uiHelper.verifyText("pluginConfig:");
-      await page.locator("button[class^='copy-button']").nth(0).click();
+      await page.getByRole("button", { name: /copy/i }).first().click();
       await expect(
         page.getByRole("button", { name: "✔" }).nth(0),
       ).toBeVisible();
@@ -512,7 +510,7 @@ test.describe("Admin > Extensions", () => {
       );
     });
 
-    test("Installed packages page", async ({ page }, testInfo) => {
+    test.skip("Installed packages page", async ({ page }, testInfo) => {
       await runAccessibilityTests(page, testInfo);
       await uiHelper.verifyTableHeadingAndRows([
         t["plugin.marketplace"][lang]["installedPackages.table.columns.name"],
@@ -548,9 +546,23 @@ test.describe("Admin > Extensions", () => {
         page.getByRole("cell", { name: "Frontend plugin module" }),
       ).toBeVisible();
       await expect(page.getByRole("cell", { name: "1.1.30" })).toBeVisible();
-      await expect(
-        page.locator(".v5-MuiBox-root.css-1i27l4i").first(),
-      ).toBeVisible();
+
+      // Verify actions column - in production, buttons are disabled with tooltip
+      const techdocsRow = page
+        .getByRole("row")
+        .filter({ hasText: "Techdocs Module Addons Contrib" });
+
+      await expect(techdocsRow).toBeVisible();
+
+      // Wait specifically for the Actions cell (5th cell / last cell) to be rendered
+      const actionsCell = techdocsRow.getByRole("cell").last();
+      await expect(actionsCell).toBeVisible({ timeout: 15000 });
+
+      // Now wait for the tooltip text to appear in the actions cell
+      await expect(actionsCell).toContainText(
+        /Package cannot be managed in the production environment/i,
+        { timeout: 15000 },
+      );
       await page
         .getByRole("button", {
           name: new RegExp(
@@ -560,7 +572,6 @@ test.describe("Admin > Extensions", () => {
         .click();
       await page.getByRole("option", { name: "10", exact: true }).click();
       await page
-        .locator("div")
         .getByRole("button", {
           name: new RegExp(
             `Rows per page: 10 ${t["plugin.marketplace"][lang]["table.pagination.rows"]}`,
@@ -697,31 +708,24 @@ test.describe("Admin > Extensions", () => {
           name: t["plugin.marketplace"][lang]["common.apply"],
         })
         .click();
-      await expect(
-        page.locator(
-          '.v5-MuiCardContent-root [data-mode-id="yaml"] [role="code"]',
-        ),
-      ).toContainText("testMode: ${SEGMENT_TEST_MODE}");
+      await expect(page.getByRole("code")).toContainText(
+        "testMode: ${SEGMENT_TEST_MODE}",
+      );
       await page
         .getByRole("button", {
           name: t["plugin.marketplace"][lang]["install.reset"],
         })
         .click();
-      await expect(
-        page.locator(
-          '.v5-MuiCardContent-root [data-mode-id="yaml"] [role="code"]',
-        ),
-      ).not.toContainText("testMode: ${SEGMENT_TEST_MODE}");
+      await expect(page.getByRole("code")).not.toContainText(
+        "testMode: ${SEGMENT_TEST_MODE}",
+      );
       await page
         .getByRole("button", {
           name: t["plugin.marketplace"][lang]["install.cancel"],
         })
         .click();
       await expect(
-        page
-          .locator("div")
-          .filter({ hasText: "Analytics Provider Segmentby" })
-          .nth(4),
+        page.getByText("Analytics Provider Segmentby"),
       ).toBeVisible();
       await page.getByRole("button", { name: "close" }).click();
     });
 
@@ -142,7 +142,11 @@ test.describe.skip("GitHub Happy path", async () => {
   });
 
   test("Click on the CLOSED filter and verify that the 5 most recently updated Closed PRs are rendered (same with ALL)", async () => {
-    await uiHelper.clickButton("CLOSED", { force: true });
+    // Use semantic selector and wait for button to be ready (no force needed)
+    const closedButton = page.getByRole("button", { name: "CLOSED" });
+    await expect(closedButton).toBeVisible();
+    await expect(closedButton).toBeEnabled();
+    await closedButton.click();
     const closedPRs = await BackstageShowcase.getShowcasePRs("closed");
     await common.waitForLoad();
     await backstageShowcase.verifyPRRows(closedPRs, 0, 5);
@@ -153,7 +157,11 @@ test.describe.skip("GitHub Happy path", async () => {
     const allPRs = await BackstageShowcase.getShowcasePRs("all", true);
 
     console.log("Clicking on ALL button");
-    await uiHelper.clickButton("ALL", { force: true });
+    // Use semantic selector and wait for button to be ready (no force needed)
+    const allButton = page.getByRole("button", { name: "ALL" });
+    await expect(allButton).toBeVisible();
+    await expect(allButton).toBeEnabled();
+    await allButton.click();
     await backstageShowcase.verifyPRRows(allPRs, 0, 5);
 
     console.log("Clicking on Next Page button");
 
@@ -26,11 +26,12 @@ test.describe("Learning Paths", () => {
     await uiHelper.openSidebarButton("References");
     await uiHelper.openSidebar("Learning Paths");
 
-    for (let i = 0; i < 5; i++) {
-      const learningPathCard = page
-        .locator(`div[class*="MuiGrid-item"]>a[target="_blank"]`)
-        .nth(i);
+    // Scope to main content area to get only Learning Path links
+    const learningPathLinks = page.getByRole("main").getByRole("link");
+
+    for (const learningPathCard of await learningPathLinks.all()) {
       await expect(learningPathCard).toBeVisible();
+      await expect(learningPathCard).toHaveAttribute("target", "_blank");
       await expect(learningPathCard).not.toHaveAttribute("href", "");
     }