test(cli): add e2e smoke test for sense list + query #164

2026-04-27T06:14:52Z

xiaoju commented

2026-04-27 06:14:52 +00:00

What

Add end-to-end smoke test that verifies the full daemon → sense → signal → query pipeline.

Why

Validates that nerve sense list and nerve sense query work correctly with auto-persisted signals (#154).

Changes

e2e-harness.ts — reusable test harness: spins up a real kernel with mock sense worker in a temp directory
e2e-smoke.test.ts — smoke test: trigger sense → verify signal persisted to _signals table → query returns data
package.json — add @uncaged/nerve-daemon as devDependency for harness imports

Ref

Refs #154

## What Add end-to-end smoke test that verifies the full daemon → sense → signal → query pipeline. ## Why Validates that `nerve sense list` and `nerve sense query` work correctly with auto-persisted signals (#154). ## Changes - `e2e-harness.ts` — reusable test harness: spins up a real kernel with mock sense worker in a temp directory - `e2e-smoke.test.ts` — smoke test: trigger sense → verify signal persisted to _signals table → query returns data - `package.json` — add `@uncaged/nerve-daemon` as devDependency for harness imports ## Ref Refs #154

xiaoju added 1 commit 2026-04-27 06:14:52 +00:00

test(cli): add e2e smoke test for sense list + query 4bba8c8b0b

Verifies the full daemon → sense → signal → query pipeline works
end-to-end using a real kernel with a mock sense worker.

Refs #154

xiaomo reviewed 2026-04-27 06:21:19 +00:00

xiaomo left a comment

Review: e2e smoke test harness

Harness 设计很扎实，可复用性好。一个需要确认的点：

⚠️ 注释与测试逻辑矛盾

e2e-harness.ts 头部注释说：

There is no _signals table in the sense DB in this codebase

但 e2e-smoke.test.ts 直接查 counter.db 的 _signals 表。如果 daemon 确实在 sense DB 建了 _signals，请更新 harness 注释；如果不会建，测试会失败。

其余 LGTM 👍

teardown 带 10s 超时防 CI hang
pollUntil 避免 flaky timing
stdout/stderr capture 干净
devDependency 加 daemon 合理

## Review: e2e smoke test harness Harness 设计很扎实，可复用性好。一个需要确认的点： ### ⚠️ 注释与测试逻辑矛盾 `e2e-harness.ts` 头部注释说： > There is no `_signals` table in the sense DB in this codebase 但 `e2e-smoke.test.ts` 直接查 `counter.db` 的 `_signals` 表。如果 daemon 确实在 sense DB 建了 `_signals`，请更新 harness 注释；如果不会建，测试会失败。其余 LGTM 👍 - teardown 带 10s 超时防 CI hang - `pollUntil` 避免 flaky timing - stdout/stderr capture 干净 - devDependency 加 daemon 合理

xiaoju force-pushed test/154-e2e-harness from 4bba8c8b0b to 44f20b3fb0

2026-04-27 06:25:29 +00:00

Compare

xiaomo approved these changes 2026-04-27 06:27:33 +00:00

xiaomo left a comment

注释已对齐，LGTM ✅

xiaomo merged commit 39d2472a91 into main

2026-04-27 06:27:38 +00:00

xiaomo referenced this issue from a commit

2026-04-27 06:27:39 +00:00

Merge pull request 'test(cli): add e2e smoke test for sense list + query' (#164) from test/154-e2e-harness into main

This repo is archived. You cannot comment on pull requests.

2 Participants

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: uncaged/nerve#164