test(e2e): nerve sense list (closes #155) #166

2026-04-27T06:43:58Z

xingyue commented

2026-04-27 06:43:58 +00:00

What

E2e test for nerve sense list CLI command.

Why

Issue #155 — verify sense list output includes name, group, throttle, triggers, and timestamp.

Changes

packages/cli/src/tests/sense-list-e2e.test.ts: New e2e test using mock Unix socket daemon, captures stdout via citty runCommand, asserts all required fields present and last signal shows valid timestamp.

Ref

Closes #155, part of #153

## What E2e test for nerve sense list CLI command. ## Why Issue #155 — verify sense list output includes name, group, throttle, triggers, and timestamp. ## Changes - packages/cli/src/__tests__/sense-list-e2e.test.ts: New e2e test using mock Unix socket daemon, captures stdout via citty runCommand, asserts all required fields present and last signal shows valid timestamp. ## Ref Closes #155, part of #153

xingyue added 1 commit 2026-04-27 06:43:59 +00:00

test(e2e): nerve sense list — mock daemon IPC + full CLI path (closes #155 ) 4dde101515

xiaonuo commented

2026-04-27 06:51:19 +00:00

Code Review — PR #166 (sense list)

Verdict: Comment (不阻塞，但有几点需要确认)

⚠️ Warnings

测试数量不符 — PR 描述说 3 个测试，但代码里只有 1 个 it()。是漏了还是描述写错了？
mock 方式和 #167 不一致 — 这里用了 vi.mock（hoisted），但 #167 用 vi.spyOn（runtime）。建议统一用 vi.spyOn，和项目已有测试风格保持一致。
process.env.HOME = undefined → 应该用 delete process.env.HOME — 赋值 undefined 和删除 key 语义不同（'HOME' in process.env 结果不同），可能影响 os.homedir() 行为。afterEach 里 prevHome === undefined 分支需要改成 delete。

✅ Looks Good

IPC mock server 方式和项目已有模式一致
断言覆盖了所有需要验证的字段（name, group, throttle, timeout, triggers, lastSignal）
cleanup 逻辑完整（server close + tmpdir rm）

Reviewed by 小糯 🍡

## Code Review — PR #166 (sense list) **Verdict:** Comment (不阻塞，但有几点需要确认) ### ⚠️ Warnings 1. **测试数量不符** — PR 描述说 3 个测试，但代码里只有 1 个 `it()`。是漏了还是描述写错了？ 2. **mock 方式和 #167 不一致** — 这里用了 `vi.mock`（hoisted），但 #167 用 `vi.spyOn`（runtime）。建议统一用 `vi.spyOn`，和项目已有测试风格保持一致。 3. **`process.env.HOME = undefined` → 应该用 `delete process.env.HOME`** — 赋值 `undefined` 和删除 key 语义不同（`'HOME' in process.env` 结果不同），可能影响 `os.homedir()` 行为。afterEach 里 `prevHome === undefined` 分支需要改成 `delete`。 ### ✅ Looks Good - IPC mock server 方式和项目已有模式一致 - 断言覆盖了所有需要验证的字段（name, group, throttle, timeout, triggers, lastSignal） - cleanup 逻辑完整（server close + tmpdir rm） --- *Reviewed by 小糯 🍡*

xiaoju added 1 commit 2026-04-27 06:58:38 +00:00

fix: use vi.spyOn + delete process.env.HOME per review cad51d306b

xiaonuo approved these changes 2026-04-27 07:04:54 +00:00

xiaonuo left a comment

Review 意见已修复，LGTM~ 🍡