test : add unit tests for _serialize_notification_history helper

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

@aaniya22 已经在做这个了。

开始于 2026年8月16日。

评估

这个 Issue 还没有评估数据。

描述

Summary of What Needs to be Done:
Add unit tests for the _serialize_notification_history function in backend/secuscan/routes.py. This function transforms a raw database row dict into the API response shape for notification history records.

Changes that Need to be Made:

  • Create testing/backend/unit/test_routes_notification_history_serializer.py
  • Test cases should cover:
    • Complete row with all fields returns correct shape
    • Missing error_message field uses .get() default (None)
    • Missing sent_at field uses .get() default (None)
    • is_active field correctly converted from int to bool

Impact that it would Provide:

  • Ensures notification history API responses have consistent shape
  • Catches bugs when the schema changes
  • Documents expected response fields

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 摘要。