Skip to content

test: add runtime integration tests and parser AST snapshot tests#76

Merged
mostafa merged 1 commit into
mainfrom
fix/testing-gaps
May 6, 2026
Merged

test: add runtime integration tests and parser AST snapshot tests#76
mostafa merged 1 commit into
mainfrom
fix/testing-gaps

Conversation

@mostafa
Copy link
Copy Markdown
Member

@mostafa mostafa commented May 6, 2026

Summary

Depends on #71.

  • Add crates/rsigma-runtime/tests/integration.rs with three happy-path tests (single detection, no match, reload picks up new rules) exercising LogProcessor without Docker or feature flags
  • Add crates/rsigma-parser/tests/ast_snapshots.rs with five insta debug snapshot tests covering simple detection, complex condition, event count correlation, filter rule, and multi-value modifier rules

Test plan

  • All 3 runtime integration tests pass
  • All 5 parser snapshot tests pass and are stable across repeated runs
  • Snapshots committed under tests/snapshots/

@mostafa mostafa force-pushed the fix/security-critical branch from c78c56f to 3c8e980 Compare May 6, 2026 10:15
Base automatically changed from fix/security-critical to main May 6, 2026 10:44
@mostafa mostafa force-pushed the fix/testing-gaps branch from 2211dbf to 37fa5c2 Compare May 6, 2026 10:57
- Add crates/rsigma-runtime/tests/integration.rs with three happy-path
  tests (single detection, no match, reload picks up new rules) that
  exercise LogProcessor without Docker or feature flags
- Add crates/rsigma-parser/tests/ast_snapshots.rs with five insta
  snapshot tests covering simple detection, complex condition, event
  count correlation, filter rule, and multi-value modifier rules
@mostafa mostafa force-pushed the fix/testing-gaps branch from 37fa5c2 to 2864237 Compare May 6, 2026 11:14
@mostafa mostafa merged commit 97fda35 into main May 6, 2026
12 of 13 checks passed
@mostafa mostafa deleted the fix/testing-gaps branch May 6, 2026 11:20
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