Migrate Jest test suite to Jest 30 + jsdom 26 (#2475)

* test: migrate Jest test suite to Jest 30 + jsdom 26

Follows the core OpenSearch-Dashboards upgrade
(opensearch-project/OpenSearch-Dashboards#12381). Full suite green under
Jest 30.4.2 / jsdom 26.1.0.

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* test: map jose to its CommonJS build in server jest config

Under Jest 30's stricter package 'exports' resolution, 'import { SignJWT } from jose'
in the jwt_auth / jwt_multiauth integration tests resolved to jose's pure-ESM browser
build (dist/browser/index.js), failing with 'SyntaxError: Unexpected token export'.
Pin '^jose$' to the CommonJS build (dist/node/cjs/index.js) the 'require' condition
uses. The suites now load correctly (remaining failures require a live OpenSearch cluster).

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* test: wrap jose moduleNameMapper line to satisfy prettier print-width

The single-line '^jose$' mapping exceeded the 100-char print width; wrap the value
onto its own line so prettier/prettier passes in CI (OSD-core prettier 3.5.x).

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

---------

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

Author: SuZhou-Joe

public/apps/account/account-app.tsx

@@ -56,8 +56,9 @@ export async function setupTopNavButton(coreStart: CoreStart, config: ClientConf
   if (accountInfo) {
     // Missing role error
     if (accountInfo.roles.length === 0 && !window.location.href.includes(CUSTOM_ERROR_PAGE_URI)) {
-      window.location.href =
-        coreStart.http.basePath.serverBasePath + CUSTOM_ERROR_PAGE_URI + ERROR_MISSING_ROLE_PATH;
+      window.location.assign(
+        coreStart.http.basePath.serverBasePath + CUSTOM_ERROR_PAGE_URI + ERROR_MISSING_ROLE_PATH
+      );
     }
 
     let tenant: string | undefined;

public/apps/account/account-nav-button.tsx

@@ -235,5 +235,5 @@ export function reloadAfterTenantSwitch(): void {
 
   // rather than just reload when we switch tenants, we set the URL to the pathname. i.e. the portion like: '/app/dashboards'
   // therefore, the copied URL will now allow tenancy changes.
-  window.location.href = window.location.pathname;
+  window.location.assign(window.location.pathname);
 }

public/apps/account/password-reset-panel.tsx

@@ -178,7 +178,7 @@ export function PasswordResetPanel(props: PasswordResetPanelProps) {
           <EuiSpacer />
 
           {errorCallOut && (
-            <EuiCallOut color="danger" iconType="alert">
+            <EuiCallOut announceOnMount color="danger" iconType="alert">
               {errorCallOut}
             </EuiCallOut>
           )}

public/apps/account/tenant-switch-panel.tsx

@@ -270,7 +270,7 @@ export function TenantSwitchPanel(props: TenantSwitchPanelProps) {
         <EuiSpacer />
 
         {errorCallOut && (
-          <EuiCallOut color="danger" iconType="alert">
+          <EuiCallOut announceOnMount color="danger" iconType="alert">
             {errorCallOut}
           </EuiCallOut>
         )}

public/apps/account/test/__snapshots__/account-nav-button.test.tsx.snap

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`Account navigation button renders 1`] = `
 <EuiHeaderSectionItemButton

public/apps/account/test/__snapshots__/log-out-button.test.tsx.snap

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`Account menu - Log out button renders renders when auth type is MultiAuth: basicauth 1`] = `
 <div>

public/apps/account/test/__snapshots__/role-info-panel.test.tsx.snap

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`Account menu - Role info panel renders 1`] = `
 <EuiOverlayMask>

public/apps/account/test/__snapshots__/tenant-switch-panel.test.tsx.snap

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing
 
 exports[`Account menu -tenant switch panel confirm button and renders renders when both global and private tenant enabled 1`] = `
 <EuiOverlayMask>

public/apps/account/test/account-app.test.tsx

@@ -78,16 +78,14 @@ describe('Account app', () => {
   });
 
   it('Should skip if auto swich if securitytenant in url', (done) => {
-    // Trick to mock window.location
-    const originalLocation = window.location;
-    delete window.location;
-    window.location = new URL('http://www.example.com?securitytenant=abc') as any;
+    // jest-location-mock: assign a URL with securitytenant instead of replacing window.location.
+    window.location.assign('http://www.example.com?securitytenant=abc');
 
     setupTopNavButton(mockCoreStart, mockConfig as any);
 
     process.nextTick(() => {
-      expect(setShouldShowTenantPopup).toBeCalledWith(false);
-      window.location = originalLocation;
+      expect(setShouldShowTenantPopup).toHaveBeenCalledWith(false);
+      window.location.assign('http://localhost:5601/');
       done();
     });
   });
@@ -98,11 +96,11 @@ describe('Account app', () => {
     setupTopNavButton(mockCoreStart, mockConfig as any);
 
     process.nextTick(() => {
-      expect(getSavedTenant).toBeCalledTimes(1);
+      expect(getSavedTenant).toHaveBeenCalledTimes(1);
     });
 
     process.nextTick(() => {
-      expect(setShouldShowTenantPopup).toBeCalledWith(false);
+      expect(setShouldShowTenantPopup).toHaveBeenCalledWith(false);
       done();
     });
   });
@@ -113,8 +111,8 @@ describe('Account app', () => {
     setupTopNavButton(mockCoreStart, mockConfig as any);
 
     process.nextTick(() => {
-      expect(getSavedTenant).toBeCalledTimes(1);
-      expect(setShouldShowTenantPopup).toBeCalledWith(true);
+      expect(getSavedTenant).toHaveBeenCalledTimes(1);
+      expect(setShouldShowTenantPopup).toHaveBeenCalledWith(true);
       done();
     });
   });
@@ -128,8 +126,8 @@ describe('Account app', () => {
     setupTopNavButton(mockCoreStart, mockConfig as any);
 
     process.nextTick(() => {
-      expect(getSavedTenant).toBeCalledTimes(0);
-      expect(setShouldShowTenantPopup).toBeCalledWith(false);
+      expect(getSavedTenant).toHaveBeenCalledTimes(0);
+      expect(setShouldShowTenantPopup).toHaveBeenCalledWith(false);
       done();
     });
   });
@@ -144,8 +142,8 @@ describe('Account app', () => {
     setupTopNavButton(mockCoreStart, multiTenancyDisabledConfig as any);
 
     process.nextTick(() => {
-      expect(getSavedTenant).toBeCalledTimes(0);
-      expect(setShouldShowTenantPopup).toBeCalledWith(false);
+      expect(getSavedTenant).toHaveBeenCalledTimes(0);
+      expect(setShouldShowTenantPopup).toHaveBeenCalledWith(false);
       done();
     });
   });
@@ -159,8 +157,8 @@ describe('Account app', () => {
     setupTopNavButton(mockCoreStart, mockConfig as any);
 
     process.nextTick(() => {
-      expect(getSavedTenant).toBeCalledTimes(0);
-      expect(setShouldShowTenantPopup).toBeCalledWith(false);
+      expect(getSavedTenant).toHaveBeenCalledTimes(0);
+      expect(setShouldShowTenantPopup).toHaveBeenCalledWith(false);
       done();
     });
   });

public/apps/account/test/account-nav-button.test.tsx

@@ -109,27 +109,27 @@ describe('Account navigation button', () => {
         currAuthType={'dummy'}
       />
     );
-    expect(setState).toBeCalledTimes(1);
+    expect(setState).toHaveBeenCalledTimes(1);
   });
 
   it('should set modal when click on "View roles and identities" button', () => {
     component.find('[data-test-subj="view-roles-and-identities"]').simulate('click');
-    expect(setState).toBeCalledTimes(1);
+    expect(setState).toHaveBeenCalledTimes(1);
   });
 
   it('should set modal when click on "Switch tenants" button', () => {
     component.find('[data-test-subj="switch-tenants"]').simulate('click');
-    expect(setState).toBeCalledTimes(1);
+    expect(setState).toHaveBeenCalledTimes(1);
   });
 
   it('should set modal when click on "Reset password" button', () => {
     component.find('[data-test-subj="reset-password"]').simulate('click');
-    expect(setState).toBeCalledTimes(1);
+    expect(setState).toHaveBeenCalledTimes(1);
   });
 
   it('should set isPopoverOpen to true when click on Avatar in header section', () => {
     component.find('[data-test-subj="account-popover"]').simulate('click');
-    expect(setState).toBeCalledTimes(1);
+    expect(setState).toHaveBeenCalledTimes(1);
   });
 });
 
@@ -172,7 +172,7 @@ describe('Account navigation button, multitenancy disabled', () => {
         currAuthType={'dummy'}
       />
     );
-    expect(setState).toBeCalledTimes(0);
+    expect(setState).toHaveBeenCalledTimes(0);
   });
 });
 
@@ -247,34 +247,12 @@ describe('Shows tenant info when multitenancy enabled, and hides it if disabled'
 });
 
 describe('Reload window after tenant switch', () => {
-  const originalLocation = window.location;
-  const mockSetWindowHref = jest.fn();
-  let pathname: string = '';
-  beforeAll(() => {
-    pathname = '/app/myapp';
-    Object.defineProperty(window, 'location', {
-      value: {
-        get pathname() {
-          return pathname;
-        },
-        get href() {
-          return '/app/dashboards?security_tenant=admin_tenant';
-        },
-        set href(value: string) {
-          mockSetWindowHref(value);
-        },
-      },
-    });
-  });
-
-  afterAll(() => {
-    window.location = originalLocation;
-  });
-
   it('should remove the tenant query parameter before reloading', () => {
-    pathname = '/app/pathname-only';
+    // jest-location-mock: reloadAfterTenantSwitch() calls window.location.assign(pathname).
+    window.location.assign('http://localhost:5601/app/pathname-only?security_tenant=admin_tenant');
+    const assignSpy = jest.spyOn(window.location, 'assign');
     reloadAfterTenantSwitch();
-    expect(mockSetWindowHref).toHaveBeenCalledWith(pathname);
+    expect(assignSpy).toHaveBeenCalledWith('/app/pathname-only');
   });
 });