fix(ci): fix admin-web test mocks + workflow build command
- Add getCurrentUserFromRequest + requireAdmin to all 8 auth-server test mocks - Fixes Vitest 4.x strict mock requiring all accessed exports - Skip pnpm install in CI workflow (deps already installed locally)
This commit is contained in:
parent
e427282234
commit
0ffbdce0ef
@ -20,11 +20,8 @@ jobs:
|
|||||||
- name: Pull latest
|
- name: Pull latest
|
||||||
run: git pull --ff-only origin main || true
|
run: git pull --ff-only origin main || true
|
||||||
|
|
||||||
- name: Install dependencies
|
|
||||||
run: pnpm install --frozen-lockfile
|
|
||||||
|
|
||||||
- name: Build all packages
|
- name: Build all packages
|
||||||
run: pnpm build
|
run: pnpm -r --filter './packages/**' build
|
||||||
|
|
||||||
- name: Typecheck
|
- name: Typecheck
|
||||||
run: pnpm typecheck
|
run: pnpm typecheck
|
||||||
|
|||||||
@ -13,6 +13,8 @@ import { describe, it, expect, vi, beforeEach } from 'vitest';
|
|||||||
const mockGetCurrentUser = vi.fn();
|
const mockGetCurrentUser = vi.fn();
|
||||||
vi.mock('@/lib/auth-server', () => ({
|
vi.mock('@/lib/auth-server', () => ({
|
||||||
getCurrentUser: (...args: unknown[]) => mockGetCurrentUser(...args),
|
getCurrentUser: (...args: unknown[]) => mockGetCurrentUser(...args),
|
||||||
|
getCurrentUserFromRequest: (...args: unknown[]) => mockGetCurrentUser(...args),
|
||||||
|
requireAdmin: (...args: unknown[]) => mockGetCurrentUser(...args),
|
||||||
}));
|
}));
|
||||||
|
|
||||||
const mockQueryAudit = vi.fn();
|
const mockQueryAudit = vi.fn();
|
||||||
|
|||||||
@ -12,6 +12,8 @@ import { describe, it, expect, vi, beforeEach } from 'vitest';
|
|||||||
const mockRequireAdmin = vi.fn();
|
const mockRequireAdmin = vi.fn();
|
||||||
vi.mock('@/lib/auth-server', () => ({
|
vi.mock('@/lib/auth-server', () => ({
|
||||||
requireAdmin: (...args: unknown[]) => mockRequireAdmin(...args),
|
requireAdmin: (...args: unknown[]) => mockRequireAdmin(...args),
|
||||||
|
getCurrentUser: (...args: unknown[]) => mockRequireAdmin(...args),
|
||||||
|
getCurrentUserFromRequest: (...args: unknown[]) => mockRequireAdmin(...args),
|
||||||
}));
|
}));
|
||||||
|
|
||||||
const mockCountActiveTokens = vi.fn();
|
const mockCountActiveTokens = vi.fn();
|
||||||
@ -55,7 +57,10 @@ describe('GET /api/dashboard/stats', () => {
|
|||||||
|
|
||||||
it('returns aggregated stats on success', async () => {
|
it('returns aggregated stats on success', async () => {
|
||||||
mockRequireAdmin.mockResolvedValue(admin);
|
mockRequireAdmin.mockResolvedValue(admin);
|
||||||
mockGetUserCounts.mockResolvedValue({ total: 150, byPlan: { free: 100, pro: 40, enterprise: 10 } });
|
mockGetUserCounts.mockResolvedValue({
|
||||||
|
total: 150,
|
||||||
|
byPlan: { free: 100, pro: 40, enterprise: 10 },
|
||||||
|
});
|
||||||
mockCountActiveTokens.mockResolvedValue(25);
|
mockCountActiveTokens.mockResolvedValue(25);
|
||||||
mockGetUsageSummary.mockResolvedValue({
|
mockGetUsageSummary.mockResolvedValue({
|
||||||
totalWords: 500000,
|
totalWords: 500000,
|
||||||
@ -95,7 +100,9 @@ describe('GET /api/dashboard/stats', () => {
|
|||||||
mockGetUserCounts.mockResolvedValue({ total: 0, byPlan: {} });
|
mockGetUserCounts.mockResolvedValue({ total: 0, byPlan: {} });
|
||||||
mockCountActiveTokens.mockResolvedValue(0);
|
mockCountActiveTokens.mockResolvedValue(0);
|
||||||
mockGetUsageSummary.mockResolvedValue({
|
mockGetUsageSummary.mockResolvedValue({
|
||||||
totalWords: 0, totalDictations: 0, totalCost: 0,
|
totalWords: 0,
|
||||||
|
totalDictations: 0,
|
||||||
|
totalCost: 0,
|
||||||
});
|
});
|
||||||
mockGetAuditStats.mockResolvedValue({ stats: {}, days: 90 });
|
mockGetAuditStats.mockResolvedValue({ stats: {}, days: 90 });
|
||||||
|
|
||||||
|
|||||||
@ -7,6 +7,8 @@ import { describe, it, expect, vi, beforeEach } from 'vitest';
|
|||||||
const mockRequireAdmin = vi.fn();
|
const mockRequireAdmin = vi.fn();
|
||||||
vi.mock('@/lib/auth-server', () => ({
|
vi.mock('@/lib/auth-server', () => ({
|
||||||
requireAdmin: (...args: unknown[]) => mockRequireAdmin(...args),
|
requireAdmin: (...args: unknown[]) => mockRequireAdmin(...args),
|
||||||
|
getCurrentUser: (...args: unknown[]) => mockRequireAdmin(...args),
|
||||||
|
getCurrentUserFromRequest: (...args: unknown[]) => mockRequireAdmin(...args),
|
||||||
}));
|
}));
|
||||||
|
|
||||||
const mockListFlags = vi.fn();
|
const mockListFlags = vi.fn();
|
||||||
|
|||||||
@ -23,6 +23,8 @@ import { describe, it, expect, vi, beforeEach } from 'vitest';
|
|||||||
const mockGetCurrentUser = vi.fn();
|
const mockGetCurrentUser = vi.fn();
|
||||||
vi.mock('@/lib/auth-server', () => ({
|
vi.mock('@/lib/auth-server', () => ({
|
||||||
getCurrentUser: (...args: unknown[]) => mockGetCurrentUser(...args),
|
getCurrentUser: (...args: unknown[]) => mockGetCurrentUser(...args),
|
||||||
|
getCurrentUserFromRequest: (...args: unknown[]) => mockGetCurrentUser(...args),
|
||||||
|
requireAdmin: (...args: unknown[]) => mockGetCurrentUser(...args),
|
||||||
}));
|
}));
|
||||||
|
|
||||||
const mockListInvitations = vi.fn();
|
const mockListInvitations = vi.fn();
|
||||||
|
|||||||
@ -7,6 +7,8 @@ import { describe, it, expect, vi, beforeEach } from 'vitest';
|
|||||||
const mockRequireAdmin = vi.fn();
|
const mockRequireAdmin = vi.fn();
|
||||||
vi.mock('@/lib/auth-server', () => ({
|
vi.mock('@/lib/auth-server', () => ({
|
||||||
requireAdmin: (...args: unknown[]) => mockRequireAdmin(...args),
|
requireAdmin: (...args: unknown[]) => mockRequireAdmin(...args),
|
||||||
|
getCurrentUser: (...args: unknown[]) => mockRequireAdmin(...args),
|
||||||
|
getCurrentUserFromRequest: (...args: unknown[]) => mockRequireAdmin(...args),
|
||||||
}));
|
}));
|
||||||
|
|
||||||
const mockGetUserLicenses = vi.fn();
|
const mockGetUserLicenses = vi.fn();
|
||||||
|
|||||||
@ -26,6 +26,8 @@ import { describe, it, expect, vi, beforeEach } from 'vitest';
|
|||||||
const mockGetCurrentUser = vi.fn();
|
const mockGetCurrentUser = vi.fn();
|
||||||
vi.mock('@/lib/auth-server', () => ({
|
vi.mock('@/lib/auth-server', () => ({
|
||||||
getCurrentUser: (...args: unknown[]) => mockGetCurrentUser(...args),
|
getCurrentUser: (...args: unknown[]) => mockGetCurrentUser(...args),
|
||||||
|
getCurrentUserFromRequest: (...args: unknown[]) => mockGetCurrentUser(...args),
|
||||||
|
requireAdmin: (...args: unknown[]) => mockGetCurrentUser(...args),
|
||||||
}));
|
}));
|
||||||
|
|
||||||
const mockListPromos = vi.fn();
|
const mockListPromos = vi.fn();
|
||||||
|
|||||||
@ -15,6 +15,8 @@ import { describe, it, expect, vi, beforeEach } from 'vitest';
|
|||||||
const mockGetCurrentUser = vi.fn();
|
const mockGetCurrentUser = vi.fn();
|
||||||
vi.mock('@/lib/auth-server', () => ({
|
vi.mock('@/lib/auth-server', () => ({
|
||||||
getCurrentUser: (...args: unknown[]) => mockGetCurrentUser(...args),
|
getCurrentUser: (...args: unknown[]) => mockGetCurrentUser(...args),
|
||||||
|
getCurrentUserFromRequest: (...args: unknown[]) => mockGetCurrentUser(...args),
|
||||||
|
requireAdmin: (...args: unknown[]) => mockGetCurrentUser(...args),
|
||||||
}));
|
}));
|
||||||
|
|
||||||
const mockListReferrals = vi.fn();
|
const mockListReferrals = vi.fn();
|
||||||
|
|||||||
@ -13,6 +13,8 @@ import { describe, it, expect, vi, beforeEach } from 'vitest';
|
|||||||
const mockRequireAdmin = vi.fn();
|
const mockRequireAdmin = vi.fn();
|
||||||
vi.mock('@/lib/auth-server', () => ({
|
vi.mock('@/lib/auth-server', () => ({
|
||||||
requireAdmin: (...args: unknown[]) => mockRequireAdmin(...args),
|
requireAdmin: (...args: unknown[]) => mockRequireAdmin(...args),
|
||||||
|
getCurrentUser: (...args: unknown[]) => mockRequireAdmin(...args),
|
||||||
|
getCurrentUserFromRequest: (...args: unknown[]) => mockRequireAdmin(...args),
|
||||||
}));
|
}));
|
||||||
|
|
||||||
const mockListUsers = vi.fn();
|
const mockListUsers = vi.fn();
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user