OCPBUGS-82511: re-enable helm-release.feature

Changes to fix flaky Helm release Cypress tests and re-enable the test suite:

Navigation and test structure improvements:
- Add navigation constants (navPaths) to DRY up cy.clickNavLink calls
- Replace navigateTo() with cy.clickNavLink() for Helm and Topology navigation
- Remove duplicate cy.clickNavLink() command definition in dev-console
- Move cy.clickNavLink() to shared integration-tests package
- Add app.waitForLoad() calls to ensure pages fully load before interactions
- Update helm-release.feature precondition to use admin perspective setup

Timing and race condition fixes:
- Wait for chart version dropdowns to be enabled before clicking
- Wait for button loading states to complete before proceeding
- Wait for catalog cards to be filtered and displayed after type selection
- Increase numTestsKeptInMemory from 5 to 50 to prevent memory cleanup mid-test
- Add verifyPageTitle() check after selecting catalog type
- Change detailsPage titleShouldContain to use 'exist' instead of 'be.visible'

Topology sidebar improvements:
- Fix topology sidebar flake by searching for helm group before clicking
- Update verifyActions to check exact action list length and order
- Fix malformed CSS selector in topology-po.ts (missing closing quote)

Test cleanup:
- Remove duplicate step definitions for "user will be redirected to Topology page"
- Remove duplicate step definition for "user is on the topology sidebar"
- Simplify hooks.ts to use cy.login() without explicit credentials
- Remove beforeEach hook that was causing state pollution
- Reorder test scenarios to match dependency flow

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: rhamilto

frontend/packages/dev-console/integration-tests/support/commands/app.ts

