From 9b63f369bf3a2fb044b7de270f30dc414ccca23b Mon Sep 17 00:00:00 2001
From: Derek Ho <dxho@amazon.com>
Date: Wed, 17 Jul 2024 12:05:01 -0400
Subject: [PATCH] Update tests

Signed-off-by: Derek Ho <dxho@amazon.com>
---
 .../service-account-list.test.tsx.snap        | 37 --------
 .../__snapshots__/app-router.test.tsx.snap    | 90 -------------------
 .../test/internal-user-list-utils.test.tsx    | 40 ++-------
 3 files changed, 6 insertions(+), 161 deletions(-)
 delete mode 100644 public/apps/configuration/panels/test/__snapshots__/service-account-list.test.tsx.snap

diff --git a/public/apps/configuration/panels/test/__snapshots__/service-account-list.test.tsx.snap b/public/apps/configuration/panels/test/__snapshots__/service-account-list.test.tsx.snap
deleted file mode 100644
index 0617b7a8b..000000000
--- a/public/apps/configuration/panels/test/__snapshots__/service-account-list.test.tsx.snap
+++ /dev/null
@@ -1,37 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`Service Account list AccessError component should load access error component 1`] = `
-<Fragment>
-  <Memo()
-    config={Object {}}
-    coreStart={
-      Object {
-        "http": 1,
-      }
-    }
-    dataSourcePickerReadOnly={true}
-    navigation={Object {}}
-    params={Object {}}
-    selectedDataSource={
-      Object {
-        "id": "",
-        "label": "Local cluster",
-      }
-    }
-    setDataSource={[Function]}
-  />
-  <EuiPageHeader>
-    <EuiTitle
-      size="l"
-    >
-      <h1>
-        Service accounts
-      </h1>
-    </EuiTitle>
-  </EuiPageHeader>
-  <AccessErrorComponent
-    dataSourceLabel="Local cluster"
-    loading={false}
-  />
-</Fragment>
-`;
diff --git a/public/apps/configuration/test/__snapshots__/app-router.test.tsx.snap b/public/apps/configuration/test/__snapshots__/app-router.test.tsx.snap
index 0c19e737f..21aa4e515 100644
--- a/public/apps/configuration/test/__snapshots__/app-router.test.tsx.snap
+++ b/public/apps/configuration/test/__snapshots__/app-router.test.tsx.snap
@@ -39,10 +39,6 @@ exports[`SecurityPluginTopNavMenu renders DataSourceMenu when dataSource is enab
                   "href": "/users",
                   "name": "Internal users",
                 },
-                Object {
-                  "href": "/serviceAccounts",
-                  "name": "Service Accounts",
-                },
                 Object {
                   "href": "/permissions",
                   "name": "Permissions",
@@ -85,10 +81,6 @@ exports[`SecurityPluginTopNavMenu renders DataSourceMenu when dataSource is enab
                   "href": "/users",
                   "name": "Internal users",
                 },
