Skip to content

test(missing): added missing tests for azure query "activity_log_alert_for_service_health_not_configured"#7897

Merged
cx-vasco-oliveira merged 1 commit into
AST-116647_40_6.1.2.11_ensure_activity_log_alert_for_service_healthfrom
AST-116647_40_6.1.2.11_ensure_activity_log_alert_for_service_health-QA-Tests
Dec 4, 2025
Merged

test(missing): added missing tests for azure query "activity_log_alert_for_service_health_not_configured"#7897
cx-vasco-oliveira merged 1 commit into
AST-116647_40_6.1.2.11_ensure_activity_log_alert_for_service_healthfrom
AST-116647_40_6.1.2.11_ensure_activity_log_alert_for_service_health-QA-Tests

Conversation

@cx-vasco-oliveira

Copy link
Copy Markdown
Contributor

Reason for Proposed Changes

  • The tests added for this query did not cover all detection flows.

Proposed Changes

  • Added a test with multiple values inside the events field.
  • Added a test for a resource with a different scope other then subscription.
  • Added a test with just one subscription, to see if it detects the fact that's not being used even when there is no resource on the folder.
  • Removed wrong comments inside the positive2 test.

I submit this contribution under the Apache-2.0 license.

@cx-vasco-oliveira cx-vasco-oliveira requested a review from a team as a code owner December 4, 2025 16:05
@cx-vasco-oliveira cx-vasco-oliveira merged commit 0c9bbfd into AST-116647_40_6.1.2.11_ensure_activity_log_alert_for_service_health Dec 4, 2025
2 of 9 checks passed
@cx-vasco-oliveira cx-vasco-oliveira deleted the AST-116647_40_6.1.2.11_ensure_activity_log_alert_for_service_health-QA-Tests branch December 4, 2025 16:05
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