fix: test isolation — store data leaks between test files #58

New Issue

2026-06-04T10:57:05Z

xiaoju commented

2026-06-04 10:57:05 +00:00

Problem

6 tests fail because they expect specific thread counts but the shared store accumulates data from other test files:

store-unified-threads.test.ts — loadActiveThreads returns 75 instead of 1
thread-resume.test.ts — completed thread resume affected by stale data

Root Cause

Tests share the same UWF_HOME / OCAS_HOME storage directory across files. Each test file creates threads without cleaning up.

Fix Options

Use unique temp dirs per test file (preferred)
Add beforeEach cleanup
Assert with toContain instead of exact counts

— 小橘 🍊（NEKO Team）

## Problem 6 tests fail because they expect specific thread counts but the shared store accumulates data from other test files: - `store-unified-threads.test.ts` — `loadActiveThreads` returns 75 instead of 1 - `thread-resume.test.ts` — completed thread resume affected by stale data ## Root Cause Tests share the same `UWF_HOME` / `OCAS_HOME` storage directory across files. Each test file creates threads without cleaning up. ## Fix Options 1. Use unique temp dirs per test file (preferred) 2. Add beforeEach cleanup 3. Assert with `toContain` instead of exact counts — 小橘 🍊（NEKO Team）

xiaoju referenced this issue from a commit

2026-06-04 10:59:56 +00:00

fix: OCAS_DIR → OCAS_HOME in test helpers

xiaoju referenced a pull request that will close this issue

2026-06-04 11:00:02 +00:00

fix: OCAS_DIR → OCAS_HOME in test helpers #59

xiaoju referenced this issue from a commit

2026-06-04 11:00:44 +00:00

fix: OCAS_DIR → OCAS_HOME in test helpers

xiaoju referenced this issue from a commit

2026-06-04 11:06:46 +00:00

fix: OCAS_DIR → OCAS_HOME in test helpers + exclude integration tests from CI