Skip to content

Commit 52e1f18

Browse files
markgohoclaude
andcommitted
fix: remove vi.mock() from admin-messages test for Angular 21.2 compatibility
Angular 21.2 introduced a vitest-mock-patch plugin that blocks vi.mock() in Angular tests. Replace module-level mock with provideRouter([]) to support the real MessagesTable child component. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 6594d0e commit 52e1f18

1 file changed

Lines changed: 5 additions & 6 deletions

File tree

members/src/app/admin/messages/admin-messages.spec.ts

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,10 @@
1+
import { provideRouter } from '@angular/router';
12
import { render, screen } from '@testing-library/angular';
23
import { describe, expect, it, vi } from 'vitest';
34
import type { Message } from '../admin.types';
45
import { AdminMessagesService } from '../services/admin-messages.service';
56
import { AdminMessages } from './admin-messages';
67

7-
// Mock child component to isolate test
8-
vi.mock('./messages-table/messages-table', () => ({
9-
MessagesTable: class MockMessagesTable {},
10-
}));
11-
128
interface SetupOptions {
139
messages?: Message[];
1410
total?: number;
@@ -32,7 +28,10 @@ async function setup({
3228
};
3329

3430
const view = await render(AdminMessages, {
35-
providers: [{ provide: AdminMessagesService, useValue: mockAdminMessagesService }],
31+
providers: [
32+
{ provide: AdminMessagesService, useValue: mockAdminMessagesService },
33+
provideRouter([]),
34+
],
3635
});
3736

3837
return {

0 commit comments

Comments
 (0)