Skip to content

Add test coverage for _serialize_notification_rule#1692

Open
tmdeveloper007 wants to merge 1 commit into
utksh1:mainfrom
tmdeveloper007:#1692
Open

Add test coverage for _serialize_notification_rule#1692
tmdeveloper007 wants to merge 1 commit into
utksh1:mainfrom
tmdeveloper007:#1692

Conversation

@tmdeveloper007

Copy link
Copy Markdown
Contributor

Problem

_serialize_notification_rule in backend/secuscan/routes.py serializes a DB row into a notification rule dict. No direct unit tests exist for it.

Fix

  1. Extract _serialize_notification_rule into backend/secuscan/routes_notification_helpers.py.
  2. Write unit tests covering all serialized fields, missing optional fields, and boolean coercion for is_active.

Acceptance Criteria

  • New routes_notification_helpers.py contains the extracted function.
  • Tests pass locally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant