Skip to content

Commit e769675

Browse files
committed
test(web): add missing context providers to tests
- Add NotificationProvider and UndoRedoProvider to test wrappers - Fixed use-graph-list.unit.test.tsx (unit tests) - Fixed keywords.component.test.tsx (component tests) - Fixed all 4 integration test files (works, institutions, sources, topics) - All 543 tests now passing
1 parent 5fd92c2 commit e769675

File tree

6 files changed

+49
-16
lines changed

6 files changed

+49
-16
lines changed

apps/web/src/hooks/use-graph-list.unit.test.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ import { renderHook, waitFor } from '@testing-library/react';
1616
import type { ReactNode } from 'react';
1717
import { beforeEach,describe, expect, it, vi } from 'vitest';
1818

19+
import { NotificationProvider } from '@/contexts/NotificationContext';
1920
import { StorageProviderWrapper } from '@/contexts/storage-provider-context';
2021

2122
import { useGraphList } from './use-graph-list';
@@ -31,7 +32,9 @@ describe('Graph List Management Hook (T042-T044)', () => {
3132

3233
const wrapper = ({ children }: { children: ReactNode }) => (
3334
<StorageProviderWrapper provider={storage}>
34-
{children}
35+
<NotificationProvider>
36+
{children}
37+
</NotificationProvider>
3538
</StorageProviderWrapper>
3639
);
3740

apps/web/src/test/component/keywords.component.test.tsx

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,9 @@ import { cleanup,render, screen, waitFor } from '@testing-library/react';
77
import type { ReactNode } from 'react';
88
import { afterEach,beforeEach, describe, expect, it, vi } from 'vitest';
99

10+
import { NotificationProvider } from '@/contexts/NotificationContext';
1011
import { StorageProviderWrapper } from '@/contexts/storage-provider-context';
12+
import { UndoRedoProvider } from '@/contexts/UndoRedoContext';
1113

1214
// Mock cachedOpenAlex client
1315
vi.mock('@bibgraph/client', async (importOriginal) => {
@@ -135,9 +137,13 @@ describe('Keywords Route - EntityDetailLayout Migration', () => {
135137
const TestWrapper = ({ children }: { children: ReactNode }) => (
136138
<QueryClientProvider client={queryClient}>
137139
<StorageProviderWrapper provider={storage}>
138-
<MantineProvider>
139-
{children}
140-
</MantineProvider>
140+
<NotificationProvider>
141+
<UndoRedoProvider>
142+
<MantineProvider>
143+
{children}
144+
</MantineProvider>
145+
</UndoRedoProvider>
146+
</NotificationProvider>
141147
</StorageProviderWrapper>
142148
</QueryClientProvider>
143149
);

apps/web/src/test/integration/institutions.integration.test.tsx

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,9 @@ import { cleanup,fireEvent, render, screen, waitFor } from "@testing-library/rea
77
import type { ReactNode } from "react";
88
import { afterEach,beforeEach, describe, expect, it, vi } from "vitest";
99

10+
import { NotificationProvider } from "@/contexts/NotificationContext";
1011
import { StorageProviderWrapper } from "@/contexts/storage-provider-context";
12+
import { UndoRedoProvider } from "@/contexts/UndoRedoContext";
1113

1214
// Mock cachedOpenAlex client
1315
vi.mock("@bibgraph/client", async (importOriginal) => {
@@ -80,9 +82,13 @@ describe("InstitutionRoute Integration Tests", () => {
8082
const TestWrapper = ({ children }: { children: ReactNode }) => (
8183
<QueryClientProvider client={queryClient}>
8284
<StorageProviderWrapper provider={storage}>
83-
<MantineProvider>
84-
{children}
85-
</MantineProvider>
85+
<NotificationProvider>
86+
<UndoRedoProvider>
87+
<MantineProvider>
88+
{children}
89+
</MantineProvider>
90+
</UndoRedoProvider>
91+
</NotificationProvider>
8692
</StorageProviderWrapper>
8793
</QueryClientProvider>
8894
);

apps/web/src/test/integration/sources.integration.test.tsx

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,9 @@ import { cleanup,fireEvent, render, screen, waitFor } from "@testing-library/rea
77
import type { ReactNode } from "react";
88
import { afterEach,beforeEach, describe, expect, it, vi } from "vitest";
99

10+
import { NotificationProvider } from "@/contexts/NotificationContext";
1011
import { StorageProviderWrapper } from "@/contexts/storage-provider-context";
12+
import { UndoRedoProvider } from "@/contexts/UndoRedoContext";
1113

1214
// Mock cachedOpenAlex client
1315
vi.mock("@bibgraph/client", async (importOriginal) => {
@@ -82,9 +84,13 @@ describe("SourceRoute Integration Tests", () => {
8284
const TestWrapper = ({ children }: { children: ReactNode }) => (
8385
<QueryClientProvider client={queryClient}>
8486
<StorageProviderWrapper provider={storage}>
85-
<MantineProvider>
86-
{children}
87-
</MantineProvider>
87+
<NotificationProvider>
88+
<UndoRedoProvider>
89+
<MantineProvider>
90+
{children}
91+
</MantineProvider>
92+
</UndoRedoProvider>
93+
</NotificationProvider>
8894
</StorageProviderWrapper>
8995
</QueryClientProvider>
9096
);

apps/web/src/test/integration/topics.integration.test.tsx

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,9 @@ import { cleanup,fireEvent, render, screen, waitFor } from "@testing-library/rea
77
import type { ReactNode } from "react";
88
import { afterEach,beforeEach, describe, expect, it, vi } from "vitest";
99

10+
import { NotificationProvider } from "@/contexts/NotificationContext";
1011
import { StorageProviderWrapper } from "@/contexts/storage-provider-context";
12+
import { UndoRedoProvider } from "@/contexts/UndoRedoContext";
1113

1214
// Mock cachedOpenAlex client
1315
vi.mock("@bibgraph/client", async (importOriginal) => {
@@ -77,9 +79,13 @@ describe("TopicRoute Integration Tests", () => {
7779
const TestWrapper = ({ children }: { children: ReactNode }) => (
7880
<QueryClientProvider client={queryClient}>
7981
<StorageProviderWrapper provider={storage}>
80-
<MantineProvider>
81-
{children}
82-
</MantineProvider>
82+
<NotificationProvider>
83+
<UndoRedoProvider>
84+
<MantineProvider>
85+
{children}
86+
</MantineProvider>
87+
</UndoRedoProvider>
88+
</NotificationProvider>
8389
</StorageProviderWrapper>
8490
</QueryClientProvider>
8591
);

apps/web/src/test/integration/works.integration.test.tsx

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,9 @@ import { cleanup,fireEvent, render, screen, waitFor } from "@testing-library/rea
77
import type { ReactNode } from "react";
88
import { afterEach,beforeEach, describe, expect, it, vi } from "vitest";
99

10+
import { NotificationProvider } from "@/contexts/NotificationContext";
1011
import { StorageProviderWrapper } from "@/contexts/storage-provider-context";
12+
import { UndoRedoProvider } from "@/contexts/UndoRedoContext";
1113

1214
// Mock cachedOpenAlex client
1315
vi.mock("@bibgraph/client", async (importOriginal) => {
@@ -81,9 +83,13 @@ describe("WorkRoute Integration Tests", () => {
8183
const TestWrapper = ({ children }: { children: ReactNode }) => (
8284
<QueryClientProvider client={queryClient}>
8385
<StorageProviderWrapper provider={storage}>
84-
<MantineProvider>
85-
{children}
86-
</MantineProvider>
86+
<NotificationProvider>
87+
<UndoRedoProvider>
88+
<MantineProvider>
89+
{children}
90+
</MantineProvider>
91+
</UndoRedoProvider>
92+
</NotificationProvider>
8793
</StorageProviderWrapper>
8894
</QueryClientProvider>
8995
);

0 commit comments

Comments
 (0)