test : add unit tests for executor._row_value helper

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

还没有人认领这个 Issue。

  • #2253 来自 @tmdeveloper007 —— 已关闭,未合并

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
1/100
Issue 类型
文档
描述清晰度
描述清楚
活跃度
停滞
技术栈
python
领域
testing

调研方向

Open testing/backend/unit/test_executor_helpers.py and locate the executor helper tests; read _row_value() to confirm its behavior. Add tests for the listed dictionary, missing-key, None, non-dictionary, and key coercion cases, then run the relevant unit tests. The issue says tmdeveloper007 is handling the work, and a linked pull request exists, so check their status before starting.

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

描述

Summary of What Needs to be Done:
Add unit tests for executor._row_value() in testing/backend/unit/test_executor_helpers.py. This helper function is not directly tested despite being used in the task execution pipeline.

Changes that Need to be Made:
Add a new test class in testing/backend/unit/test_executor_helpers.py:

For _row_value(row, key, default=None):

  • returns the value when key exists in dict
  • returns default when key is absent
  • returns default when row is None
  • returns default when row is not a dict (e.g., None, list)
  • coercion: key as string works even if row has non-string keys (dict.get behavior)

Impact that it would Provide:

  • Ensures the task result parsing helpers handle malformed rows gracefully
  • Improves robustness of the scan result processing pipeline
  • Pure helper, fully mockable with no external dependencies

Note: This task is being handled by tmdeveloper007 — please assign to that account when picking it up.

主要语言
Python
星标
36
派生
234
PR 合并指标
30 天内没有已合并 PR

环境准备

从这里开始

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

utksh1/SecuScan 的其他 Issue

查看 utksh1/SecuScan 的全部 Issue

相似的 Issue

更多 Python Issue

把新 issue 发到你的邮箱

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