Deleting the human view of `daemon status` leaves the whole suite green
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 68/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 活跃
- 技术栈
- typescript
- 领域
- cli, testing-qa
调研方向
未测试的分支是 src/cli/index.ts 第 1667-1668 行附近的人类可读输出，其中在未设置 --json 时，daemon status 会写入 formatStatus(...)。请阅读 src/cli/index.test.ts 中现有的 daemon start 和 status 断言，使新测试与其风格保持一致，然后添加一个单元测试：运行不带 --json 的 daemon status，并期望输出 Daemon: running (worker)。当该测试在当前代码上通过，并且在将人类可读分支替换为无条件的 writeResult 后失败时，即完成。请查看 #422 中提出的修正版 it，因为这项工作可能会有重叠。
由索引模型根据 Issue 内容生成。
描述
Testing rule 3: a code path that can be deleted with a green suite is untested.
I deleted the human branch of simlock daemon status in src/cli/index.ts:1667-1668 — turning
if (values.json) writeResult(environment, status);
else environment.stdout.write(`${formatStatus(status, environment.clock.now())}\n`);
into an unconditional writeResult(environment, status) — and every check stayed green:
pnpm exec vitest run --project unit src/cli/index.test.ts: 219 passed. That suite assertsDaemon: running (worker)forsimlock statusandDaemon runningforsimlock daemon start, so onlydaemon status's own human view has no unit assertion.pnpm test:e2e e2e/daemon-lifecycle.test.ts: 3 passed, 1 expected fail. The one test that namesdaemon status's default view is anit.fails, which passes whether the view exists or not.pnpm test:changed: green apart from the repro test from #422.
So the CLI publishes a human view of daemon status that no test can see. The corrected it proposed in #422 (asserting Daemon: running (worker) on daemon status with no --json) does cover it: run against this same deletion it fails with expected '{"devices":[]…' to contain 'Daemon: running (worker)'.
Found while triaging #422.
Written by an agent.
- 主要语言
- TypeScript
- 星标
- 15
- 派生
- 1
- 平均合并
- 9 小时 6 分钟
- 30 天内合并 PR
- 138
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南，环境需要你自己搭建：先看它的 README，通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
callstackincubator/simlock 的其他 Issue
-
bug:new
难度 1/5 1 小时以内 新手友好度 70/100
callstackincubator/simlock#430 ·
维护者通常 1 天内回复
-
bug:new
难度 2/5 1-3 小时 新手友好度 78/100
callstackincubator/simlock#424 ·
维护者通常 1 天内回复
-
daemon-lifecycle e2e: the it.fails status test passes for the wrong reason可能已有人在做 @V3RON 于 1 天前认领。 未关闭bug:triage
难度 2/5 1-3 小时 新手友好度 84/100
callstackincubator/simlock#422 · 已指派 1 人 ·
维护者通常 1 天内回复
-
bug:new
难度 1/5 1 小时以内 新手友好度 88/100
callstackincubator/simlock#420 ·
维护者通常 1 天内回复
-
bug:new
难度 2/5 1-3 小时 新手友好度 76/100
callstackincubator/simlock#350 · 1 条评论 ·
维护者通常 1 天内回复
查看 callstackincubator/simlock 的全部 Issue
相似的 Issue
-
[Bug]: [MCP/CLI] Bare loopback IP addresses (127.0.0.1:port) and hosts with ports fail to navigate due to erroneous scheme inference可能已有人在做 @alok-108 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 78/100
microsoft/playwright#43263 ·
维护者通常 1 天内回复
-
bug priority:medium
难度 2/5 1-3 小时 新手友好度 65/100
维护者通常 1 天内回复
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
难度 1/5 1 小时以内 新手友好度 75/100
lingdojo/kana-dojo#32018 · 1 条评论 · 5 个 reaction ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100
paperclipai/paperclip#15751 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100
BuilderIO/agent-native#7275 ·
维护者通常 1 天内回复