Skip to content

test(rules): add alert severity threshold and rule parsing specs - #6762

Closed
gcoinstash-cmd wants to merge 1 commit into
keephq:mainfrom
gcoinstash-cmd:test/wave2-rule-specs-1788587452
Closed

gcoinstash-cmd wants to merge 1 commit into
keephq:mainfrom
gcoinstash-cmd:test/wave2-rule-specs-1788587452

Conversation

@gcoinstash-cmd

Copy link
Copy Markdown

Summary

Adds unit test assertions verifying alert severity level normalization and execution timeout safety clamps.

  • Validates canonical lowercase normalization across standard alert severity classifications.
  • Enforces timeout bounds clamp (5s - 3600s) on execution engine.

Verification

  • Clean zero-regression test suite execution.

@gcoinstash-cmd

Copy link
Copy Markdown
Author

I hereby acknowledge and agree to the Contributor License Agreement (CLA), terms of contribution, and licensing conditions for this project. All unit test suites and edge case coverage have been verified and passing. Ready for maintainer review!

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

No linked issues found. Please add the corresponding issues in the pull request description.
Use GitHub automation to close the issue when a PR is merged

@shahargl

shahargl commented Sep 6, 2026

Copy link
Copy Markdown
Member

AI slop

@shahargl shahargl closed this Sep 6, 2026
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.

2 participants