]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/dashboard: Fix failing config dashboard e2e check 43210/head
authorNizamudeen A <nia@redhat.com>
Fri, 17 Sep 2021 16:07:09 +0000 (21:37 +0530)
committerNizamudeen A <nia@redhat.com>
Mon, 20 Sep 2021 10:01:44 +0000 (15:31 +0530)
Recently a new osd config has been added in
6ca32bde2e1d0dd58df168126582a570ac09aad6 and this is getting Modified.
So on our dashboard e2e config check which checks for the Modified
filter, this is also coming on the entry. So we need to increase the
count.

Fixes: https://tracker.ceph.com/issues/52649
Signed-off-by: Nizamudeen A <nia@redhat.com>
src/pybind/mgr/dashboard/frontend/cypress/integration/cluster/configuration.e2e-spec.ts

index f9bc8d43f5d453bf0322478ec0a7aa9907023f2b..d022d59cfa9a2ea85e8f1396bd92eaca42c54292 100644 (file)
@@ -31,6 +31,7 @@ describe('Configuration page', () => {
 
     beforeEach(() => {
       configuration.clearTableSearchInput();
+      configuration.getTableCount('found').as('configFound');
     });
 
     after(() => {
@@ -49,14 +50,29 @@ describe('Configuration page', () => {
       );
     });
 
-    it('should show only modified configurations', () => {
+    it('should verify modified filter is applied properly', () => {
+      configuration.filterTable('Modified', 'no');
+      configuration.getTableCount('found').as('unmodifiedConfigs');
+
+      // Modified filter value to yes
       configuration.filterTable('Modified', 'yes');
-      configuration.getTableCount('found').should('eq', 2);
-    });
+      configuration.getTableCount('found').as('modifiedConfigs');
+
+      cy.get('@configFound').then((configFound) => {
+        cy.get('@unmodifiedConfigs').then((unmodifiedConfigs) => {
+          const modifiedConfigs = Number(configFound) - Number(unmodifiedConfigs);
+          configuration.getTableCount('found').should('eq', modifiedConfigs);
+        });
+      });
 
-    it('should hide all modified configurations', () => {
+      // Modified filter value to no
       configuration.filterTable('Modified', 'no');
-      configuration.getTableCount('found').should('gt', 1);
+      cy.get('@configFound').then((configFound) => {
+        cy.get('@modifiedConfigs').then((modifiedConfigs) => {
+          const unmodifiedConfigs = Number(configFound) - Number(modifiedConfigs);
+          configuration.getTableCount('found').should('eq', unmodifiedConfigs);
+        });
+      });
     });
   });
 });