Make apps/api test fixtures cross-platform (Windows)
还没有人认领这个 Issue。
评估
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 新手友好度
- 78/100
- Issue 类型
- 缺陷
- 描述清晰度
- 基本清楚
- 活跃度
- 活跃
- 技术栈
- typescript
调研方向
从 apps/api tests/scrub-data.test.ts、internal-reload.test.ts 和 store.test.ts 开始,然后在 Windows 上分别独立运行每个文件,并与 develop 的结果进行比较。检查依赖 POSIX 特定路径或写入失败的 fixtures。完成标准是三个文件都能在 Windows 上通过,同时保留现有的 Linux 行为。
由索引模型根据 Issue 内容生成。
描述
Surfaced while closing out plans/site-check-153.md (PR #154).
On a Windows dev box the apps/api suite finishes 3 failed | 30 passed files, 10 failed | 413 passed tests, on a tree where git diff develop..HEAD -- apps/api packages/ is empty. The failures spread across scrub-data.test.ts (4), internal-reload.test.ts (4), and store.test.ts (2), and reproduce with each file run alone, so it is not runner contention. Checking out develop and re-running store.test.ts reproduces its 2 failures exactly.
Mechanism (at least for store.test.ts): the fixture injects a write failure by pointing the private store at /dev/null/impossible-path and asserting the transaction rejects. On Windows that is an ordinary creatable directory, so the write succeeds and the expected throw never happens. The other seven likely share a POSIX-ism of the same flavor.
CI runs the gate on Linux where the fixtures behave as intended, so this only blocks the documented local validation gate on Windows. Worth a cross-platform fixture cleanup (e.g. a path that cannot exist on any OS, or a mocked write failure) if Windows dev boxes are to be supported.
- 主要语言
- TypeScript
- 星标
- 1
- 派生
- 1
- 平均合并
- 1 天 20 小时
- 30 天内合并 PR
- 25
贡献指南
这个仓库没有索引到贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
CodeForPhilly/codeforphilly-ng 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 84/100
-
MarkdownEditor toolbar: use Radix Toolbar from radix-ui instead of the hand-rolled roving tabindex 未关闭enhancement
难度 2/5 1-3 小时 新手友好度 82/100
-
难度 2/5 1-3 小时 新手友好度 76/100
-
难度 2/5 1-3 小时 新手友好度 88/100
-
难度 2/5 1-3 小时 新手友好度 74/100
查看 CodeForPhilly/codeforphilly-ng 的全部 Issue
相似的 Issue
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs 未关闭
难度 2/5 1-3 小时 新手友好度 70/100
-
Crush 未关闭
难度 1/5 1 小时以内 新手友好度 85/100
catppuccin/catppuccin#3125 ·
-
难度 1/5 1 小时以内 新手友好度 90/100
ElementsProject/cln-application#167 · 1 条评论 · 1 个 reaction ·
-
难度 2/5 1-3 小时 新手友好度 75/100
Quantco/pnpm-licenses#17 ·
-
难度 2/5 1-3 小时 新手友好度 75/100