@openmrs/esm-report-builder 0.1.1-pre.92 → 0.1.1-pre.97

package/src/components/linelist/design/panels/column-category-selector.scss

@@ -46,10 +46,6 @@
     }
   }
 
-  .accordion {
-    // Override Carbon accordion defaults if needed
-  }
-
   .categoryHeader {
     display: flex;
     align-items: center;
@@ -316,14 +312,13 @@
     margin: 0;
     color: #161616;
   }
-}
 
-// Debug button styling
-button[icondescription="View column debug info"] {
-  svg {
-    fill: #0f62fe;
-  }
-  &:hover {
-    background-color: #e5f1ff !important;
+  button[icondescription="View column debug info"] {
+    svg {
+      fill: #0f62fe;
+    }
+    &:hover {
+      background-color: #e5f1ff !important;
+    }
   }
 }

package/src/components/linelist/design/panels/query-config-panel.scss

@@ -11,6 +11,13 @@
   &:last-child {
     border-bottom: none;
   }
+
+  :global {
+    .cds--tag {
+      font-size: 0.75rem;
+      font-weight: 500;
+    }
+  }
 }
 
 .sectionHeader {
@@ -332,11 +339,3 @@
   padding-top: 1.5rem;
   border-top: 1px solid #e5e7eb;
 }
-
-/* Teal accent for interactive elements */
-:global {
-  .cds--tag {
-    font-size: 0.75rem;
-    font-weight: 500;
-  }
-}

package/src/components/security/access-denied-page.component.tsx

@@ -1,69 +1,64 @@
 import React from 'react';
 import { useTranslation } from 'react-i18next';
-import { Button, InlineNotification, Tag } from '@carbon/react';
 import { useNavigate } from 'react-router-dom';
