IQSS / IQSS/dataverse-client-javascript

Fix branch test coverage in classes with 0% coverage and restore branch expected coverage back to 95%

未关闭
#186 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 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%

Screenshot 2024-09-03 at 12 22 19

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

调研方向

从 pre-commit hook 的分支覆盖率配置开始,检查 GetCollection 和 GetCollectionFacets 类及其现有测试用例。复现报告的 0% 分支覆盖率,确定这些分支未被计数的原因,并验证在将所需分支覆盖率恢复到 95% 后 hook 能够通过。

由索引模型根据 Issue 内容生成。

评估

技术栈
typescript
领域
testing-qa, tooling
Issue 类型
缺陷
难度
4/5
预计耗时
3-5 天
活跃度
停滞
描述清晰度
需要澄清
新手友好度
35/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。