fix: adjust e2e tests

.github/workflows/pull-integration-cluster-k3d.yml

@@ -16,12 +16,6 @@ jobs:
     runs-on: ubuntu-latest
     if: github.event.pull_request.draft == false
     steps:
-      - name: Install Chrome # this step could be removed after https://github.com/cypress-io/cypress/issues/30374 is resolved
-        shell: bash
-        run: |
-          wget --no-verbose -O /tmp/chrome.deb https://dl.google.com/linux/chrome/deb/pool/main/g/google-chrome-stable/google-chrome-stable_127.0.6533.119-1_amd64.deb 
-          sudo apt install --allow-downgrades -y /tmp/chrome.deb 
-          rm /tmp/chrome.deb
       - uses: gardenlinux/workflow-telemetry-action@v2
         with:
           comment_on_pr: false

.github/workflows/pull-integration-namespace-k3d.yml

@@ -16,12 +16,6 @@ jobs:
     runs-on: ubuntu-latest
     if: github.event.pull_request.draft == false
     steps:
-      - name: Install Chrome
-        shell: bash
-        run: |
-          wget --no-verbose -O /tmp/chrome.deb https://dl.google.com/linux/chrome/deb/pool/main/g/google-chrome-stable/google-chrome-stable_127.0.6533.119-1_amd64.deb 
-          sudo apt install --allow-downgrades -y /tmp/chrome.deb 
-          rm /tmp/chrome.deb
       - uses: gardenlinux/workflow-telemetry-action@v2
         with:
           comment_on_pr: false

src/command-pallette/CommandPaletteProvider.tsx

