Improve code

appsmithorg · Jan 13, 2025 · 6fd0683 · 6fd0683
1 parent 97ebfaa
commit 6fd0683
Showing 1 changed file with 16 additions and 1 deletion.
diff --git a/app/client/cypress/e2e/Regression/ClientSide/AdminSettings/Admin_settings_spec.ts b/app/client/cypress/e2e/Regression/ClientSide/AdminSettings/Admin_settings_spec.ts
@@ -3,6 +3,8 @@ import {
   agHelper,
   adminSettings as adminSettingsHelper,
   homePage,
+  locators,
+  assertHelper,
 } from "../../../../support/Objects/ObjectsCore";
 
 const {
@@ -25,6 +27,9 @@ describe("Admin settings page", { tags: ["@tag.Settings"] }, function () {
   it("1. Should test that settings page is accessible to super user", () => {
     cy.LogOut();
     cy.LoginFromAPI(Cypress.env("USERNAME"), Cypress.env("PASSWORD"));
+    agHelper.AssertElementAbsence(locators._loading);
+    assertHelper.AssertDocumentReady();
+    agHelper.WaitUntilEleAppear(adminSettingsHelper._adminSettingsBtn);
     agHelper.GetNClick(adminSettingsHelper._adminSettingsBtn);
     agHelper.AssertURL(adminSettingsHelper.routes.GENERAL);
     cy.wait("@getEnvVariables");
@@ -33,6 +38,8 @@ describe("Admin settings page", { tags: ["@tag.Settings"] }, function () {
 
   it("2. Should test that settings page is not accessible to normal users", () => {
     cy.LoginFromAPI(Cypress.env("TESTUSERNAME3"), Cypress.env("TESTPASSWORD3"));
+    agHelper.AssertElementAbsence(locators._loading);
+    assertHelper.AssertDocumentReady();
     agHelper.AssertElementAbsence(adminSettingsHelper._adminSettingsBtn);
     agHelper.VisitNAssert(adminSettingsHelper.routes.GENERAL);
     // non super users are redirected to home page
@@ -54,7 +61,9 @@ describe("Admin settings page", { tags: ["@tag.Settings"] }, function () {
     () => {
       cy.visit("/applications", { timeout: 60000 });
       if (!Cypress.env("AIRGAPPED")) cy.wait("@getAllWorkspaces");
-
+      agHelper.AssertElementAbsence(locators._loading);
+      assertHelper.AssertDocumentReady();
+      agHelper.WaitUntilEleAppear(adminSettingsHelper._adminSettingsBtn);
       agHelper.GetNClick(adminSettingsHelper._adminSettingsBtn);
       cy.wait("@getEnvVariables");
       agHelper.GetNClick(adminsSettings.generalTab);
@@ -199,6 +208,9 @@ describe("Admin settings page", { tags: ["@tag.Settings"] }, function () {
   it("10. Verify default instance name", () => {
     cy.LogOut();
     cy.LoginFromAPI(Cypress.env("USERNAME"), Cypress.env("PASSWORD"));
+    agHelper.AssertElementAbsence(locators._loading);
+    assertHelper.AssertDocumentReady();
+    agHelper.WaitUntilEleAppear(adminSettingsHelper._adminSettingsBtn);
     agHelper.GetNClick(adminSettingsHelper._adminSettingsBtn);
     agHelper.AssertURL(adminSettingsHelper.routes.GENERAL);
     cy.wait("@getEnvVariables");
@@ -213,6 +225,9 @@ describe("Admin settings page", { tags: ["@tag.Settings"] }, function () {
     () => {
       homePage.LogOutviaAPI();
       cy.LoginFromAPI(Cypress.env("USERNAME"), Cypress.env("PASSWORD"));
+      agHelper.AssertElementAbsence(locators._loading);
+      assertHelper.AssertDocumentReady();
+      agHelper.WaitUntilEleAppear(adminSettingsHelper._adminSettingsBtn);
       agHelper.VisitNAssert("/applications", "getAllWorkspaces");
       agHelper.GetNClick(adminSettingsHelper._adminSettingsBtn);
       cy.wait("@getEnvVariables");