Changes after review

Signed-off-by: Maayan Hadasi <[email protected]>
konveyor · Dec 12, 2024 · 9673717 · 9673717
1 parent 5a52d66
commit 9673717
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/cypress/e2e/models/migration/applicationinventory/analysis.ts b/cypress/e2e/models/migration/applicationinventory/analysis.ts
@@ -364,13 +364,13 @@ export class Analysis extends Application {
         doesExistSelector(analyzeAppButton, rbacRules["Analyze"]);
     }
 
-    verifyAnalysisStatus(status: string) {
+    verifyAnalysisStatus(status: string, timeout?: number) {
         cy.log(`Verifying analysis status, expecting ${status}`);
         cy.get(tdTag, { log: false })
             .contains(this.name, { log: false })
             .closest(trTag, { log: false })
             .within(() => {
-                Analysis.verifyStatus(cy.get(analysisColumn, { log: false }), status);
+                Analysis.verifyStatus(cy.get(analysisColumn, { log: false }), status, timeout);
             });
         this.selectApplication();
     }
@@ -391,9 +391,9 @@ export class Analysis extends Application {
         });
     }
 
-    public static verifyStatus(element: Cypress.Chainable, status: string) {
+    public static verifyStatus(element: Cypress.Chainable, status: string, timeout = 10 * MIN) {
         element
-            .find("div > div:nth-child(2)", { timeout: 10 * MIN, log: false })
+            .find("div > div:nth-child(2)", { timeout: timeout, log: false })
             .should("not.have.text", AnalysisStatuses.notStarted)
             .and("not.have.text", AnalysisStatuses.scheduled)
             .and("not.have.text", AnalysisStatuses.inProgress)

diff --git a/cypress/e2e/tests/migration/applicationinventory/analysis/load_analysis.test.ts b/cypress/e2e/tests/migration/applicationinventory/analysis/load_analysis.test.ts
@@ -24,7 +24,7 @@ import {
     deleteApplicationTableRows,
 } from "../../../../../utils/utils";
 import { Analysis } from "../../../../models/migration/applicationinventory/analysis";
-import { AnalysisStatuses, SEC } from "../../../../types/constants";
+import { AnalysisStatuses, MIN, SEC } from "../../../../types/constants";
 import { Application } from "../../../../models/migration/applicationinventory/application";
 import { CustomMigrationTarget } from "../../../../models/administration/custom-migration-targets/custom-migration-target";
 import * as data from "../../../../../utils/data_utils";
@@ -93,7 +93,7 @@ describe(["@tier2"], "Source Analysis of big applications", () => {
         cy.wait("@getApplication");
         cy.wait(2 * SEC);
         application.analyze();
-        application.verifyAnalysisStatus(AnalysisStatuses.completed);
+        application.verifyAnalysisStatus(AnalysisStatuses.completed, 60 * MIN);
     });
 
     it("Source Analysis on OpenMRS app", function () {
@@ -108,7 +108,7 @@ describe(["@tier2"], "Source Analysis of big applications", () => {
         cy.wait("@getApplication");
         cy.wait(2 * SEC);
         application.analyze();
-        application.verifyAnalysisStatus(AnalysisStatuses.completed);
+        application.verifyAnalysisStatus(AnalysisStatuses.completed, 30 * MIN);
     });
 
     after("Test data clean up", function () {