@@ -16,7 +16,7 @@ export const CommandPaletteProvider = ({
   >();
 
   const setShowDialog = (value: boolean) => {
-    const modalPresent = document.querySelector('ui5-dialog[open="true"]');
+    const modalPresent = document.querySelector('ui5-dialog[open]');
     // disable opening palette if other modal is present
     if (!modalPresent || !value) {
       _setShowDialog(value);

src/header/NamespaceChooser/NamespaceChooser.tsx

@@ -42,7 +42,7 @@ export function NamespaceChooser() {
     />,
   ];
 
-  allNamespaces.map(ns =>
+  allNamespaces?.map(ns =>
     namespaces.push(
       <SideNavigationSubItem
         text={ns}

src/hooks/useAvailableNamespaces.ts

@@ -14,9 +14,9 @@ export function useAvailableNamespaces() {
   const { data: allNamespaces, error, refetch, silentRefetch } = useGetList()(
     '/api/v1/namespaces',
     {
-      skip: false,
       pollingInterval: 3000,
       onDataReceived: () => {},
+      skip: false,
     },
   ) as {
     loading: boolean;
@@ -37,7 +37,10 @@ export function useAvailableNamespaces() {
         if (showHiddenNamespaces) return true;
         return !hiddenNamespaces.includes(n);
       });
-    if (filteredNamespaces) {
+    if (
+      filteredNamespaces &&
+      JSON.stringify(filteredNamespaces) !== JSON.stringify(namespaces)
+    ) {
       setNamespaces(filteredNamespaces);
     }
   }, [
@@ -46,6 +49,7 @@ export function useAvailableNamespaces() {
     hiddenNamespaces,
     setNamespaces,
     showHiddenNamespaces,
+    namespaces,
   ]);
 
   return { namespaces, refetch, silentRefetch, setNamespaces };

src/shared/components/Labels/Labels.js

@@ -14,6 +14,7 @@ export const Labels = ({
   style = null,
   displayLabelForLabels = false,
   disableMarginBottom = false,
+  hideIcon = true,
 }) => {
   const { t } = useTranslation();
   if (!labels || Object.keys(labels).length === 0) {
@@ -54,6 +55,7 @@ export const Labels = ({
             className={`sap-margin-end-tiny ${
               disableMarginBottom ? '' : 'sap-margin-bottom-tiny'
             }`}
+            hideStateIcon={hideIcon}
           >
             {shortenLongLabels && label.length > SHORTENING_TRESHOLD
               ? shortenLabel(label)

src/shared/hooks/BackendAPI/useGet.js

@@ -14,7 +14,7 @@ const ERROR_TOLERANCY = 2;
 const useGetHook = processDataFn =>
   function(
     path,
-    { pollingInterval, onDataReceived, skip, errorTolerancy = undefined } = {},
+    { pollingInterval, onDataReceived, skip, errorTolerancy } = {},
   ) {
     const authData = useRecoilValue(authDataState);
     const lastAuthData = useRef(null);

src/state/useAfterInitHook.ts

@@ -48,7 +48,7 @@ export function useAfterInitHook(handledKubeconfigId: KubeconfigIdHandleState) {
     }
 
     // cluster not yet loaded
-    if (!cluster === undefined) {
+    if (cluster === undefined) {
       return;
     }
 

tests/integration/support/commands.js

@@ -43,14 +43,14 @@ Cypress.Commands.add(
 Cypress.Commands.add('checkItemOnGenericListLink', resourceName => {
   cy.get('ui5-table-row')
     .find('ui5-table-cell')
-    .contains('span', resourceName)
+    .contains('ui5-text', resourceName)
     .should('be.visible');
 });
 
 Cypress.Commands.add('clickGenericListLink', resourceName => {
   cy.get('ui5-table-row')
     .find('ui5-table-cell')
-    .contains('span', resourceName)
+    .contains('ui5-text', resourceName)
     .click();
 });
 
@@ -61,6 +61,7 @@ Cypress.Commands.add('filterWithNoValue', { prevSubject: true }, $elements =>
 Cypress.Commands.add('goToNamespaceDetails', () => {
   // Go to the details of namespace
   cy.getLeftNav()
+    .find('ui5-side-navigation-item')
     .contains('Namespaces')
     .click();
 
@@ -177,8 +178,8 @@ Cypress.Commands.add(
     cy.wait(1000);
 
     if (selectSearchResult) {
-      cy.get('ui5-li-suggestion-item:visible')
-        .contains(resourceName)
+      cy.get('ui5-suggestion-item:visible')
+        .contains('li', resourceName)
         .click();
     }
 
@@ -233,7 +234,7 @@ Cypress.Commands.add(
 
     cy.get('ui5-table-row')
       .find('ui5-table-cell')
-      .contains('span', resourceName)
+      .contains('ui5-text', resourceName)
       .should('not.be.visible');
 
     cy.getMidColumn()
@@ -255,7 +256,7 @@ Cypress.Commands.add(
 
     cy.get('ui5-table-row')
       .find('ui5-table-cell')
-      .contains('span', resourceName)
+      .contains('ui5-text', resourceName)
       .should('not.be.visible');
 
     cy.getEndColumn()

tests/integration/support/inspect-list.js

@@ -6,5 +6,5 @@ Cypress.Commands.add('inspectList', (resourceName, hiddenButtons = false) => {
     .wait(1000)
     .type(`${resourceName}{enter}`);
 
-  cy.contains('span', resourceName).should('be.visible');
+  cy.contains('ui5-text', resourceName).should('be.visible');
 });

tests/integration/tests/accessibility/test-acc-custom-resoureces.spec.js

@@ -15,7 +15,7 @@ context('Accessibility test Custom Resources', () => {
 
     cy.navigateTo('Configuration', 'Custom Resource Definitions');
 
-    cy.openCreate().click();
+    cy.openCreate();
 
     cy.wrap(loadFile(FILE_NAME)).then(CRD_CONFIG => {
       const CRD = JSON.stringify(CRD_CONFIG);

tests/integration/tests/accessibility/test-acc-performance.spec.js

@@ -13,7 +13,7 @@ context('Accessibility test Cluster list and overview', () => {
   it('Acc test with performance panel open', () => {
     cy.get('[title="Profile"]').click();
 
-    cy.get('ui5-menu-li:visible')
+    cy.get('ui5-menu-item:visible')
       .contains('Preferences')
       .click({ force: true });
 

tests/integration/tests/cluster/test-cluster-configuration.spec.js

@@ -65,7 +65,7 @@ context('Test Cluster configuration', () => {
 
     cy.get('[title="Profile"]').click();
 
-    cy.get('ui5-menu-li:visible')
+    cy.get('ui5-menu-item:visible')
       .contains('Preferences')
       .click({ force: true });
 
@@ -77,7 +77,7 @@ context('Test Cluster configuration', () => {
       .contains('20')
       .click();
 
-    cy.get('ui5-list:visible')
+    cy.get('ui5-option:visible')
       .contains('10')
       .click();
 

tests/integration/tests/cluster/test-cluster-overview.spec.js

@@ -52,7 +52,7 @@ context('Test Cluster Overview', () => {
       .get('.radial-chart')
       .contains('text.progress-label', '50%')
       .get('.radial-chart')
-      .contains('span.additional-info', 'test1233456');
+      .contains('ui5-text.additional-info', 'test1233456');
 
     // test injected statistical card exists and works
     cy.contains(
@@ -62,7 +62,8 @@ context('Test Cluster Overview', () => {
       .find('ui5-link.counting-card__link')
       .click();
 
-    cy.get('ui5-title')
+    cy.get('ui5-dynamic-page-title')
+      .find('ui5-title')
       .contains('Hpatest')
       .should('be.visible');
 
@@ -93,13 +94,17 @@ context('Test Cluster Overview', () => {
 
     cy.loginAndSelectCluster();
 
-    cy.get('[name="feedback"]').should('exist');
+    cy.get('ui5-shellbar')
+      .find('[name="feedback"]')
+      .should('exist');
 
     cy.setBusolaFeature('FEEDBACK', false);
 
     cy.loginAndSelectCluster();
 
-    cy.get('[name="feedback"]').should('not.exist');
+    cy.get('ui5-shellbar')
+      .find('[name="feedback"]')
+      .should('not.exist');
   });
 
   it('Go to Node details', () => {

tests/integration/tests/cluster/test-cluster-role-bindings.spec.js

@@ -52,11 +52,11 @@ context('Test Cluster Role Bindings', () => {
     cy.clickGenericListLink(CRB_NAME);
 
     cy.getMidColumn()
-      .contains('User')
+      .contains('ui5-table-cell', 'User')
       .should('be.visible');
 
     cy.getMidColumn()
-      .contains(USER_NAME)
+      .contains('ui5-table-cell', USER_NAME)
       .should('be.visible');
 
     cy.getMidColumn()
@@ -71,7 +71,7 @@ context('Test Cluster Role Bindings', () => {
 
     cy.contains('[role="combobox"]', 'User').click();
 
-    cy.get('ui5-li:visible')
+    cy.get('ui5-option:visible')
       .contains('ServiceAccount')
       .find('li')
       .click({ force: true });
@@ -96,6 +96,7 @@ context('Test Cluster Role Bindings', () => {
     cy.inspectTab('View');
 
     cy.getMidColumn()
+      .find('ui5-table-cell')
       .contains('Group')
       .should('be.visible');
 

tests/integration/tests/cluster/test-command-palette.spec.js

@@ -149,9 +149,8 @@ context('Test Command Palette navigation', () => {
     getQueryInput().type('pref');
 
     // autocomplete
-    cy.get('body')
-      .tab()
-      .type('{enter}', { force: true });
+    cy.get('body').tab();
+    getQueryInput().type('{enter}', { force: true });
 
     cy.contains('Cluster interaction').should('be.visible');
 

tests/integration/tests/cluster/test-custom-resources.spec.js

@@ -19,7 +19,7 @@ context('Test Custom Resources', () => {
 
     cy.navigateTo('Configuration', 'Custom Resource Definitions');
 
-    cy.openCreate().click();
+    cy.openCreate();
 
     cy.wrap(loadFile(FILE_NAME)).then(CRD_CONFIG => {
       const CRD = JSON.stringify(CRD_CONFIG);
@@ -92,6 +92,7 @@ context('Test Custom Resources', () => {
   });
 
   it('Test column layout and remove CRD', () => {
+    cy.reload();
     cy.getLeftNav()
       .contains('Custom Resources')
       .click();
@@ -114,15 +115,15 @@ context('Test Custom Resources', () => {
     cy.getMidColumn()
       .get('ui5-table-row')
       .find('ui5-table-cell')
-      .contains('span', 'tcluster-test')
+      .contains('ui5-text', 'tcluster-test')
       .click();
 
     cy.testEndColumnLayout('tcluster-test', false);
 
     cy.getMidColumn()
       .get('ui5-table-row')
       .find('ui5-table-cell')
-      .contains('span', 'tcluster-test')
+      .contains('ui5-text', 'tcluster-test')
       .click();
 
     cy.getMidColumn()

tests/integration/tests/cluster/test-dashboard-version.spec.js

@@ -10,13 +10,12 @@ context('Test Kyma Dashboard Version link', () => {
   it('Check Kyma Dashboard Version link', () => {
     cy.get('[title="Profile"]').click();
 
-    cy.get('.ui5-menu-rp')
-      .find('ui5-menu-li:visible')
+    cy.get('ui5-menu')
+      .find('ui5-menu-item:visible')
       .contains('Legal Information')
       .click({ force: true });
 
-    cy.get('ui5-responsive-popover[placement-type="Right"]')
-      .find('ui5-menu-li:visible')
+    cy.get('li:visible')
       .contains('Kyma Dashboard version:')
       .should('be.visible');
   });

tests/integration/tests/cluster/test-multiple-context-kubeconfig.spec.js

@@ -64,21 +64,27 @@ context('Test multiple context kubeconfig', () => {
         .contains(kubeconfig['current-context'])
         .click();
 
-      cy.get('ui5-li:visible').contains(kubeconfig.contexts[1].name);
+      cy.get('ui5-option:visible').contains(kubeconfig.contexts[1].name);
 
-      cy.get('ui5-li:visible')
+      cy.wait(3000);
+
+      cy.get('ui5-option:visible')
         .contains('All contexts')
         .click();
+      cy.wait(3000);
 
       cy.contains('Next').click({ force: true });
+      cy.wait(3000);
 
       cy.get('ui5-button:visible')
         .contains('Next step')
         .click();
+      cy.wait(3000);
 
       cy.get(`[accessible-name="last-step"]:visible`)
         .contains('Connect cluster')
         .click({ force: true });
+      cy.wait(3000);
 
       cy.contains('Cluster Details').should('exist');
 

tests/integration/tests/cluster/test-navigation-features.spec.js

@@ -36,12 +36,17 @@ context('Test navigation features', () => {
       .wait(1000)
       .type('cronjob-controller');
 
-    cy.get('ui5-li-suggestion-item:visible')
-      .contains('cronjob-controller')
+    cy.get('ui5-suggestion-item:visible')
+      .contains('li', /cronjob-controller/)
       .click();
 
-    cy.contains('cronjob-controller (SA)') // link wrapper
-      .contains('cronjob-controller') // link itself
+    cy.wait(1000);
+
+    cy.contains('ui5-text', 'system:controller:cronjob-controller').click();
+
+    cy.getMidColumn()
+      .contains('ui5-Panel', 'Subjects')
+      .contains('ui5-link', 'cronjob-controller')
       .click();
 
     cy.contains('disabled').should('exist');