test: core module unit test coverage (#35) #36

Merged

xiaomo merged 1 commits from test/35-unit-test-coverage into main 2026-06-04 04:42:16 +00:00

Conversation 1 Commits 1 Files Changed 14

+873

xiaoju commented 2026-06-04 04:36:15 +00:00

Owner

Copy Link

Copy Source

What

补全核心模块单元测试，627 → 719 tests (+92)。

Why

开源前补齐高优模块测试覆盖。

Changes

• util: base32, result, refs-field, storage-root, log-tag (+49 tests)
• util-agent: storage normalizeWorkflowConfig, run parseArgv (+25 tests)
• agent-builtin: tools, session, detail (+24 tests)

Ref

Refs #35

## What 补全核心模块单元测试，627 → 719 tests (+92)。 ## Why 开源前补齐高优模块测试覆盖。 ## Changes - **util**: base32, result, refs-field, storage-root, log-tag (+49 tests) - **util-agent**: storage normalizeWorkflowConfig, run parseArgv (+25 tests) - **agent-builtin**: tools, session, detail (+24 tests) ## Ref Refs #35

xiaoju added 1 commit 2026-06-04 04:36:15 +00:00

test: add unit tests for core modules (#35) ... 06e959e7a5

Cover high-priority untested modules:
- util: base32, result, refs-field, storage-root, log-tag
- util-agent: storage (normalizeWorkflowConfig, resolveStorageRoot), run (parseArgv)
- agent-builtin: tools (read-file, write-file, run-command), session, detail

627 → 719 tests (+92), all passing.

Refs #35

xiaomo approved these changes 2026-06-04 04:42:14 +00:00

xiaomo left a comment

Owner

Copy Link

Copy Source

LGTM ✅ 测试覆盖扎实，边界用例、环境隔离、临时资源清理都到位。

LGTM ✅ 测试覆盖扎实，边界用例、环境隔离、临时资源清理都到位。

xiaomo merged commit 84bdd81317 into main 2026-06-04 04:42:16 +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

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: shazhou/united-workforce#36