@@ -9,7 +9,6 @@ declare global {
   namespace Cypress {
     interface Chainable {
       alertTitleShouldContain(title: string): Chainable<Element>;
-      clickNavLink(path: string[]): Chainable<Element>;
       selectByDropDownText(selector: string, dropdownText: string): Chainable<Element>;
       selectByAutoCompleteDropDownText(selector: string, dropdownText: string): Chainable<Element>;
       verifyDropdownselected(selector: string): Chainable<Element>;
@@ -31,15 +30,6 @@ Cypress.Commands.add('alertTitleShouldContain', (alertTitle: string) => {
   cy.byLegacyTestID('modal-title').should('contain.text', alertTitle);
 });
 
-Cypress.Commands.add('clickNavLink', (path: string[]) => {
-  cy.get(`[data-test="nav"]`) // this assumes all top level menu items are expandable
-    .contains(path[0])
-    .click(); // open top, expandable menu
-  // eslint-disable-next-line cypress/no-unnecessary-waiting
-  cy.wait(500); // wait for animation
-  cy.get('#page-sidebar').contains(path[1]).click();
-});
-
 Cypress.Commands.add('selectByDropDownText', (selector: string, dropdownText: string) => {
   cy.get(selector).click();
   cy.get('li').contains(dropdownText).click({ force: true });

frontend/packages/dev-console/integration-tests/support/pages/add-flow/catalog-page.ts

@@ -82,6 +82,7 @@ export const catalogPage = {
         throw new Error('Card is not available in Catalog');
       }
     }
+    catalogPage.verifyPageTitle(type);
   },
   selectTemplateCategory: (templateCategoryTitle: string) => {
     const selector =
@@ -100,6 +101,7 @@ export const catalogPage = {
   selectHelmChartCard: (cardName: string) => cy.byTestID(`HelmChart-${cardName}`).first().click(),
   clickOnInstallButton: () => {
     cy.get(catalogPO.installHelmChart.create).click();
+    cy.get(catalogPO.installHelmChart.create).find('.pf-v6-c-button__progress').should('not.exist');
     cy.get('.co-m-loader', { timeout: 40000 }).should('not.exist');
   },
   enterReleaseName: (releaseName: string) =>

frontend/packages/helm-plugin/integration-tests/features/helm-release.feature

@@ -1,12 +1,12 @@
-# Disabled due to createRoot concurrent rendering failures (OCPBUGS-82511)
-@helm @smoke @manual
+@helm @smoke
 Feature: Helm Release
               As a user, I want to perform actions on the helm release
 
 
         @pre-condition
         Scenario: Create or Select the project namespace
-            Given user has created or selected namespace "aut-ci-helm"
+            Given user is at the Helm Release tab in admin perspective
+             And user has created or selected namespace "aut-ci-helm"
 
 
         Scenario: Open the Helm tab on the navigation bar when helm charts are absent: HR-05-TC01
@@ -17,8 +17,7 @@ Feature: Helm Release
 
 
         Scenario: Create Helm Release page details: HR-05-TC02
-            Given user has created or selected namespace "aut-ci-helm"
-              And user is at Software Catalog page
+            Given user is at Software Catalog page
              When user selects Helm Charts type from Software Catalog page
               And user searches and selects "Nodejs" card from catalog page
               And user clicks on the Create button on side bar
@@ -39,6 +38,7 @@ Feature: Helm Release
              Then user will be redirected to Topology page
               And Topology page have the helm chart workload "nodejs-release"
 
+
         Scenario: Helm release status verification: HR-01-TC04
             Given user is at the Helm Release tab in admin perspective
               And user is able to see "nodejs-release" in helm page in admin view
@@ -51,7 +51,7 @@ Feature: Helm Release
               And user is able to see the status and status icon of Revision history page
 
         Scenario: Context menu options of helm release:  HR-01-TC01
-            Given user is at the Topology page in admin view
+            Given user is at the Topology page
              When user right clicks on the helm release "nodejs-release" to open the context menu
              Then user is able to see the context menu with actions Upgrade and Delete Helm Release
 
@@ -78,21 +78,22 @@ Feature: Helm Release
               And user will see the Release Notes tab
               And user will see the Actions drop down menu with options Upgrade, Rollback, and Delete Helm Release
 
-        Scenario: Actions menu on Helm page after helm chart upgrade: HR-08-TC01
-            Given user is on the Helm page with helm release "nodejs-release"
-             When user clicks on the Kebab menu
-             Then user is able to see kebab menu with actions Upgrade, Rollback and Delete Helm Release
-
 
         Scenario: Perform Upgrade action on Helm Release through Context Menu: HR-08-TC04
-            Given user is at the Topology page in admin view
+            Given user is at the Topology page
              When user right clicks on the helm release "nodejs-release" to open the context menu
               And user clicks on the "Upgrade" action
               And user upgrades the chart Version
               And user clicks on the upgrade button
              Then user will be redirected to Topology page
 
 
+        Scenario: Actions menu on Helm page after helm chart upgrade: HR-08-TC01
+            Given user is on the Helm page with helm release "nodejs-release"
+             When user clicks on the Kebab menu
+             Then user is able to see kebab menu with actions Upgrade, Rollback and Delete Helm Release
+
+
         Scenario: Perform the helm chart upgrade for already upgraded helm chart : HR-08-TC02
             Given user is on the Helm page with helm release "nodejs-release"
              When user clicks on the Kebab menu
@@ -103,7 +104,7 @@ Feature: Helm Release
 
 
         Scenario: Perform Rollback action on Helm Release through Context Menu: HR-08-TC03
-            Given user is at the Topology page in admin view
+            Given user is at the Topology page
               And user is on the topology sidebar of the helm release "nodejs-release"
              When user clicks on the Actions drop down menu
               And user clicks on the "Rollback" action
@@ -113,7 +114,7 @@ Feature: Helm Release
 
 
         Scenario: Delete Helm Release through Context Menu: HR-01-TC03
-            Given user is at the Topology page in admin view
+            Given user is at the Topology page
              When user right clicks on the helm release "nodejs-release" to open the context menu
               And user clicks on the "Delete Helm Release" action
               And user enters the release name "nodejs-release"

frontend/packages/helm-plugin/integration-tests/support/commands/hooks.ts

@@ -1,9 +1,5 @@
 before(() => {
-  const bridgePasswordIDP: string = Cypress.expose('BRIDGE_HTPASSWD_IDP') || 'test';
-  const bridgePasswordUsername: string = Cypress.expose('BRIDGE_HTPASSWD_USERNAME') || 'test';
-  cy.env(['BRIDGE_HTPASSWD_PASSWORD']).then(({ BRIDGE_HTPASSWD_PASSWORD }) => {
-    cy.login(bridgePasswordIDP, bridgePasswordUsername, BRIDGE_HTPASSWD_PASSWORD || 'test');
-  });
+  cy.login();
   cy.document().its('readyState').should('eq', 'complete');
   cy.window().then((win: any) => {
     win.SERVER_FLAGS.userSettingsLocation = 'localstorage';
@@ -12,11 +8,6 @@ before(() => {
   cy.exec('oc apply -f test-data/red-hat-helm-charts.yaml');
 });
 
-beforeEach(() => {
-  cy.initAdmin();
-  cy.byLegacyTestID('topology-header').should('exist').click({ force: true });
-});
-
 after(() => {
   cy.exec(`oc delete namespace ${Cypress.expose('NAMESPACE')}`, { failOnNonZeroExit: false });
 });

frontend/packages/helm-plugin/integration-tests/support/constants/index.ts

@@ -1 +1,2 @@
+export * from './navigation';
 export * from './static-text/helm-text';

frontend/packages/helm-plugin/integration-tests/support/constants/navigation.ts

@@ -0,0 +1,9 @@
+/**
+ * Navigation paths for cy.clickNavLink()
+ * Format: [parent menu, submenu]
+ */
+export const navPaths = {
+  helm: ['Ecosystem', 'Helm'],
+  topology: ['Workloads', 'Topology'],
+  softwareCatalog: ['Ecosystem', 'Software Catalog'],
+};

frontend/packages/helm-plugin/integration-tests/support/pages/helm/upgrade-helm-release-page.ts

@@ -6,6 +6,8 @@ export const upgradeHelmRelease = {
   updateReplicaCount: (replicaCount: string = '2') =>
     cy.get(helmPO.upgradeHelmRelease.replicaCount).clear().type(replicaCount),
   upgradeChartVersion: () => {
+    // Wait for the dropdown to be enabled (starts disabled while loading chart versions)
+    cy.get(helmPO.upgradeHelmRelease.chartVersion).should('not.be.disabled');
     cy.get(helmPO.upgradeHelmRelease.chartVersion).click();
     const count = Cypress.$('[data-test="console-select"]').length;
     const randNum = Math.floor(Math.random() * count);
@@ -14,5 +16,6 @@ export const upgradeHelmRelease = {
   },
   clickOnUpgrade: () => {
     cy.get(helmPO.upgradeHelmRelease.upgrade).click();
+    cy.get(helmPO.upgradeHelmRelease.upgrade).find('.pf-v6-c-button__progress').should('not.exist');
   },
 };

frontend/packages/helm-plugin/integration-tests/support/step-definitions/common/common.ts

@@ -16,8 +16,10 @@ import {
   gitPage,
   catalogPage,
   addPage,
+  app,
 } from '@console/dev-console/integration-tests/support/pages';
 import { checkDeveloperPerspective } from '@console/dev-console/integration-tests/support/pages/functions/checkDeveloperPerspective';
+import { navPaths } from '../../constants';
 
 Given('user is at developer perspective', () => {
   checkDeveloperPerspective();
@@ -33,12 +35,8 @@ Given('user has created or selected namespace {string}', (projectName: string) =
 });
 
 Given('user is at the Topology page', () => {
-  navigateTo(devNavigationMenu.Topology);
-  topologyPage.verifyTopologyPage();
-});
-
-Given('user is at the Topology page in admin view', () => {
-  cy.byLegacyTestID('topology-header').should('exist').click({ force: true });
+  cy.clickNavLink(navPaths.topology);
+  app.waitForLoad();
   topologyPage.verifyTopologyPage();
 });
 
@@ -77,10 +75,6 @@ When('user clicks node {string} to open the side bar', (name: string) => {
   topologyPage.componentNode(name).click({ force: true });
 });
 
-When('user navigates to Topology page', () => {
-  navigateTo(devNavigationMenu.Topology);
-});
-
 Then('modal with {string} appears', (header: string) => {
   modal.modalTitleShouldContain(header);
 });
@@ -94,11 +88,14 @@ When('user selects {string} card from add page', (cardName: string) => {
 });
 
 Given('user is at Software Catalog page', () => {
-  cy.byLegacyTestID('developer-catalog-header').should('exist').click({ force: true });
+  cy.clickNavLink(navPaths.softwareCatalog);
+  catalogPage.verifyTitle();
 });
 
 When('user selects Helm Charts type from Software Catalog page', () => {
   catalogPage.selectCatalogType(catalogTypes.HelmCharts);
+  // Wait for catalog cards to be filtered and displayed
+  catalogPage.isCardsDisplayed();
 });
 
 When('user switches to the {string} tab', (tab: string) => {

frontend/packages/helm-plugin/integration-tests/support/step-definitions/helm/helm-installation-view.ts

@@ -17,6 +17,8 @@ When('user clicks on the Create button on side bar', () => {
 });
 
 When('user clicks on the chart versions dropdown menu', () => {
+  // Wait for the dropdown to be enabled (starts disabled while loading chart versions)
+  cy.get(helmPO.upgradeHelmRelease.chartVersion).should('not.be.disabled');
   cy.get(helmPO.upgradeHelmRelease.chartVersion).click();
 });
 

frontend/packages/helm-plugin/integration-tests/support/step-definitions/helm/helm-navigation.ts

@@ -19,6 +19,7 @@ import {
 } from '@console/dev-console/integration-tests/support/pages';
 import { checkDeveloperPerspective } from '@console/dev-console/integration-tests/support/pages/functions/checkDeveloperPerspective';
 import { topologyPage } from '@console/topology/integration-tests/support/pages/topology/topology-page';
+import { navPaths } from '../../constants';
 import { helmPage, helmDetailsPage } from '../../pages';
 
 const deleteChartRepositoryFromDetailsPage = (name: string, type: string) => {
@@ -35,7 +36,7 @@ Given('user is at developer perspective', () => {
 
 When('user clicks on the Helm tab in dev perspective', () => {
   cy.get('[data-quickstart-id="qs-admin-nav-helm"]').should('be.visible').click({ force: true });
-  navigateTo(devNavigationMenu.Helm);
+  cy.clickNavLink(navPaths.helm);
 });
 
 Then('user will be redirected to Helm releases page', () => {
@@ -89,16 +90,17 @@ Then('Topology page have the helm chart workload {string}', (nodeName: string) =
 });
 
 Given('user has installed helm chart', () => {
-  navigateTo(devNavigationMenu.Topology);
+  cy.clickNavLink(navPaths.topology);
+  topologyPage.verifyTopologyPage();
   topologyPage.verifyWorkloadInTopologyPage('nodejs-release');
 });
 
 Given('user is at the Helm page', () => {
-  navigateTo(devNavigationMenu.Helm);
+  cy.clickNavLink(navPaths.helm);
 });
 
 Given('user is at the Helm Release tab in admin perspective', () => {
-  cy.clickNavLink(['Ecosystem', 'Helm']);
+  cy.clickNavLink(navPaths.helm);
   cy.byLegacyTestID('horizontal-link-Helm Releases').should('exist').click({ force: true });
 });
 
@@ -315,7 +317,7 @@ When('user selects cluster-scoped scope type', () => {
 });
 
 When('user navigates to Helm page', () => {
-  navigateTo(devNavigationMenu.Helm);
+  cy.clickNavLink(navPaths.helm);
 });
 
 When('user can see {string} {string} details page', (type: string, name: string) => {
@@ -331,7 +333,7 @@ Given(
 );
 
 Given('user is able to see {string} in helm page', (helmRelease: string) => {
-  navigateTo(devNavigationMenu.Helm);
+  cy.clickNavLink(navPaths.helm);
   helmPage.search(helmRelease);
 });