Fix main branch precommit issues (#1433)

ansible · Nov 28, 2024 · d36f7f5 · d36f7f5
1 parent e129407
commit d36f7f5
Show file tree

Hide file tree

Showing 7 changed files with 29 additions and 14 deletions.
diff --git 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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -6,15 +6,19 @@ 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");
   });
 
   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");
   });