Remove every as any from the test suite and enforce it
ESLint turned `no-explicit-any` off for test files, and 219 casts had accumulated across 12 of them. Two shapes, very different risks: most were `(useWorkspace as any).mockReturnValue(...)`, mere laziness that `vi.mocked()` does properly; the rest cast partial fixtures onto API types, which erases the contract exactly where a test should enforce it. That second kind matters here specifically. This project's recurring bug is a field silently dropped between backend and frontend -- the hand-written mapping in models/plan.py has done it twice, and apiFetch mishandling structured errors was the same shape. A test that casts an incomplete fixture to `any` keeps passing through precisely that change. A typed fixture layer already existed at tests/fixtures/mockData.ts and was imported by exactly zero tests: the right tool was there and invisible, so everyone reached for a cast. It is now a set of `make*(overrides: Partial<T>) => T` factories returning complete, valid objects, split by domain with a barrel. When a type gains a field, one factory fails to compile instead of thirty tests silently passing. With the count at zero the ESLint exemption is deleted, so this cannot regrow. Removing it immediately caught two `}: any` prop stubs that a grep for "as any" had missed; those mocks now use the real components' exported prop types. `as unknown as T` survives in five places, each a test deliberately feeding invalid input (an undefined breakdown, missing tierData, a bogus leave type). CLAUDE.md now documents that as the only accepted escape, and points at the factories. interface PollResponse in useScraperFetch.ts is exported so its shape can be derived rather than duplicated -- pollScraperSession declares Promise<unknown>, so there was no other honest way to type it. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01RXMhc9nT6sNz1SL4j8dznL
This commit is contained in:
@@ -6,6 +6,15 @@ import { useWorkspace } from '@/context'
|
||||
import * as api from '@/services/api'
|
||||
import { ApiRequestError } from '@/services/errors'
|
||||
import { LanguageProvider } from '@/i18n'
|
||||
import {
|
||||
makeWorkspaceContextValue,
|
||||
makeWorkspaceMembersResult,
|
||||
makeVacationDaysResult,
|
||||
makeShowWeekNumbersTuple,
|
||||
makeChildSettingsResult,
|
||||
makeDayOverridesResult,
|
||||
makeCalendarDaysResult,
|
||||
} from '../../tests/fixtures'
|
||||
|
||||
vi.mock('@/hooks')
|
||||
vi.mock('@/context')
|
||||
@@ -39,26 +48,6 @@ const CURRENT_YEAR = new Date().getFullYear()
|
||||
const CHILD_BIRTH_DATE = `${CURRENT_YEAR - 1}-01-01`
|
||||
|
||||
describe('Settings', () => {
|
||||
const mockWorkspaceContext = {
|
||||
workspace: {
|
||||
id: 'ws-1',
|
||||
name: 'Test Workspace',
|
||||
planningGoal: 'balanced' as const,
|
||||
planningMode: 'solo' as const,
|
||||
createdAt: '2026-01-01',
|
||||
},
|
||||
isLoading: false,
|
||||
error: null,
|
||||
workspaceId: 'ws-1',
|
||||
setWorkspaceId: vi.fn(),
|
||||
refetch: vi.fn(),
|
||||
children: [{ id: 'c-1', name: 'Child 1', birth_date: CHILD_BIRTH_DATE }],
|
||||
selectedChildId: 'c-1',
|
||||
selectChild: vi.fn(),
|
||||
personalDataUuid: 'pd-1',
|
||||
parentName: 'Test Parent',
|
||||
refreshContext: vi.fn(),
|
||||
}
|
||||
|
||||
const mockVacationDays = [
|
||||
{
|
||||
@@ -123,49 +112,96 @@ describe('Settings', () => {
|
||||
mockRemoveDayOverride = vi.fn().mockResolvedValue(undefined)
|
||||
mockRefreshMembers = vi.fn().mockResolvedValue(undefined)
|
||||
|
||||
vi.mocked(useWorkspace).mockReturnValue(mockWorkspaceContext as any)
|
||||
vi.mocked(useWorkspace).mockReturnValue(
|
||||
makeWorkspaceContextValue({
|
||||
workspace: {
|
||||
id: 'ws-1',
|
||||
name: 'Test Workspace',
|
||||
planningGoal: 'balanced',
|
||||
planningMode: 'solo',
|
||||
createdAt: '2026-01-01',
|
||||
},
|
||||
workspaceId: 'ws-1',
|
||||
children: [{ id: 'c-1', name: 'Child 1', birth_date: CHILD_BIRTH_DATE }],
|
||||
selectedChildId: 'c-1',
|
||||
personalDataUuid: 'pd-1',
|
||||
parentName: 'Test Parent',
|
||||
})
|
||||
)
|
||||
// Default: solo workspace (only the logged-in user) — the add-partner
|
||||
// form should be visible. Tests that need a linked partner override this.
|
||||
vi.mocked(hooks.useWorkspaceMembers).mockReturnValue({
|
||||
data: [selfMember],
|
||||
isLoading: false,
|
||||
error: null,
|
||||
refresh: mockRefreshMembers,
|
||||
} as any)
|
||||
vi.mocked(hooks.useVacationDays).mockReturnValue({
|
||||
data: mockVacationDays,
|
||||
isLoading: false,
|
||||
error: null,
|
||||
upsert: mockUpsertVacationDays,
|
||||
} as any)
|
||||
vi.mocked(hooks.useShowWeekNumbers).mockReturnValue([false, vi.fn()] as any)
|
||||
vi.mocked(hooks.useChildSettings).mockReturnValue({
|
||||
data: mockChildDetails,
|
||||
isLoading: false,
|
||||
error: null,
|
||||
update: mockUpdateChildDetails,
|
||||
} as any)
|
||||
vi.mocked(hooks.useDayOverrides).mockReturnValue({
|
||||
data: [],
|
||||
isLoading: false,
|
||||
error: null,
|
||||
upsert: mockUpsertDayOverride,
|
||||
remove: mockRemoveDayOverride,
|
||||
} as any)
|
||||
vi.mocked(hooks.useCalendarDays).mockReturnValue({
|
||||
data: mockCalendarWindow,
|
||||
isLoading: false,
|
||||
error: null,
|
||||
} as any)
|
||||
vi.mocked(api.updateWorkspace).mockResolvedValue({ ...mockWorkspaceContext.workspace } as any)
|
||||
vi.mocked(api.updateParentSettings).mockResolvedValue({} as any)
|
||||
vi.mocked(api.addPartner).mockResolvedValue({} as any)
|
||||
vi.mocked(api.renameWorkspaceParent).mockResolvedValue({ ...partnerMember } as any)
|
||||
vi.mocked(hooks.useWorkspaceMembers).mockReturnValue(
|
||||
makeWorkspaceMembersResult({
|
||||
data: [selfMember],
|
||||
refresh: mockRefreshMembers,
|
||||
})
|
||||
)
|
||||
vi.mocked(hooks.useVacationDays).mockReturnValue(
|
||||
makeVacationDaysResult({
|
||||
data: mockVacationDays,
|
||||
upsert: mockUpsertVacationDays,
|
||||
})
|
||||
)
|
||||
vi.mocked(hooks.useShowWeekNumbers).mockReturnValue(makeShowWeekNumbersTuple(false, vi.fn()))
|
||||
vi.mocked(hooks.useChildSettings).mockReturnValue(
|
||||
makeChildSettingsResult({
|
||||
data: mockChildDetails,
|
||||
update: mockUpdateChildDetails,
|
||||
})
|
||||
)
|
||||
vi.mocked(hooks.useDayOverrides).mockReturnValue(
|
||||
makeDayOverridesResult({
|
||||
data: [],
|
||||
upsert: mockUpsertDayOverride,
|
||||
remove: mockRemoveDayOverride,
|
||||
})
|
||||
)
|
||||
vi.mocked(hooks.useCalendarDays).mockReturnValue(
|
||||
makeCalendarDaysResult({
|
||||
data: mockCalendarWindow,
|
||||
})
|
||||
)
|
||||
vi.mocked(api.updateWorkspace).mockResolvedValue({
|
||||
id: 'ws-1',
|
||||
name: 'Test Workspace',
|
||||
planningGoal: 'balanced',
|
||||
planningMode: 'solo',
|
||||
createdAt: '2026-01-01',
|
||||
})
|
||||
vi.mocked(api.updateParentSettings).mockResolvedValue({
|
||||
uuid: 'pd-1',
|
||||
name: 'Test Parent',
|
||||
salary_type: 'monthly',
|
||||
employment_status: 'employed',
|
||||
})
|
||||
vi.mocked(api.addPartner).mockResolvedValue({
|
||||
uuid: 'pd-2',
|
||||
name: 'Sam',
|
||||
salary_type: 'monthly',
|
||||
employment_status: 'employed',
|
||||
})
|
||||
vi.mocked(api.renameWorkspaceParent).mockResolvedValue(partnerMember)
|
||||
vi.mocked(api.removeWorkspaceParent).mockResolvedValue(undefined)
|
||||
})
|
||||
|
||||
it('renders loading spinner while workspace loads', () => {
|
||||
vi.mocked(useWorkspace).mockReturnValue({ ...mockWorkspaceContext, isLoading: true } as any)
|
||||
vi.mocked(useWorkspace).mockReturnValue(
|
||||
makeWorkspaceContextValue({
|
||||
workspace: {
|
||||
id: 'ws-1',
|
||||
name: 'Test Workspace',
|
||||
planningGoal: 'balanced',
|
||||
planningMode: 'solo',
|
||||
createdAt: '2026-01-01',
|
||||
},
|
||||
workspaceId: 'ws-1',
|
||||
children: [{ id: 'c-1', name: 'Child 1', birth_date: CHILD_BIRTH_DATE }],
|
||||
selectedChildId: 'c-1',
|
||||
personalDataUuid: 'pd-1',
|
||||
parentName: 'Test Parent',
|
||||
isLoading: true,
|
||||
})
|
||||
)
|
||||
renderSettings()
|
||||
expect(screen.getByText(/Laddar inställningar/)).toBeInTheDocument()
|
||||
})
|
||||
@@ -326,12 +362,12 @@ describe('Settings', () => {
|
||||
|
||||
describe('workspace members', () => {
|
||||
beforeEach(() => {
|
||||
vi.mocked(hooks.useWorkspaceMembers).mockReturnValue({
|
||||
data: [selfMember, partnerMember],
|
||||
isLoading: false,
|
||||
error: null,
|
||||
refresh: mockRefreshMembers,
|
||||
} as any)
|
||||
vi.mocked(hooks.useWorkspaceMembers).mockReturnValue(
|
||||
makeWorkspaceMembersResult({
|
||||
data: [selfMember, partnerMember],
|
||||
refresh: mockRefreshMembers,
|
||||
})
|
||||
)
|
||||
})
|
||||
|
||||
it('shows the linked partner and hides the add-partner form', () => {
|
||||
@@ -469,7 +505,7 @@ describe('Settings', () => {
|
||||
|
||||
it('toggles week numbers', () => {
|
||||
const setShowWeekNumbers = vi.fn()
|
||||
vi.mocked(hooks.useShowWeekNumbers).mockReturnValue([false, setShowWeekNumbers] as any)
|
||||
vi.mocked(hooks.useShowWeekNumbers).mockReturnValue(makeShowWeekNumbersTuple(false, setShowWeekNumbers))
|
||||
renderSettings()
|
||||
fireEvent.click(screen.getByTestId('show-week-numbers-checkbox'))
|
||||
expect(setShowWeekNumbers).toHaveBeenCalledWith(true)
|
||||
@@ -553,26 +589,26 @@ describe('Settings', () => {
|
||||
|
||||
describe('red days list (§3.8)', () => {
|
||||
it('passes the fetched calendar window through to the list, four years at a time', () => {
|
||||
vi.mocked(hooks.useCalendarDays).mockReturnValue({
|
||||
data: {
|
||||
startYear: CURRENT_YEAR,
|
||||
endYear: CURRENT_YEAR + 3,
|
||||
minYear: 2026,
|
||||
maxYear: 2035,
|
||||
days: [
|
||||
{
|
||||
date: `${CURRENT_YEAR}-01-01`,
|
||||
name: 'Nyårsdagen',
|
||||
dayType: 'red_day',
|
||||
isOverride: false,
|
||||
overrideIsWorkingDay: null,
|
||||
isFree: true,
|
||||
},
|
||||
],
|
||||
},
|
||||
isLoading: false,
|
||||
error: null,
|
||||
} as any)
|
||||
vi.mocked(hooks.useCalendarDays).mockReturnValue(
|
||||
makeCalendarDaysResult({
|
||||
data: {
|
||||
startYear: CURRENT_YEAR,
|
||||
endYear: CURRENT_YEAR + 3,
|
||||
minYear: 2026,
|
||||
maxYear: 2035,
|
||||
days: [
|
||||
{
|
||||
date: `${CURRENT_YEAR}-01-01`,
|
||||
name: 'Nyårsdagen',
|
||||
dayType: 'red_day',
|
||||
isOverride: false,
|
||||
overrideIsWorkingDay: null,
|
||||
isFree: true,
|
||||
},
|
||||
],
|
||||
},
|
||||
})
|
||||
)
|
||||
|
||||
renderSettings()
|
||||
|
||||
@@ -590,11 +626,11 @@ describe('Settings', () => {
|
||||
|
||||
it('requests the previous 4-year window when paginating backward', () => {
|
||||
// minYear well below the current window so the "previous" button isn't disabled.
|
||||
vi.mocked(hooks.useCalendarDays).mockReturnValue({
|
||||
data: { ...mockCalendarWindow, minYear: 2000 },
|
||||
isLoading: false,
|
||||
error: null,
|
||||
} as any)
|
||||
vi.mocked(hooks.useCalendarDays).mockReturnValue(
|
||||
makeCalendarDaysResult({
|
||||
data: { ...mockCalendarWindow, minYear: 2000 },
|
||||
})
|
||||
)
|
||||
|
||||
renderSettings()
|
||||
|
||||
|
||||
Reference in New Issue
Block a user