Skip to content

PRs #108 and #109: vitest.config.ts missing isolate: true — inconsistent with #106 and #107 #125

@bmersereau

Description

@bmersereau

Problem

PRs #106 and #107 set isolate: true in backend/vitest.config.ts to ensure each test file runs in its own module registry. PRs #108 and #109 added vitest configs without this flag, relying instead on inline vi.resetModules() calls per test.

This inconsistency means module state could leak between test files in PRs #108/#109 if tests are added in the future that don't call vi.resetModules() manually.

Fix

Add isolate: true to backend/vitest.config.ts in both the fix/101-102-storage-improvements and fix/103-auth-client-singleton branches.

Affected

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions