Enabled more tests (#2399)

cypress/integration/client_authorization_test.spec.ts

@@ -7,7 +7,7 @@ import SidebarPage from "../support/pages/admin_console/SidebarPage";
 import AuthorizationTab from "../support/pages/admin_console/manage/clients/AuthorizationTab";
 import ModalUtils from "../support/util/ModalUtils";
 
-describe.skip("Client authentication subtab", () => {
+describe("Client authentication subtab", () => {
   const loginPage = new LoginPage();
   const listingPage = new ListingPage();
   const masthead = new Masthead();
@@ -56,6 +56,7 @@ describe.skip("Client authentication subtab", () => {
       .save();
 
     masthead.checkNotificationMessage("Resource created successfully", true);
+    sidebarPage.waitForPageLoad();
     authenticationTab.cancel();
   });
 
@@ -96,6 +97,7 @@ describe.skip("Client authentication subtab", () => {
 
     cy.wait(["@get"]);
     masthead.checkNotificationMessage("Successfully created the policy", true);
+    sidebarPage.waitForPageLoad();
     authenticationTab.cancel();
   });
 
@@ -124,6 +126,7 @@ describe.skip("Client authentication subtab", () => {
 
     cy.wait(["@get"]);
     masthead.checkNotificationMessage("Successfully created the policy", true);
+    sidebarPage.waitForPageLoad();
     authenticationTab.cancel();
   });
 
@@ -146,17 +149,20 @@ describe.skip("Client authentication subtab", () => {
       true
     );
     cy.wait(["@load"]);
+    sidebarPage.waitForPageLoad();
     authenticationTab.cancel();
   });
 
-  it("Should copy auth details", () => {
+  it.skip("Should copy auth details", () => {
     authenticationTab.goToExportSubTab();
+    sidebarPage.waitForPageLoad();
     authenticationTab.copy();
     masthead.checkNotificationMessage("Authorization details copied.", true);
   });
 
   it("Should export auth details", () => {
     authenticationTab.goToExportSubTab();
+    sidebarPage.waitForPageLoad();
     authenticationTab.export();
 
     masthead.checkNotificationMessage(

src/clients/authorization/PermissionDetails.tsx

@@ -33,6 +33,7 @@ import { ResourcesPolicySelect } from "./ResourcesPolicySelect";
 import { toAuthorizationTab } from "../routes/AuthenticationTab";
 import { ScopeSelect } from "./ScopeSelect";
 import { toUpperCase } from "../../util";
+import { KeycloakSpinner } from "../../components/keycloak-spinner/KeycloakSpinner";
 
 const DECISION_STRATEGIES = ["UNANIMOUS", "AFFIRMATIVE", "CONSENSUS"] as const;
 
@@ -163,13 +164,17 @@ export default function PermissionDetails() {
     defaultValue: [],
   });
 
+  if (!permission) {
+    return <KeycloakSpinner />;
+  }
+
   return (
     <>
       <DeleteConfirm />
       <ViewHeader
         titleKey={
           permissionId
-            ? permission?.name!
+            ? permission.name!
             : `clients:create${toUpperCase(permissionType)}BasedPermission`
         }
         dropdownItems={