+import { Button, InlineNotification, Tag } from '@carbon/react';
 
 import { rbDisplay, RB } from '../../constants/privileges';
 import { useReportBuilderPrivileges } from '../../hooks/use-report-builder-privileges';
 import styles from './access-denied-page.scss';
 
 interface AccessDeniedPageProps {
-    /** Full privilege names, e.g. `Task: reportbuilder.report.edit`. */
-    requiredPrivileges?: string[];
+  /** Full privilege names, e.g. `Task: reportbuilder.report.edit`. */
+  requiredPrivileges?: string[];
 }
 
 const AccessDeniedPage: React.FC<AccessDeniedPageProps> = ({ requiredPrivileges }) => {
-    const { t } = useTranslation();
-    const navigate = useNavigate();
-    const { has } = useReportBuilderPrivileges();
-    const privileges = requiredPrivileges ?? [];
+  const { t } = useTranslation();
+  const navigate = useNavigate();
+  const { has } = useReportBuilderPrivileges();
+  const privileges = requiredPrivileges ?? [];
 
-    return (
-        <div className={styles.container}>
-            <InlineNotification
-                lowContrast
-                kind="error"
-                title={t('accessDenied', 'Access denied')}
-                subtitle={t(
-                    'accessDeniedSubtitle',
-                    'Your account does not have permission to open this page.',
-                )}
-                hideCloseButton
-            />
+  return (
+    <div className={styles.container}>
+      <InlineNotification
+        lowContrast
+        kind="error"
+        title={t('accessDenied', 'Access denied')}
+        subtitle={t('accessDeniedSubtitle', 'Your account does not have permission to open this page.')}
+        hideCloseButton
+      />
 
-            {privileges.length > 0 && (
-                <div className={styles.section}>
-                    <p className={styles.label}>
-                        {t('requiredPrivileges', 'Required privilege(s):')}
-                    </p>
-                    <div className={styles.tags}>
-                        {privileges.map((privilege) => (
-                            <Tag key={privilege} type="red">
-                                {rbDisplay(privilege)}
-                            </Tag>
-                        ))}
-                    </div>
-                    <p className={styles.hint}>
-                        {t(
-                            'accessDeniedHint',
-                            'Ask your administrator to grant you the report builder role carrying these privileges.',
-                        )}
-                    </p>
-                </div>
+      {privileges.length > 0 && (
+        <div className={styles.section}>
+          <p className={styles.label}>{t('requiredPrivileges', 'Required privilege(s):')}</p>
+          <div className={styles.tags}>
+            {privileges.map((privilege) => (
+              <Tag key={privilege} type="red">
+                {rbDisplay(privilege)}
+              </Tag>
+            ))}
+          </div>
+          <p className={styles.hint}>
+            {t(
+              'accessDeniedHint',
+              'Ask your administrator to grant you the report builder role carrying these privileges.',
             )}
-
-            <div className={styles.actions}>
-                {has(RB.REPORT_VIEW) && (
-                    <Button kind="primary" onClick={() => navigate('/')}>
-                        {t('goToHome', 'Go to Home')}
-                    </Button>
-                )}
-                <Button kind="secondary" onClick={() => navigate(-1)}>
-                    {t('goBack', 'Go back')}
-                </Button>
-            </div>
+          </p>
         </div>
-    );
+      )}
+
+      <div className={styles.actions}>
+        {has(RB.REPORT_VIEW) && (
+          <Button kind="primary" onClick={() => navigate('/')}>
+            {t('goToHome', 'Go to Home')}
+          </Button>
+        )}
+        <Button kind="secondary" onClick={() => navigate(-1)}>
+          {t('goBack', 'Go back')}
+        </Button>
+      </div>
+    </div>
+  );
 };
 
 export default AccessDeniedPage;

package/src/components/security/privilege-route.component.test.tsx

@@ -1,79 +1,76 @@
-import React from 'react';
+import React, { act } from 'react';
 import { createRoot } from 'react-dom/client';
-import { act } from 'react-dom/test-utils';
 import { MemoryRouter } from 'react-router-dom';
-
-jest.mock('@openmrs/esm-framework', () => ({
-    useSession: jest.fn(),
-}));
-
-import { useSession } from '@openmrs/esm-framework';
+import { describe, expect, it, vi } from 'vitest';
+import { useSession, type Session } from '@openmrs/esm-framework';
 import { RB } from '../../constants/privileges';
 import PrivilegeRoute from './privilege-route.component';
 
+const mockUseSession = vi.mocked(useSession);
+
 (globalThis as any).IS_REACT_ACT_ENVIRONMENT = true;
 
 function sessionWith(privileges: string[], authenticated = true) {
-    return {
-        authenticated,
-        user: {
-            uuid: 'u-1',
-            display: 'User',
-            privileges: privileges.map((name) => ({ uuid: name, name, display: name })),
-            roles: [],
-        },
-    } as any;
+  return {
+    authenticated,
+    user: {
+      uuid: 'u-1',
+      display: 'User',
+      privileges: privileges.map((name) => ({ uuid: name, name, display: name })),
+      roles: [],
+    },
+  } as Session;
 }
 
 function renderAt(route: string, ui: React.ReactElement): { text: () => string; unmount: () => void } {
-    const container = document.createElement('div');
-    document.body.appendChild(container);
-    const root = createRoot(container);
-    act(() => {
-        root.render(<MemoryRouter initialEntries={[route]}>{ui}</MemoryRouter>);
-    });
-    return {
-        text: () => container.textContent ?? '',
-        unmount: () => act(() => root.unmount()),
-    };
+  const container = document.createElement('div');
+  document.body.appendChild(container);
+  const root = createRoot(container);
+  act(() => {
+    root.render(<MemoryRouter initialEntries={[route]}>{ui}</MemoryRouter>);
+  });
+  return {
+    text: () => container.textContent ?? '',
+    unmount: () => act(() => root.unmount()),
+  };
 }
 
 describe('PrivilegeRoute', () => {
-    it('renders children when a required privilege is held (any-of)', () => {
-        (useSession as unknown as jest.Mock).mockReturnValue(sessionWith([RB.PACKAGE_EXPORT]));
-        const view = renderAt(
-            '/import-export',
-            <PrivilegeRoute required={[RB.PACKAGE_IMPORT, RB.PACKAGE_EXPORT]}>
-                <div>secret content</div>
-            </PrivilegeRoute>,
-        );
-        expect(view.text()).toContain('secret content');
-        view.unmount();
-    });
+  it('renders children when a required privilege is held (any-of)', () => {
+    mockUseSession.mockReturnValue(sessionWith([RB.PACKAGE_EXPORT]));
+    const view = renderAt(
+      '/import-export',
+      <PrivilegeRoute required={[RB.PACKAGE_IMPORT, RB.PACKAGE_EXPORT]}>
+        <div>secret content</div>
+      </PrivilegeRoute>,
+    );
+    expect(view.text()).toContain('secret content');
+    view.unmount();
+  });
 
-    it('renders the access denied page naming the privilege when unauthorised', () => {
-        (useSession as unknown as jest.Mock).mockReturnValue(sessionWith([RB.REPORT_VIEW]));
-        const view = renderAt(
-            '/run',
-            <PrivilegeRoute required={[RB.REPORT_RUN]}>
-                <div>secret content</div>
-            </PrivilegeRoute>,
-        );
-        expect(view.text()).not.toContain('secret content');
-        expect(view.text()).toContain('Access denied');
-        expect(view.text()).toContain('report.run');
-        view.unmount();
-    });
+  it('renders the access denied page naming the privilege when unauthorised', () => {
+    mockUseSession.mockReturnValue(sessionWith([RB.REPORT_VIEW]));
+    const view = renderAt(
+      '/run',
+      <PrivilegeRoute required={[RB.REPORT_RUN]}>
+        <div>secret content</div>
+      </PrivilegeRoute>,
+    );
+    expect(view.text()).not.toContain('secret content');
+    expect(view.text()).toContain('Access denied');
+    expect(view.text()).toContain('report.run');
+    view.unmount();
+  });
 
-    it('renders nothing and does not crash for an unauthenticated visitor', () => {
-        (useSession as unknown as jest.Mock).mockReturnValue(sessionWith([], false));
-        const view = renderAt(
-            '/',
-            <PrivilegeRoute required={[RB.REPORT_VIEW]}>
-                <div>secret content</div>
-            </PrivilegeRoute>,
-        );
-        expect(view.text()).toBe('');
-        view.unmount();
-    });
+  it('renders nothing and does not crash for an unauthenticated visitor', () => {
+    mockUseSession.mockReturnValue(undefined);
+    const view = renderAt(
+      '/',
+      <PrivilegeRoute required={[RB.REPORT_VIEW]}>
+        <div>secret content</div>
+      </PrivilegeRoute>,
+    );
+    expect(view.text()).toBe('');
+    view.unmount();
+  });
 });

package/src/components/security/privilege-route.component.tsx

@@ -5,9 +5,9 @@ import { redirectToLogin } from '../../utils/api-error.utils';
 import AccessDeniedPage from './access-denied-page.component';
 
 interface PrivilegeRouteProps {
-    /** Privileges granting access; any-of semantics, matching the backend's @Authorized. */
-    required: string[];
-    children: React.ReactNode;
+  /** Privileges granting access; any-of semantics, matching the backend's @Authorized. */
+  required: string[];
+  children: React.ReactNode;
 }
 
 /**

package/src/constants/privileges.test.ts

@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest';
 import { RB, RB_PREFIX, ROUTE_PRIVILEGES, rbDisplay } from './privileges';
 
 describe('privilege catalog', () => {

package/src/features/dashboards/utils/dashboard-config.util.test.ts

@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest';
 import {
   DEFAULT_SECTIONS,
   extractMonitorLayout,
@@ -22,7 +23,6 @@ function monitor(overrides: Partial<ETLMonitorDto> = {}): ETLMonitorDto {
     }),
     ...overrides,
   } as ETLMonitorDto;
-
 }
 
 function report(overrides: Partial<ReportDto> = {}): ReportDto {
@@ -118,7 +118,14 @@ describe('resolveDashboardLayout', () => {
         widgets: [],
         autoInclude: { etlMonitors: { enabled: true } },
       },
-      [monitor(), monitor({ uuid: 'm2', name: 'Status', displayConfigJson: JSON.stringify({ schemaVersion: 2, component: 'STATUS_CARD', fields: [] }) })],
+      [
+        monitor(),
+        monitor({
+          uuid: 'm2',
+          name: 'Status',
+          displayConfigJson: JSON.stringify({ schemaVersion: 2, component: 'STATUS_CARD', fields: [] }),
+        }),
+      ],
       [],
     );
     const history = resolved.sections.find((s) => s.key === 'history');
@@ -132,7 +139,10 @@ describe('resolveDashboardLayout', () => {
   it('explicit widgets win over auto-include (no duplicates, by uuid and by code)', () => {
     const config = {
       schemaVersion: 1 as const,
-      sections: [{ key: 'overview', label: 'Overview', order: 10 }, { key: 'history', label: 'History', order: 20 }],
+      sections: [
+        { key: 'overview', label: 'Overview', order: 10 },
+        { key: 'history', label: 'History', order: 20 },
+      ],
       widgets: [{ widgetType: 'ETL_MONITOR' as const, refUuid: 'm-uuid', sectionKey: 'overview' }],
       autoInclude: { etlMonitors: { enabled: true } },
     };
@@ -203,13 +213,34 @@ describe('resolveDashboardLayout', () => {
   });
 
   it('orders slots explicit-first, then by order/priority/title', () => {
-    const autoMonitor = monitor({ uuid: 'auto', name: 'beta auto', displayConfigJson: JSON.stringify({ schemaVersion: 2, component: 'STATUS_CARD', layout: { section: 'overview', span: {}, priority: 5 }, fields: [] }) });
+    const autoMonitor = monitor({
+      uuid: 'auto',
+      name: 'beta auto',
+      displayConfigJson: JSON.stringify({
+        schemaVersion: 2,
+        component: 'STATUS_CARD',
+        layout: { section: 'overview', span: {}, priority: 5 },
+        fields: [],
+      }),
+    });
     const config = {
       schemaVersion: 1 as const,
       sections: [{ key: 'overview', label: 'Overview', order: 10 }],
       widgets: [
-        { widgetType: 'ETL_MONITOR' as const, refUuid: 'm-uuid', sectionKey: 'overview', order: 200, titleOverride: 'second' },
-        { widgetType: 'ETL_MONITOR' as const, refUuid: 'm2', sectionKey: 'overview', order: 100, titleOverride: 'first' },
+        {
+          widgetType: 'ETL_MONITOR' as const,
+          refUuid: 'm-uuid',
+          sectionKey: 'overview',
+          order: 200,
+          titleOverride: 'second',
+        },
+        {
+          widgetType: 'ETL_MONITOR' as const,
+          refUuid: 'm2',
+          sectionKey: 'overview',
+          order: 100,
+          titleOverride: 'first',
+        },
       ],
       autoInclude: { etlMonitors: { enabled: true } },
     };

package/src/globals.d.ts

@@ -7,8 +7,3 @@ declare module '*.module.scss' {
   const classes: { [className: string]: string };
   export default classes;
 }
-
-// Webpack-specific typing used by OpenMRS translations loading
-interface Require {
-  context(directory: string, useSubdirectories: boolean, regExp: RegExp, mode?: string): any;
-}

package/src/hooks/use-report-builder-privileges.test.tsx

@@ -1,98 +1,92 @@
-import React from 'react';
+import React, { act } from 'react';
 import { createRoot } from 'react-dom/client';
-import { act } from 'react-dom/test-utils';
-
 import { useSession } from '@openmrs/esm-framework';
+import { describe, expect, it, vi } from 'vitest';
 import { useReportBuilderPrivileges } from './use-report-builder-privileges';
 import { RB } from '../constants/privileges';
 
-jest.mock('@openmrs/esm-framework', () => ({
-    useSession: jest.fn(),
-}));
-
-const mockUseSession = useSession as unknown as jest.Mock;
+const mockUseSession = vi.mocked(useSession);
 
 (globalThis as any).IS_REACT_ACT_ENVIRONMENT = true;
 
 const priv = (name: string) => ({ uuid: name, name, display: name });
 
 function renderHook<T>(hook: () => T): { current: T | undefined; rerender: () => void; unmount: () => void } {
-    const container = document.createElement('div');
-    const root = createRoot(container);
-    let current: T | undefined;
-    const Probe = () => {
-        current = hook();
-        return null;
-    };
-    const render = () => act(() => {
-        root.render(React.createElement(Probe));
+  const container = document.createElement('div');
+  const root = createRoot(container);
+  let current: T | undefined;
+  const Probe = () => {
+    current = hook();
+    return null;
+  };
+  const render = () =>
+    act(() => {
+      root.render(React.createElement(Probe));
     });
-    render();
-    return {
-        get current() {
-            return current;
-        },
-        rerender: render,
-        unmount: () => act(() => root.unmount()),
-    };
+  render();
+  return {
+    get current() {
+      return current;
+    },
+    rerender: render,
+    unmount: () => act(() => root.unmount()),
+  };
 }
 
 function sessionWith(privileges: string[], roles: string[] = [], authenticated = true) {
-    return {
-        authenticated,
-        user: {
-            uuid: 'u-1',
-            display: 'User',
-            privileges: privileges.map(priv),
-            roles: roles.map((r) => ({ uuid: r, display: r })),
-        },
-    } as any;
+  return {
+    authenticated,
+    user: {
+      uuid: 'u-1',
+      display: 'User',
+      privileges: privileges.map(priv),
+      roles: roles.map((r) => ({ uuid: r, display: r })),
+    },
+  } as any;
 }
 
 describe('useReportBuilderPrivileges', () => {
-    it('matches held privileges by name', () => {
-        mockUseSession.mockReturnValue(sessionWith([RB.REPORT_VIEW]));
-        const { current, unmount } = renderHook(() => useReportBuilderPrivileges());
-        expect(current!.has(RB.REPORT_VIEW)).toBe(true);
-        expect(current!.has(RB.REPORT_EDIT)).toBe(false);
-        expect(current!.isAuthenticated).toBe(true);
-        unmount();
-    });
+  it('matches held privileges by name', () => {
+    mockUseSession.mockReturnValue(sessionWith([RB.REPORT_VIEW]));
+    const { current, unmount } = renderHook(() => useReportBuilderPrivileges());
+    expect(current!.has(RB.REPORT_VIEW)).toBe(true);
+    expect(current!.has(RB.REPORT_EDIT)).toBe(false);
+    expect(current!.isAuthenticated).toBe(true);
+    unmount();
+  });
 
-    it('uses OR semantics like the backend @Authorized({a, b})', () => {
-        mockUseSession.mockReturnValue(sessionWith([RB.REPORT_ADD]));
-        const { current, unmount } = renderHook(() => useReportBuilderPrivileges());
-        expect(current!.has(RB.REPORT_ADD, RB.REPORT_EDIT)).toBe(true);
-        expect(current!.hasAll(RB.REPORT_ADD, RB.REPORT_EDIT)).toBe(false);
-        unmount();
-    });
+  it('uses OR semantics like the backend @Authorized({a, b})', () => {
+    mockUseSession.mockReturnValue(sessionWith([RB.REPORT_ADD]));
+    const { current, unmount } = renderHook(() => useReportBuilderPrivileges());
+    expect(current!.has(RB.REPORT_ADD, RB.REPORT_EDIT)).toBe(true);
+    expect(current!.hasAll(RB.REPORT_ADD, RB.REPORT_EDIT)).toBe(false);
+    unmount();
+  });
 
-    it('treats the System Developer role as superuser', () => {
-        mockUseSession.mockReturnValue(sessionWith([], ['System Developer']));
-        const { current, unmount } = renderHook(() => useReportBuilderPrivileges());
-        expect(current!.isSuperUser).toBe(true);
-        expect(current!.has(RB.ETLMONITOR_PURGE)).toBe(true);
-        expect(current!.hasAll(RB.PACKAGE_IMPORT, RB.PACKAGE_EXPORT)).toBe(true);
-        unmount();
-    });
+  it('treats the System Developer role as superuser', () => {
+    mockUseSession.mockReturnValue(sessionWith([], ['System Developer']));
+    const { current, unmount } = renderHook(() => useReportBuilderPrivileges());
+    expect(current!.isSuperUser).toBe(true);
+    expect(current!.has(RB.ETLMONITOR_PURGE)).toBe(true);
+    expect(current!.hasAll(RB.PACKAGE_IMPORT, RB.PACKAGE_EXPORT)).toBe(true);
+    unmount();
+  });
 
-    it('exposes convenience flags', () => {
-        mockUseSession.mockReturnValue(
-            sessionWith([RB.REPORT_VIEW, RB.REPORT_RUN, RB.PACKAGE_EXPORT]),
-        );
-        const { current, unmount } = renderHook(() => useReportBuilderPrivileges());
-        expect(current!.canAuthor).toBe(false);
-        expect(current!.canRun).toBe(true);
-        expect(current!.canPublish).toBe(true);
-        expect(current!.canManageEtl).toBe(false);
-        unmount();
-    });
+  it('exposes convenience flags', () => {
+    mockUseSession.mockReturnValue(sessionWith([RB.REPORT_VIEW, RB.REPORT_RUN, RB.PACKAGE_EXPORT]));
+    const { current, unmount } = renderHook(() => useReportBuilderPrivileges());
+    expect(current!.canAuthor).toBe(false);
+    expect(current!.canRun).toBe(true);
+    expect(current!.canPublish).toBe(true);
+    expect(current!.canManageEtl).toBe(false);
+    unmount();
+  });
 
-    it('is false on everything for an unauthenticated session', () => {
-        mockUseSession.mockReturnValue({ authenticated: false, user: undefined });
-        const { current, unmount } = renderHook(() => useReportBuilderPrivileges());
-        expect(current!.isAuthenticated).toBe(false);
-        expect(current!.has(RB.REPORT_VIEW)).toBe(false);
-        unmount();
-    });
+  it('is false on everything for an unauthenticated session', () => {
+    mockUseSession.mockReturnValue({ authenticated: false, user: undefined, sessionId: '' });
+    const { current, unmount } = renderHook(() => useReportBuilderPrivileges());
+    expect(current!.isAuthenticated).toBe(false);
+    expect(current!.has(RB.REPORT_VIEW)).toBe(false);
+    unmount();
+  });
 });

package/src/routes.json

@@ -1,5 +1,8 @@
 {
   "$schema": "https://json.openmrs.org/routes.schema.json",
+  "backendDependencies": {
+    "reportbuilder": "*"
+  },
   "pages": [
     {
       "component": "root",

package/src/utils/api-error.utils.test.ts

@@ -1,3 +1,4 @@
+import { describe, expect, it, vi } from 'vitest';
 import { RB } from '../constants/privileges';
 import { guardRejection, normalizeApiError, PrivilegeError } from './api-error.utils';
 
@@ -92,9 +93,7 @@ describe('normalizeApiError', () => {
   describe('401 (expired session)', () => {
     it('marks the error as an auth error and redirects to the SPA login page', () => {
       stubLocation();
-      const error = normalizeApiError(
-        axiosError(401, { error: { message: 'Session has expired' } }),
-      ) as PrivilegeError;
+      const error = normalizeApiError(axiosError(401, { error: { message: 'Session has expired' } })) as PrivilegeError;
 
       expect(error).toBeInstanceOf(PrivilegeError);
       expect(error.isAuthError).toBe(true);
@@ -112,9 +111,7 @@ describe('normalizeApiError', () => {
   });
 
   it('maps other statuses to a generic error carrying the server message', () => {
-    const error = normalizeApiError(
-      axiosError(500, { error: { message: 'Boom' } }),
-    ) as Error & { status?: number };
+    const error = normalizeApiError(axiosError(500, { error: { message: 'Boom' } })) as Error & { status?: number };
     expect(error).not.toBeInstanceOf(PrivilegeError);
     expect(error.message).toBe('Boom');
     expect(error.status).toBe(500);
@@ -134,18 +131,17 @@ describe('guardRejection', () => {
 });
 
 describe('redirectToLogin loop guard', () => {
-  it('navigates at most once across repeated calls (module-level latch)', () => {
+  it('navigates at most once across repeated calls (module-level latch)', async () => {
+    vi.resetModules();
+    const fresh = await import('./api-error.utils');
+
     stubLocation();
-    jest.isolateModules(() => {
-      // eslint-disable-next-line @typescript-eslint/no-var-requires -- fresh module instance for the latch test
-      const fresh = require('./api-error.utils');
-      stubLocation();
-      fresh.redirectToLogin();
-      expect(window.location.href).toContain('/login');
-      // The latch makes further calls no-ops even with a fresh path.
-      stubLocation({ pathname: '/openmrs/spa/report-builder/admin', href: 'unchanged' });
-      fresh.redirectToLogin();
-      expect(window.location.href).toBe('unchanged');
-    });
+
+    fresh.redirectToLogin();
+    expect(window.location.href).toContain('/login');
+    // The latch makes further calls no-ops even with a fresh path.
+    stubLocation({ pathname: '/openmrs/spa/report-builder/admin', href: 'unchanged' });
+    fresh.redirectToLogin();
+    expect(window.location.href).toBe('unchanged');
   });
 });