Refactored alerting test, and added wait logic to reduce flakiness. (#…

…953) (#957) Signed-off-by: AWSHurneyt <[email protected]> (cherry picked from commit 5db8065) Co-authored-by: AWSHurneyt <[email protected]>
opensearch-project · Nov 24, 2023 · 515faa4 · 515faa4
1 parent e7647a1
commit 515faa4
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/cypress/integration/plugins/alerting-dashboards-plugin/monitors_dashboard_spec.js b/cypress/integration/plugins/alerting-dashboards-plugin/monitors_dashboard_spec.js
@@ -38,7 +38,7 @@ const clusterHealthMonitor = {
         severity: '1',
         condition: {
           script: {
-            source: 'ctx.results[0].status != "green"',
+            source: 'ctx.results[0].status != "blue"',
             lang: 'painless',
           },
         },
@@ -107,8 +107,13 @@ describe('Monitors dashboard page', () => {
   });
 
   it('Displays expected number of alerts', () => {
+    // Wait for table to finish loading
+    cy.get('tbody > tr').should(($tr) =>
+      expect($tr).to.have.length.greaterThan(1)
+    );
+
     // Ensure the 'Monitor name' column is sorted in ascending order by sorting another column first
-    cy.contains('Last updated by').click({ force: true });
+    cy.contains('Last notification time').click({ force: true });
     cy.contains('Monitor name').click({ force: true });
 
     testMonitors.forEach((entry) => {