IQSS / IQSS/dataverse-client-javascript
Fix branch test coverage in classes with 0% coverage and restore branch expected coverage back to 95%
未关闭
还没有人认领这个 Issue。
bug
GREI Re-arch
Size: 10
- 主要语言
- TypeScript
- 星标
- 20
- 派生
- 11
- 平均合并
- 2 天 5 小时
- 30 天内合并 PR
- 1
描述
The pre-commit hook was failing because it expected 95% branch coverage, but certain classes (GetCollection and GetCollectionFacets) were showing 0% branch coverage, even though we have test cases that should cover these branches.
To temporarily resolve this issue, we lowered the required branch coverage to 90% for the pre-commit hook to pass. However, we need to investigate the cause of this discrepancy, fix the issue, and restore the required branch coverage back to 95%
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
从 pre-commit hook 的分支覆盖率配置开始,检查 GetCollection 和 GetCollectionFacets 类及其现有测试用例。复现报告的 0% 分支覆盖率,确定这些分支未被计数的原因,并验证在将所需分支覆盖率恢复到 95% 后 hook 能够通过。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- typescript
- 领域
- testing-qa, tooling
- Issue 类型
- 缺陷
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 活跃度
- 停滞
- 描述清晰度
- 需要澄清
- 新手友好度
- 35/100