shazhou/united-workforce
8
0
Fork 0
Code Issues 8 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

fix: OCAS_DIR → OCAS_HOME in test helpers #59

Merged
xiaomo merged 6 commits from fix/58-test-isolation into main 2026-06-04 12:16:21 +00:00
Conversation 1 Commits 6 Files Changed 6
+37 -26
xiaoju commented 2026-06-04 11:00:02 +00:00
Owner
Copy Link
Copy Source

4 remaining OCAS_DIR references in test helpers (store-unified-threads, thread-resume) caused tests to write to global ~/.ocas instead of per-test temp dirs, resulting in 6 test failures from data leakage.

All 16 affected tests now pass.

Fixes #58

4 remaining `OCAS_DIR` references in test helpers (store-unified-threads, thread-resume) caused tests to write to global `~/.ocas` instead of per-test temp dirs, resulting in 6 test failures from data leakage. All 16 affected tests now pass. Fixes #58
xiaoju force-pushed fix/58-test-isolation from d8b6974379 to a8734f29e1 2026-06-04 11:00:44 +00:00 Compare
xiaoju added 1 commit 2026-06-04 11:06:46 +00:00
fix: OCAS_DIR → OCAS_HOME in test helpers + exclude integration tests from CI
CI / check (pull_request) Failing after 2m27s
Details
63454a4cfd 
- Remaining OCAS_DIR references caused test isolation failures
- agent-hermes integration tests need 'hermes' CLI, skip in CI

Fixes #58
xiaoju force-pushed fix/58-test-isolation from a8734f29e1 to 63454a4cfd 2026-06-04 11:06:46 +00:00 Compare
xiaoju added 1 commit 2026-06-04 11:18:51 +00:00
fix: add timeout to CI-slow tests + check stderr for help output
CI / check (pull_request) Failing after 1m55s
Details
883bd79bcb
xiaoju added 1 commit 2026-06-04 11:22:48 +00:00
fix: biome format + remove unused noConsole suppressions
CI / check (pull_request) Failing after 3m58s
Details
d26f54e8ea
xiaoju added 1 commit 2026-06-04 11:34:30 +00:00
fix: use node dist/cli.js instead of npx tsx in prompt help test
CI / check (pull_request) Failing after 3m40s
Details
596c05bfcc 
npx tsx fails in CI (tsx not found, npm tries to install it)
xiaoju added 1 commit 2026-06-04 11:51:07 +00:00
fix: add timeout to cmdThreadExec count logic tests
CI / check (pull_request) Failing after 4m17s
Details
58b58d511e
xiaoju added 1 commit 2026-06-04 11:59:48 +00:00
fix: use node dist/cli.js instead of npx tsx in thread-step-count tests
CI / check (pull_request) Successful in 3m30s
Details
66c2e2a79b 
npx tsx hangs in CI Docker (30s+ timeout). node dist/cli.js runs in <2s.
xiaomo approved these changes 2026-06-04 12:16:20 +00:00
xiaomo left a comment
Owner
Copy Link
Copy Source

LGTM ✅ 环境变量漏网修复 + E2E timeout + tsx→node，测试隔离到位。

LGTM ✅ 环境变量漏网修复 + E2E timeout + tsx→node，测试隔离到位。
xiaomo merged commit 9fbdd1dd2c into main 2026-06-04 12:16:21 +00:00
xiaomo deleted branch fix/58-test-isolation 2026-06-04 12:16:21 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
xiaomo
Labels
Clear labels
bug
Something is not working
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
aobing luming scottwei tuanzi xiaoju xiaomo xiaonuo xingyue
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: shazhou/united-workforce#59
Delete Branch "fix/58-test-isolation"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?