From d36f7f522e69a7d62942c6b7754a64d9b940743c Mon Sep 17 00:00:00 2001
From: Kirill Gaevskii <kgaevski@redhat.com>
Date: Thu, 28 Nov 2024 13:01:20 +0100
Subject: [PATCH] Fix main branch precommit issues (#1433)

---
 .../__mocks__/monaco-editor.ts                           | 8 +++++---
 ansible_ai_connect_admin_portal/__mocks__/monaco-yaml.ts | 4 ++--
 ansible_ai_connect_admin_portal/src/AppHeader.tsx        | 2 +-
 .../src/PageMastheadDropdown.tsx                         | 9 +++++++--
 .../src/__tests__/App.test.tsx                           | 4 +++-
 .../src/__tests__/AppHeader.test.tsx                     | 8 +++++---
 .../src/denied/__tests__/AppDenied.test.tsx              | 8 ++++++--
 7 files changed, 29 insertions(+), 14 deletions(-)

diff --git a/ansible_ai_connect_admin_portal/__mocks__/monaco-editor.ts b/ansible_ai_connect_admin_portal/__mocks__/monaco-editor.ts
index 65b2d8b08..342864012 100644
--- a/ansible_ai_connect_admin_portal/__mocks__/monaco-editor.ts
+++ b/ansible_ai_connect_admin_portal/__mocks__/monaco-editor.ts
@@ -2,14 +2,16 @@ const editor = {
   create: () => {
     return {
       dispose: () => {},
-    }
+    };
   },
-  defineTheme: function() {}
+  defineTheme: function () {},
 };
 
 const monaco = {
   editor,
-  languages: {json: {jsonDefaults: { setDiagnosticsOptions: function () {}}}}
+  languages: {
+    json: { jsonDefaults: { setDiagnosticsOptions: function () {} } },
+  },
 };
 
 module.exports = monaco;
diff --git a/ansible_ai_connect_admin_portal/__mocks__/monaco-yaml.ts b/ansible_ai_connect_admin_portal/__mocks__/monaco-yaml.ts
index 56d80c821..9a7ef14f3 100644
--- a/ansible_ai_connect_admin_portal/__mocks__/monaco-yaml.ts
+++ b/ansible_ai_connect_admin_portal/__mocks__/monaco-yaml.ts
@@ -1,5 +1,5 @@
 const monacoYaml = {
-    configureMonacoYaml: function () {}
-  };
+  configureMonacoYaml: function () {},
+};
 
 module.exports = monacoYaml;
diff --git a/ansible_ai_connect_admin_portal/src/AppHeader.tsx b/ansible_ai_connect_admin_portal/src/AppHeader.tsx
index 9019c852b..7e457997e 100644
--- a/ansible_ai_connect_admin_portal/src/AppHeader.tsx
+++ b/ansible_ai_connect_admin_portal/src/AppHeader.tsx
@@ -35,7 +35,7 @@ export function AppHeader(props: AppHeaderProps) {
         <ToolbarItem>
           <PageMastheadDropdown
             id="account-menu"
-            icon={<UserCircleIcon/>}
+            icon={<UserCircleIcon />}
             userName={userName}
           >
             <DropdownItem
diff --git a/ansible_ai_connect_admin_portal/src/PageMastheadDropdown.tsx b/ansible_ai_connect_admin_portal/src/PageMastheadDropdown.tsx
index 275a444a7..e91994a69 100644
--- a/ansible_ai_connect_admin_portal/src/PageMastheadDropdown.tsx
+++ b/ansible_ai_connect_admin_portal/src/PageMastheadDropdown.tsx
@@ -7,7 +7,7 @@ import {
   DropdownList,
   DropdownItem,
 } from "@patternfly/react-core";
-import React from 'react';
+import React from "react";
 import { ReactNode, useCallback, useState } from "react";
 import { useBreakpoint } from "@ansible/ansible-ui-framework";
 
@@ -32,7 +32,12 @@ export function PageMastheadDropdown(props: PageMastheadDropdownProps) {
       id={id}
       onSelect={onSelect}
       toggle={(toggleRef: React.Ref<MenuToggleElement>) => (
-        <MenuToggle ref={toggleRef} onClick={onToggle} isExpanded={isOpen} data-testid="page-masthead-dropdown__button">
+        <MenuToggle
+          ref={toggleRef}
+          onClick={onToggle}
+          isExpanded={isOpen}
+          data-testid="page-masthead-dropdown__button"
+        >
           <Flex
             alignItems={{ default: "alignItemsCenter" }}
             flexWrap={{ default: "nowrap" }}
diff --git a/ansible_ai_connect_admin_portal/src/__tests__/App.test.tsx b/ansible_ai_connect_admin_portal/src/__tests__/App.test.tsx
index afd7a4075..fa58179ea 100644
--- a/ansible_ai_connect_admin_portal/src/__tests__/App.test.tsx
+++ b/ansible_ai_connect_admin_portal/src/__tests__/App.test.tsx
@@ -16,7 +16,9 @@ describe("App", () => {
         adminDashboardUrl={"http://admin_dashboard-url/"}
       />,
     );
-    const accountMenu = await screen.findByTestId("page-masthead-dropdown__button");
+    const accountMenu = await screen.findByTestId(
+      "page-masthead-dropdown__button",
+    );
     expect(accountMenu).toBeInTheDocument();
     expect(accountMenu).toHaveTextContent("Batman");
   });
diff --git a/ansible_ai_connect_admin_portal/src/__tests__/AppHeader.test.tsx b/ansible_ai_connect_admin_portal/src/__tests__/AppHeader.test.tsx
index a92882386..ac3fd067d 100644
--- a/ansible_ai_connect_admin_portal/src/__tests__/AppHeader.test.tsx
+++ b/ansible_ai_connect_admin_portal/src/__tests__/AppHeader.test.tsx
@@ -1,7 +1,7 @@
 import { render, screen } from "@testing-library/react";
 import "@testing-library/jest-dom";
 import { AppHeader } from "../AppHeader";
-import { BrowserRouter } from 'react-router-dom';
+import { BrowserRouter } from "react-router-dom";
 
 describe("AppHeader", () => {
   // Store the original 'location' object so that it can be restored for other tests.
@@ -23,9 +23,11 @@ describe("AppHeader", () => {
     render(
       <BrowserRouter>
         <AppHeader userName={"Batman"} />
-      </BrowserRouter>
+      </BrowserRouter>,
+    );
+    const accountMenu = await screen.findByTestId(
+      "page-masthead-dropdown__button",
     );
-    const accountMenu = await screen.findByTestId("page-masthead-dropdown__button");
     expect(accountMenu).toBeInTheDocument();
     expect(accountMenu).toHaveTextContent("Batman");
 
diff --git a/ansible_ai_connect_admin_portal/src/denied/__tests__/AppDenied.test.tsx b/ansible_ai_connect_admin_portal/src/denied/__tests__/AppDenied.test.tsx
index c36f2debb..34e1d9c5e 100644
--- a/ansible_ai_connect_admin_portal/src/denied/__tests__/AppDenied.test.tsx
+++ b/ansible_ai_connect_admin_portal/src/denied/__tests__/AppDenied.test.tsx
@@ -6,7 +6,9 @@ describe("App", () => {
   it("Rendering::With Username", async () => {
     window.history.pushState({}, "Test page", "/console");
     render(<AppDenied userName={"Batman"} hasSubscription={true} />);
-    const accountMenu = await screen.findByTestId("page-masthead-dropdown__button");
+    const accountMenu = await screen.findByTestId(
+      "page-masthead-dropdown__button",
+    );
     expect(accountMenu).toBeInTheDocument();
     expect(accountMenu).toHaveTextContent("Batman");
   });
@@ -14,7 +16,9 @@ describe("App", () => {
   it("Rendering::Without Username", async () => {
     window.history.pushState({}, "Test page", "/console");
     render(<AppDenied hasSubscription={false} />);
-    const accountMenu = await screen.findByTestId("page-masthead-dropdown__button");
+    const accountMenu = await screen.findByTestId(
+      "page-masthead-dropdown__button",
+    );
     expect(accountMenu).toBeInTheDocument();
     expect(accountMenu).toHaveTextContent("UnknownUser");
   });