mgr/dashboard: fix cypress tests

Signed-off-by: Aashish Sharma <[email protected]>

Author: aaSharma14

src/pybind/mgr/dashboard/frontend/cypress/e2e/block/mirroring.e2e-spec.ts

@@ -61,8 +61,8 @@ describe('Mirroring page', () => {
           cy.get('[type=submit]').click();
 
           cy.get('[data-testid="pool-name"]').clear().type(name);
-          cy.get(`select[name=poolType]`).select('replicated');
-          cy.get(`select[name=poolType] option:checked`).contains('replicated');
+          cy.get('[data-testid="pool-type-select"]').select('replicated');
+          cy.get('[data-testid="pool-type-select"] option:checked').contains('replicated');
           cy.get('.float-start.me-2.select-menu-edit').click();
           cy.get('.popover-body').should('be.visible');
           // Choose rbd as the application label

src/pybind/mgr/dashboard/frontend/cypress/e2e/multi-cluster/multi-cluster.e2e-spec.ts

@@ -1,11 +1,7 @@
-import { DashboardPageHelper } from '../ui/dashboard.po';
 import { MultiClusterPageHelper } from './multi-cluster.po';
 
 describe('Muti-cluster management page', () => {
   const multiCluster = new MultiClusterPageHelper();
-  const dashboard = new DashboardPageHelper();
-
-  const hubName = 'local-cluster';
   const url = Cypress.env('CEPH2_URL');
   const alias = 'ceph2';
   const username = 'admin';
@@ -24,35 +20,17 @@ describe('Muti-cluster management page', () => {
     multiCluster.checkConnectionStatus(alias, 'CONNECTED');
   });
 
-  it('should switch to the second cluster and back to hub', () => {
-    multiCluster.checkConnectionStatus(alias, 'CONNECTED');
-    dashboard.navigateTo();
-    cy.get('[data-testid="selected-cluster"]').click();
-    cy.get('[data-testid="select-a-cluster"]').contains(alias).click();
-    cy.get('[data-testid="selected-cluster"]').contains(alias);
-    cy.get('cd-dashboard-v3').should('exist');
-
-    // now switch back to the hub cluster
-    cy.get('[data-testid="selected-cluster"]').click();
-    cy.get('[data-testid="select-a-cluster"]').contains(hubName).click();
-    cy.get('[data-testid="selected-cluster"]').contains(hubName);
-    cy.get('cd-dashboard-v3').should('exist');
-  });
-
   it('should reconnect the second cluster', () => {
-    multiCluster.checkConnectionStatus(alias, 'CONNECTED');
     multiCluster.reconnect(alias, password);
     multiCluster.existTableCell(alias);
   });
 
   it('should edit the second cluster', () => {
-    multiCluster.checkConnectionStatus(alias, 'CONNECTED');
     multiCluster.edit(alias, editedAlias);
     multiCluster.existTableCell(editedAlias);
   });
 
   it('should disconnect the second cluster', () => {
-    multiCluster.checkConnectionStatus(editedAlias, 'CONNECTED');
     multiCluster.disconnect(editedAlias);
     multiCluster.existTableCell(editedAlias, false);
   });

src/pybind/mgr/dashboard/frontend/cypress/e2e/pools/pools.po.ts

@@ -33,7 +33,7 @@ export class PoolPageHelper extends PageHelper {
 
   edit_pool_pg(name: string, new_pg: number, wait = true, mirroring = false) {
     this.isPowerOf2(new_pg);
-    this.navigateEdit(name);
+    this.navigateEdit(name, true, false);
 
     if (mirroring) {
       cy.get('[data-testid="rbd-mirroring-check"]').should('be.checked');

src/pybind/mgr/dashboard/frontend/cypress/e2e/rgw/buckets.po.ts

@@ -35,7 +35,6 @@ export class BucketsPageHelper extends PageHelper {
     // Select bucket owner
     this.selectOwner(owner);
     cy.get('#owner').should('have.class', 'ng-valid');
-    cy.get('input[name=encryption_enabled]').should('be.disabled');
 
     if (isLocking) {
       cy.get('#lock_enabled_input').click({ force: true });

src/pybind/mgr/dashboard/frontend/cypress/e2e/rgw/users.po.ts

@@ -159,8 +159,8 @@ export class UsersPageHelper extends PageHelper {
   checkUserKeys(user_name: string) {
     this.getExpandCollapseElement(user_name).should('be.visible').click();
     cy.get('cd-table').contains('td', user_name).click();
-    cy.get('cd-rgw-user-details cd-table').eq(0).first().click();
-    cy.get("[aria-label='Show']").should('exist').click();
+    cy.get('cd-rgw-user-details cd-table [cdstablerow]').first().click();
+    cy.get("[aria-label='Show']").should('exist').click({ force: true });
     cy.get('input#user').should('exist');
     cy.get('input#access_key').should('exist');
     cy.get('input#secret_key').should('exist');

src/pybind/mgr/dashboard/frontend/cypress/e2e/ui/notification.e2e-spec.ts

@@ -10,6 +10,7 @@ describe('Notification page', () => {
     cy.login();
     pools.navigateTo('create');
     pools.create(poolName, 8, ['rbd']);
+    cy.wait(5000);
     pools.edit_pool_pg(poolName, 4, false);
   });
 

src/pybind/mgr/dashboard/frontend/src/app/ceph/cephfs/cephfs-subvolume-list/cephfs-subvolume-list.component.html

@@ -73,12 +73,12 @@
        ngbTooltip="{{row.name}} is removed after retaining the snapshots"></i>
   </ng-template>
 
-  <cd-label [value]="row.info.type"
-            *ngIf="row.info.type !== 'subvolume'"></cd-label>
+  <cd-label [value]="row?.info?.type"
+            *ngIf="row?.info?.type !== 'subvolume'"></cd-label>
 
   <cd-label value="namespaced"
-            *ngIf="row.info.pool_namespace"
-            [tooltipText]="row.info.pool_namespace"></cd-label>
+            *ngIf="row?.info?.pool_namespace"
+            [tooltipText]="row?.info?.pool_namespace"></cd-label>
 </ng-template>
 
 <ng-template #removeTmpl

src/pybind/mgr/dashboard/frontend/src/app/ceph/pool/pool-form/pool-form.component.html

@@ -46,6 +46,7 @@
                  i18n>Pool type</label>
           <div class="cd-col-form-input">
             <select class="form-select"
+                    data-testid="pool-type-select"
                     id="poolType"
                     formControlName="poolType">
               <option ngValue=""

src/pybind/mgr/dashboard/frontend/src/app/ceph/rgw/rgw-multisite-wizard/rgw-multisite-wizard.component.html

@@ -404,12 +404,12 @@
   <div class="form-group row">
     <label class="cd-col-form-label"
            i18n>Realm Name:</label>
-    <div class="cd-col-form-input mt-2 text-muted"><b>{{ multisiteSetupForm.get('realmName').value }}</b></div>
+    <div class="cd-col-form-input mt-2 text-muted"><b id="realmName">{{ multisiteSetupForm.get('realmName').value }}</b></div>
   </div>
   <div class="form-group row">
     <label class="cd-col-form-label"
            i18n>Zonegroup Name:</label>
-    <div class="cd-col-form-input mt-2 text-muted"><b>{{ multisiteSetupForm.get('zonegroupName').value }}</b></div>
+    <div class="cd-col-form-input mt-2 text-muted"><b id="zonegroupName">{{ multisiteSetupForm.get('zonegroupName').value }}</b></div>
   </div>
   <div class="form-group row">
     <label class="cd-col-form-label"
@@ -419,7 +419,7 @@
   <div class="form-group row">
     <label class="cd-col-form-label"
            i18n>Zone Name:</label>
-    <div class="cd-col-form-input mt-2 text-muted"><b>{{ multisiteSetupForm.get('zoneName').value }}</b></div>
+    <div class="cd-col-form-input mt-2 text-muted"><b id="zoneName">{{ multisiteSetupForm.get('zoneName').value }}</b></div>
   </div>
   <div class="form-group row">
     <label class="cd-col-form-label"