test(ui): Prepare for jest 30 (#94724)

Pulls just the testable window location changes out for Jest 30 #93303

window.location in the next version of jsdom is not configurable and
cannot be changed via assignment. By using testableWindowLocation we can
mock the entire module and do whatever we want instead of trying to
overwrite the jsdom globals

Author: scttcper

static/app/actionCreators/account.spec.tsx

@@ -1,5 +1,7 @@
 import {waitFor} from 'sentry-test/reactTestingLibrary';
 
+import {testableWindowLocation} from 'sentry/utils/testableWindowLocation';
+
 import {logout} from './account';
 
 describe('logout', () => {
@@ -13,6 +15,6 @@ describe('logout', () => {
     logout(mockApi);
 
     await waitFor(() => expect(mockApiDelete).toHaveBeenCalled());
-    expect(window.location.assign).toHaveBeenCalledWith('/auth/login/');
+    expect(testableWindowLocation.assign).toHaveBeenCalledWith('/auth/login/');
   });
 });

static/app/actionCreators/account.tsx

@@ -4,6 +4,7 @@ import ConfigStore from 'sentry/stores/configStore';
 import type {UserIdentityConfig} from 'sentry/types/auth';
 import type {User} from 'sentry/types/user';
 import {isDemoModeActive} from 'sentry/utils/demoMode';
+import {testableWindowLocation} from 'sentry/utils/testableWindowLocation';
 import type {ChangeAvatarUser} from 'sentry/views/settings/account/accountDetails';
 
 export async function disconnectIdentity(
@@ -51,7 +52,7 @@ export async function logout(api: Client, redirectUrl?: string) {
   const data = await api.requestPromise('/auth/', {method: 'DELETE'});
 
   // If there's a URL for SAML Single-logout, redirect back to IdP
-  window.location.assign(data?.sloUrl || getRedirectUrl(redirectUrl));
+  testableWindowLocation.assign(data?.sloUrl || getRedirectUrl(redirectUrl));
 }
 
 function getRedirectUrl(redirectUrl = '/auth/login/') {

static/app/actionCreators/organizations.spec.tsx

@@ -2,6 +2,7 @@ import {OrganizationFixture} from 'sentry-fixture/organization';
 
 import {fetchOrganizations} from 'sentry/actionCreators/organizations';
 import ConfigStore from 'sentry/stores/configStore';
+import {testableWindowLocation} from 'sentry/utils/testableWindowLocation';
 
 describe('fetchOrganizations', function () {
   const api = new MockApiClient();
@@ -74,6 +75,6 @@ describe('fetchOrganizations', function () {
     expect(organizations[0].slug).toEqual(usorg.slug);
     expect(usMock).toHaveBeenCalledTimes(1);
     expect(deMock).toHaveBeenCalledTimes(1);
-    expect(window.location.reload).not.toHaveBeenCalled();
+    expect(testableWindowLocation.reload).not.toHaveBeenCalled();
   });
 });

static/app/api.spec.tsx

@@ -1,5 +1,7 @@
 import {OrganizationFixture} from 'sentry-fixture/organization';
 
+import {setWindowLocation} from 'sentry-test/utils';
+
 import type {Client, ResponseMeta} from 'sentry/api';
 import {isSimilarOrigin, Request, resolveHostname} from 'sentry/api';
 import {PROJECT_MOVED} from 'sentry/constants/apiErrorCodes';
@@ -14,6 +16,7 @@ describe('api', function () {
 
   beforeEach(function () {
     api = new MockApiClient();
+    setWindowLocation('https://sentry.io/');
   });
 
   describe('Client', function () {
@@ -143,11 +146,7 @@ describe('resolveHostname', function () {
   });
 
   it('does not override region in _admin', function () {
-    Object.defineProperty(window, 'location', {
-      configurable: true,
-      enumerable: true,
-      value: new URL('https://sentry.io/_admin/'),
-    });
+    setWindowLocation('https://sentry.io/_admin/');
 
     // Adds domain to control paths
     let result = resolveHostname(controlPath);
@@ -166,6 +165,7 @@ describe('resolveHostname', function () {
   });
 
   it('adds domains when feature enabled', function () {
+    setWindowLocation('https://us.sentry.io/');
     let result = resolveHostname(regionPath);
     expect(result).toBe('https://us.sentry.io/api/0/organizations/slug/issues/');
 

static/app/components/modals/redirectToProject.spec.tsx

@@ -3,6 +3,7 @@ import {act, renderGlobalModal} from 'sentry-test/reactTestingLibrary';
 
 import {openModal} from 'sentry/actionCreators/modal';
 import {RedirectToProjectModal} from 'sentry/components/modals/redirectToProject';
+import {testableWindowLocation} from 'sentry/utils/testableWindowLocation';
 
 jest.unmock('sentry/utils/recreateRoute');
 
@@ -29,10 +30,10 @@ describe('RedirectToProjectModal', function () {
     );
 
     act(() => jest.advanceTimersByTime(4900));
-    expect(window.location.assign).not.toHaveBeenCalled();
+    expect(testableWindowLocation.assign).not.toHaveBeenCalled();
 
     act(() => jest.advanceTimersByTime(200));
-    expect(window.location.assign).toHaveBeenCalledTimes(1);
-    expect(window.location.assign).toHaveBeenCalledWith('/org-slug/new-slug/');
+    expect(testableWindowLocation.assign).toHaveBeenCalledTimes(1);
+    expect(testableWindowLocation.assign).toHaveBeenCalledWith('/org-slug/new-slug/');
   });
 });

static/app/components/modals/redirectToProject.tsx

@@ -7,6 +7,7 @@ import Text from 'sentry/components/text';
 import {t, tct} from 'sentry/locale';
 import type {WithRouterProps} from 'sentry/types/legacyReactRouter';
 import recreateRoute from 'sentry/utils/recreateRoute';
+import {testableWindowLocation} from 'sentry/utils/testableWindowLocation';
 // eslint-disable-next-line no-restricted-imports
 import withSentryRouter from 'sentry/utils/withSentryRouter';
 
@@ -27,7 +28,7 @@ class RedirectToProjectModal extends Component<Props, State> {
   componentDidMount() {
     this.redirectInterval = window.setInterval(() => {
       if (this.state.timer <= 1) {
-        window.location.assign(this.newPath);
+        testableWindowLocation.assign(this.newPath);
         return;
       }
 

static/app/components/modals/reprocessEventModal.spec.tsx

@@ -11,6 +11,7 @@ import {
   ModalFooter,
 } from 'sentry/components/globalModal/components';
 import {ReprocessingEventModal} from 'sentry/components/modals/reprocessEventModal';
+import {testableWindowLocation} from 'sentry/utils/testableWindowLocation';
 
 const group = GroupFixture({
   id: '1337',
@@ -104,7 +105,7 @@ describe('ReprocessEventModal', function () {
 
     await userEvent.click(screen.getByRole('button', {name: 'Reprocess Events'}));
 
-    await waitFor(() => expect(window.location.reload).toHaveBeenCalled());
+    await waitFor(() => expect(testableWindowLocation.reload).toHaveBeenCalled());
     expect(handleCloseModal).toHaveBeenCalled();
   });
 });

static/app/components/modals/reprocessEventModal.tsx

@@ -13,6 +13,7 @@ import {t, tct} from 'sentry/locale';
 import {space} from 'sentry/styles/space';
 import type {Group} from 'sentry/types/group';
 import type {Organization} from 'sentry/types/organization';
+import {testableWindowLocation} from 'sentry/utils/testableWindowLocation';
 
 export type ReprocessEventModalOptions = {
   groupId: Group['id'];
@@ -30,7 +31,7 @@ export function ReprocessingEventModal({
 
   function handleSuccess() {
     closeModal();
-    window.location.reload();
+    testableWindowLocation.reload();
   }
 
   return (

static/app/components/profiling/flamegraph/flamegraph.spec.tsx

@@ -2,6 +2,7 @@ import {ProjectFixture} from 'sentry-fixture/project';
 
 import {initializeOrg} from 'sentry-test/initializeOrg';
 import {act, render, screen, userEvent} from 'sentry-test/reactTestingLibrary';
+import {setWindowLocation} from 'sentry-test/utils';
 
 import ProjectsStore from 'sentry/stores/projectsStore';
 import {FlamegraphRendererDOM as mockFlameGraphRenderer} from 'sentry/utils/profiling/renderers/testUtils';
@@ -109,6 +110,7 @@ describe('Flamegraph', function () {
   beforeEach(() => {
     const project = ProjectFixture({slug: 'foo-project'});
     act(() => void ProjectsStore.loadInitialData([project]));
+    setWindowLocation('http://localhost/');
   });
   it('renders a missing profile', async function () {
     MockApiClient.addMockResponse({
@@ -185,8 +187,9 @@ describe('Flamegraph', function () {
       eventId: 'profile-id',
     });
 
-    window.location.search =
-      '?colorCoding=by+library&query=&sorting=alphabetical&tid=0&view=bottom+up';
+    setWindowLocation(
+      'http://localhost/?colorCoding=by+library&query=&sorting=alphabetical&tid=0&view=bottom+up'
+    );
 
     render(
       <ProfilesAndTransactionProvider>
@@ -222,7 +225,7 @@ describe('Flamegraph', function () {
       eventId: 'profile-id',
     });
 
-    window.location.search = '?query=profiling+transaction';
+    setWindowLocation('http://localhost/?query=profiling+transaction');
 
     render(
       <ProfilesAndTransactionProvider>

static/app/components/sidebar/sidebarAccordion.spec.tsx

@@ -1,12 +1,13 @@
 import {render, screen} from 'sentry-test/reactTestingLibrary';
+import {setWindowLocation} from 'sentry-test/utils';
 
 import {SidebarAccordion} from 'sentry/components/sidebar/sidebarAccordion';
 import SidebarItem from 'sentry/components/sidebar/sidebarItem';
 import {IconStar} from 'sentry/icons';
 
 describe('SidebarAccordion', function () {
   it('marks only the selected child as active', function () {
-    window.location.pathname = '/performance/queries?sort=tpm()';
+    setWindowLocation('http://localhost/performance/queries?sort=tpm()');
 
     render(
       <SidebarAccordion