-                Object {
-                  "href": "/serviceAccounts",
-                  "name": "Service Accounts",
-                },
                 Object {
                   "href": "/permissions",
                   "name": "Permissions",
@@ -131,10 +123,6 @@ exports[`SecurityPluginTopNavMenu renders DataSourceMenu when dataSource is enab
                   "href": "/users",
                   "name": "Internal users",
                 },
-                Object {
-                  "href": "/serviceAccounts",
-                  "name": "Service Accounts",
-                },
                 Object {
                   "href": "/permissions",
                   "name": "Permissions",
@@ -177,56 +165,6 @@ exports[`SecurityPluginTopNavMenu renders DataSourceMenu when dataSource is enab
                   "href": "/users",
                   "name": "Internal users",
                 },
-                Object {
-                  "href": "/serviceAccounts",
-                  "name": "Service Accounts",
-                },
-                Object {
-                  "href": "/permissions",
-                  "name": "Permissions",
-                },
-                Object {
-                  "href": "/tenants",
-                  "name": "Tenants",
-                },
-                Object {
-                  "href": "/auditLogging",
-                  "name": "Audit logs",
-                },
-              ]
-            }
-          />
-        </EuiPageSideBar>
-      </Route>
-      <Route
-        exact={true}
-        key="/serviceAccounts"
-        path="/serviceAccounts"
-      >
-        <EuiPageSideBar>
-          <NavPanel
-            items={
-              Array [
-                Object {
-                  "href": "/getstarted",
-                  "name": "Get Started",
-                },
-                Object {
-                  "href": "/auth",
-                  "name": "Authentication",
-                },
-                Object {
-                  "href": "/roles",
-                  "name": "Roles",
-                },
-                Object {
-                  "href": "/users",
-                  "name": "Internal users",
-                },
-                Object {
-                  "href": "/serviceAccounts",
-                  "name": "Service Accounts",
-                },
                 Object {
                   "href": "/permissions",
                   "name": "Permissions",
@@ -269,10 +207,6 @@ exports[`SecurityPluginTopNavMenu renders DataSourceMenu when dataSource is enab
                   "href": "/users",
                   "name": "Internal users",
                 },
-                Object {
-                  "href": "/serviceAccounts",
-                  "name": "Service Accounts",
-                },
                 Object {
                   "href": "/permissions",
                   "name": "Permissions",
@@ -315,10 +249,6 @@ exports[`SecurityPluginTopNavMenu renders DataSourceMenu when dataSource is enab
                   "href": "/users",
                   "name": "Internal users",
                 },
-                Object {
-                  "href": "/serviceAccounts",
-                  "name": "Service Accounts",
-                },
                 Object {
                   "href": "/permissions",
                   "name": "Permissions",
@@ -361,10 +291,6 @@ exports[`SecurityPluginTopNavMenu renders DataSourceMenu when dataSource is enab
                   "href": "/users",
                   "name": "Internal users",
                 },
-                Object {
-                  "href": "/serviceAccounts",
-                  "name": "Service Accounts",
-                },
                 Object {
                   "href": "/permissions",
                   "name": "Permissions",
@@ -407,10 +333,6 @@ exports[`SecurityPluginTopNavMenu renders DataSourceMenu when dataSource is enab
                   "href": "/users",
                   "name": "Internal users",
                 },
-                Object {
-                  "href": "/serviceAccounts",
-                  "name": "Service Accounts",
-                },
                 Object {
                   "href": "/permissions",
                   "name": "Permissions",
@@ -453,10 +375,6 @@ exports[`SecurityPluginTopNavMenu renders DataSourceMenu when dataSource is enab
                   "href": "/users",
                   "name": "Internal users",
                 },
-                Object {
-                  "href": "/serviceAccounts",
-                  "name": "Service Accounts",
-                },
                 Object {
                   "href": "/permissions",
                   "name": "Permissions",
@@ -499,10 +417,6 @@ exports[`SecurityPluginTopNavMenu renders DataSourceMenu when dataSource is enab
                   "href": "/users",
                   "name": "Internal users",
                 },
-                Object {
-                  "href": "/serviceAccounts",
-                  "name": "Service Accounts",
-                },
                 Object {
                   "href": "/permissions",
                   "name": "Permissions",
@@ -550,10 +464,6 @@ exports[`SecurityPluginTopNavMenu renders DataSourceMenu when dataSource is enab
             path="/users"
             render={[Function]}
           />
-          <Route
-            path="/serviceAccounts"
-            render={[Function]}
-          />
           <Route
             path="/auditLogging/edit/generalSettings"
             render={[Function]}
diff --git a/public/apps/configuration/utils/test/internal-user-list-utils.test.tsx b/public/apps/configuration/utils/test/internal-user-list-utils.test.tsx
index 7a0a819a4..fca702691 100644
--- a/public/apps/configuration/utils/test/internal-user-list-utils.test.tsx
+++ b/public/apps/configuration/utils/test/internal-user-list-utils.test.tsx
@@ -42,54 +42,26 @@ describe('Internal user list utils', () => {
     expect(result).toEqual(expectedUserList);
   });
 
-  it('getUserList calls httpGet with the correct parameters for internal users', async () => {
-    const httpMock = {}; // Mock HttpStart object
-    const userType = 'internalaccounts';
-
-    const test = await getUserList(httpMock, userType, 'test');
-
-    expect(mockedHttpGet).toHaveBeenCalledWith({
-      http: httpMock,
-      url: '/api/v1/configuration/internalaccounts',
-    });
-    expect(test).toEqual([]);
-  });
-
-  it('getUserList calls httpGet with the correct parameters for service accounts', async () => {
-    const httpMock = {};
-    const userType = 'serviceAccounts';
-
-    const test = await getUserList(httpMock, userType, 'test');
-
-    expect(mockedHttpGet).toHaveBeenCalledWith({
-      http: httpMock,
-      url: '/api/v1/configuration/serviceaccounts',
-    });
-    expect(test).toEqual([]);
-  });
-
-  it('fetchUserNameList calls httpGet with the correct parameters for service accounts', async () => {
+  it('getUserList calls httpGet with the correct parameters', async () => {
     const httpMock = {};
-    const userType = 'serviceAccounts';
 
-    const test = await fetchUserNameList(httpMock, userType, '');
+    const test = await getUserList(httpMock, 'test');
 
     expect(mockedHttpGet).toHaveBeenCalledWith({
       http: httpMock,
-      url: '/api/v1/configuration/serviceaccounts',
+      url: '/api/v1/configuration/internalusers',
     });
     expect(test).toEqual([]);
   });
 
-  it('fetchUserNameList calls httpGet with the correct parameters for internal users', async () => {
+  it('fetchUserNameList calls httpGet with the correct parameters', async () => {
     const httpMock = {};
-    const userType = 'internalaccounts';
 
-    const test = await fetchUserNameList(httpMock, userType, '');
+    const test = await fetchUserNameList(httpMock, '');
 
     expect(mockedHttpGet).toHaveBeenCalledWith({
       http: httpMock,
-      url: '/api/v1/configuration/internalaccounts',
+      url: '/api/v1/configuration/internalusers',
     });
     expect(test).toEqual([]);
   });