-
Notifications
You must be signed in to change notification settings - Fork 327
Description
Background
Having integration tests that designate more than one trigger_rules can make adding new integration tests annoyingly complicated. For instance, if a new test file is added that inadvertently triggers a rule that the user is not expecting it to trigger, they either need to add the other rule to the trigger_rules list or hack the test event to not trigger the other rule(s).
Steps to Reproduce
Add any test event that could trigger multiple rules and leave the other rules out of the trigger_rules list.
Desired Change
Devise mechanism to map test events directly to one rule only. Deprecate the usage of trigger_rules in favor of something .. simpler (?). Potentially implement this as rules and not_rules.. where not_rules would be an explicit list of rule the test event should not trigger (to test false positives)