1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798991001011021031041051061071081091101111121131141151161171181191201211221231241251261271281291301311321331341351361371381391401411421431441451461471481491501511521531541551561571581591601611621631641651661671681691701711721731741751761771781791801811821831841851861871881891901911921931941951961971981992002012022032042052062072082092102112122132142152162172182192202212222232242252262272282292302312322332342352362372382392402412422432442452462472482492502512522532542552562572582592602612622632642652662672682692702712722732742752762772782792802812822832842852862872882892902912922932942952962972982993003013023033043053063073083093103113123133143153163173183193203213223233243253263273283293303313323333343353363373383393403413423433443453463473483493503513523533543553563573583593603613623633643653663673683693703713723733743753763773783793803813823833843853863873883893903913923933943953963973983994004014024034044054064074084094104114124134144154164174184194204214224234244254264274284294304314324334344354364374384394404414424434444454464474484494504514524534544554564574584594604614624634644654664674684694704714724734744754764774784794804814824834844854864874884894904914924934944954964974984995005015025035045055065075085095105115125135145155165175185195205215225235245255265275285295305315325335345355365375385395405415425435445455465475485495505515525535545555565575585595605615625635645655665675685695705715725735745755765775785795805815825835845855865875885895905915925935945955965975985996006016026036046056066076086096106116126136146156166176186196206216226236246256266276286296306316326336346356366376386396406416426436446456466476486496506516526536546556566576586596606616626636646656666676686696706716726736746756766776786796806816826836846856866876886896906916926936946956966976986997007017027037047057067077087097107117127137147157167177187197207217227237247257267277287297307317327337347357367377387397407417427437447457467477487497507517527537547557567577587597607617627637647657667677687697707717727737747757767777787797807817827837847857867877887897907917927937947957967977987998008018028038048058068078088098108118128138148158168178188198208218228238248258268278288298308318328338348358368378388398408418428438448458468478488498508518528538548558568578588598608618628638648658668678688698708718728738748758768778788798808818828838848858868878888898908918928938948958968978988999009019029039049059069079089099109119129139149159169179189199209219229239249259269279289299309319329339349359369379389399409419429439449459469479489499509519529539549559569579589599609619629639649659669679689699709719729739749759769779789799809819829839849859869879889899909919929939949959969979989991000100110021003100410051006100710081009101010111012101310141015101610171018101910201021102210231024102510261027102810291030103110321033103410351036103710381039104010411042104310441045104610471048104910501051105210531054105510561057105810591060106110621063106410651066106710681069107010711072107310741075107610771078107910801081108210831084108510861087108810891090109110921093109410951096109710981099110011011102110311041105110611071108110911101111111211131114111511161117111811191120112111221123112411251126112711281129113011311132113311341135113611371138113911401141114211431144114511461147114811491150115111521153115411551156115711581159116011611162116311641165116611671168116911701171117211731174117511761177 |
- import time
- from copy import deepcopy
- from datetime import timedelta
- from unittest.mock import patch
- from uuid import uuid4
- import pytest
- from django.utils import timezone
- from snuba_sdk import Op
- from sentry.issues.grouptype import PerformanceNPlusOneGroupType
- from sentry.models.group import Group
- from sentry.models.project import Project
- from sentry.models.rule import Rule
- from sentry.rules.conditions.event_frequency import (
- EventFrequencyCondition,
- EventFrequencyPercentCondition,
- EventUniqueUserFrequencyCondition,
- EventUniqueUserFrequencyConditionWithConditions,
- )
- from sentry.rules.match import MatchType
- from sentry.testutils.abstract import Abstract
- from sentry.testutils.cases import (
- BaseMetricsTestCase,
- PerformanceIssueTestCase,
- RuleTestCase,
- SnubaTestCase,
- )
- from sentry.testutils.helpers.datetime import before_now, freeze_time, iso_format
- from sentry.testutils.helpers.features import apply_feature_flag_on_cls
- from sentry.testutils.skips import requires_snuba
- from sentry.utils.samples import load_data
- pytestmark = [pytest.mark.sentry_metrics, requires_snuba]
- class BaseEventFrequencyPercentTest(BaseMetricsTestCase):
- def _make_sessions(
- self, num: int, environment_name: str | None = None, project: Project | None = None
- ):
- received = time.time()
- def make_session(i):
- return dict(
- distinct_id=uuid4().hex,
- session_id=uuid4().hex,
- org_id=project.organization_id if project else self.project.organization_id,
- project_id=project.id if project else self.project.id,
- status="ok",
- seq=0,
- release="foo@1.0.0",
- environment=environment_name if environment_name else "prod",
- retention_days=90,
- duration=None,
- errors=0,
- # The line below is crucial to spread sessions throughout the time period.
- started=received - i - 1,
- received=received,
- )
- self.bulk_store_sessions([make_session(i) for i in range(num)])
- class EventFrequencyQueryTestBase(SnubaTestCase, RuleTestCase, PerformanceIssueTestCase):
- def setUp(self):
- super().setUp()
- self.start = before_now(minutes=1)
- self.end = timezone.now()
- self.event = self.store_event(
- data={
- "event_id": "a" * 32,
- "environment": self.environment.name,
- "timestamp": iso_format(before_now(seconds=30)),
- "fingerprint": ["group-1"],
- "user": {"id": uuid4().hex},
- },
- project_id=self.project.id,
- )
- self.event2 = self.store_event(
- data={
- "event_id": "b" * 32,
- "environment": self.environment.name,
- "timestamp": iso_format(before_now(seconds=12)),
- "fingerprint": ["group-2"],
- "user": {"id": uuid4().hex},
- },
- project_id=self.project.id,
- )
- self.environment2 = self.create_environment(name="staging")
- self.event3 = self.store_event(
- data={
- "event_id": "c" * 32,
- "environment": self.environment2.name,
- "timestamp": iso_format(before_now(seconds=12)),
- "fingerprint": ["group-3"],
- "user": {"id": uuid4().hex},
- },
- project_id=self.project.id,
- )
- fingerprint = f"{PerformanceNPlusOneGroupType.type_id}-something_random"
- perf_event_data = load_data(
- "transaction-n-plus-one",
- timestamp=before_now(seconds=12),
- start_timestamp=before_now(seconds=13),
- fingerprint=[fingerprint],
- )
- perf_event_data["user"] = {"id": uuid4().hex}
- perf_event_data["environment"] = self.environment.name
- # Store a performance event
- self.perf_event = self.create_performance_issue(
- event_data=perf_event_data,
- project_id=self.project.id,
- fingerprint=fingerprint,
- )
- self.data = {"interval": "5m", "value": 30}
- self.condition_inst = self.get_rule(
- data=self.data,
- project=self.event.group.project,
- rule=Rule(environment_id=self.environment.id),
- )
- self.condition_inst2 = self.get_rule(
- data=self.data,
- project=self.event.group.project,
- rule=Rule(environment_id=self.environment2.id),
- )
- class EventFrequencyQueryTest(EventFrequencyQueryTestBase):
- rule_cls = EventFrequencyCondition
- def test_batch_query(self):
- batch_query = self.condition_inst.batch_query_hook(
- group_ids=[self.event.group_id, self.event2.group_id, self.perf_event.group_id],
- start=self.start,
- end=self.end,
- environment_id=self.environment.id,
- )
- assert batch_query == {
- self.event.group_id: 1,
- self.event2.group_id: 1,
- self.perf_event.group_id: 1,
- }
- batch_query = self.condition_inst2.batch_query_hook(
- group_ids=[self.event3.group_id],
- start=self.start,
- end=self.end,
- environment_id=self.environment2.id,
- )
- assert batch_query == {self.event3.group_id: 1}
- def test_get_error_and_generic_group_ids(self):
- groups = Group.objects.filter(
- id__in=[self.event.group_id, self.event2.group_id, self.perf_event.group_id]
- ).values("id", "type", "project__organization_id")
- error_issue_ids, generic_issue_ids = self.condition_inst.get_error_and_generic_group_ids(
- groups
- )
- assert self.event.group_id in error_issue_ids
- assert self.event2.group_id in error_issue_ids
- assert self.perf_event.group_id in generic_issue_ids
- class EventUniqueUserFrequencyQueryTest(EventFrequencyQueryTestBase):
- rule_cls = EventUniqueUserFrequencyCondition
- def test_batch_query_user(self):
- batch_query = self.condition_inst.batch_query_hook(
- group_ids=[self.event.group_id, self.event2.group_id, self.perf_event.group_id],
- start=self.start,
- end=self.end,
- environment_id=self.environment.id,
- )
- assert batch_query == {
- self.event.group_id: 1,
- self.event2.group_id: 1,
- self.perf_event.group_id: 1,
- }
- batch_query = self.condition_inst2.batch_query_hook(
- group_ids=[self.event3.group_id],
- start=self.start,
- end=self.end,
- environment_id=self.environment2.id,
- )
- assert batch_query == {self.event3.group_id: 1}
- class EventFrequencyPercentConditionQueryTest(
- EventFrequencyQueryTestBase, BaseEventFrequencyPercentTest
- ):
- rule_cls = EventFrequencyPercentCondition
- @patch("sentry.rules.conditions.event_frequency.MIN_SESSIONS_TO_FIRE", 1)
- def test_batch_query_percent(self):
- self._make_sessions(60, self.environment2.name)
- self._make_sessions(60, self.environment.name)
- batch_query = self.condition_inst.batch_query_hook(
- group_ids=[self.event.group_id, self.event2.group_id, self.perf_event.group_id],
- start=self.start,
- end=self.end,
- environment_id=self.environment.id,
- )
- percent_of_sessions = 20
- assert batch_query == {
- self.event.group_id: percent_of_sessions,
- self.event2.group_id: percent_of_sessions,
- self.perf_event.group_id: 0,
- }
- batch_query = self.condition_inst2.batch_query_hook(
- group_ids=[self.event3.group_id],
- start=self.start,
- end=self.end,
- environment_id=self.environment2.id,
- )
- assert batch_query == {self.event3.group_id: percent_of_sessions}
- @patch("sentry.rules.conditions.event_frequency.MIN_SESSIONS_TO_FIRE", 100)
- def test_batch_query_percent_no_avg_sessions_in_interval(self):
- self._make_sessions(60, self.environment2.name)
- self._make_sessions(60, self.environment.name)
- batch_query = self.condition_inst.batch_query_hook(
- group_ids=[self.event.group_id, self.event2.group_id, self.perf_event.group_id],
- start=self.start,
- end=self.end,
- environment_id=self.environment.id,
- )
- percent = 0
- assert batch_query == {
- self.event.group_id: percent,
- self.event2.group_id: percent,
- self.perf_event.group_id: percent,
- }
- batch_query = self.condition_inst2.batch_query_hook(
- group_ids=[self.event3.group_id],
- start=self.start,
- end=self.end,
- environment_id=self.environment2.id,
- )
- assert batch_query == {self.event3.group_id: percent}
- class ErrorEventMixin(SnubaTestCase):
- def add_event(self, data, project_id, timestamp):
- data["timestamp"] = iso_format(timestamp)
- # Store an error event
- event = self.store_event(
- data=data,
- project_id=project_id,
- )
- return event.for_group(event.group)
- class PerfIssuePlatformEventMixin(PerformanceIssueTestCase):
- def add_event(self, data, project_id, timestamp):
- fingerprint = data["fingerprint"][0]
- fingerprint = (
- fingerprint
- if "-" in fingerprint
- else f"{PerformanceNPlusOneGroupType.type_id}-{data['fingerprint'][0]}"
- )
- event_data = load_data(
- "transaction-n-plus-one",
- timestamp=timestamp,
- start_timestamp=timestamp,
- fingerprint=[fingerprint],
- )
- event_data["user"] = {"id": uuid4().hex}
- event_data["environment"] = data.get("environment")
- for tag in event_data["tags"]:
- if tag[0] == "environment":
- tag[1] = data.get("environment")
- break
- else:
- event_data["tags"].append(data.get("environment"))
- # Store a performance event
- event = self.create_performance_issue(
- event_data=event_data,
- project_id=project_id,
- fingerprint=fingerprint,
- )
- return event
- @pytest.mark.snuba_ci
- class StandardIntervalTestBase(SnubaTestCase, RuleTestCase, PerformanceIssueTestCase):
- __test__ = Abstract(__module__, __qualname__)
- def add_event(self, data, project_id, timestamp):
- raise NotImplementedError
- def increment(self, event, count, environment=None, timestamp=None):
- raise NotImplementedError
- def _run_test(self, minutes, data, passes, add_events=False):
- if not self.environment:
- self.environment = self.create_environment(name="prod")
- rule = self.get_rule(data=data, rule=Rule(environment_id=None))
- environment_rule = self.get_rule(data=data, rule=Rule(environment_id=self.environment.id))
- event = self.add_event(
- data={
- "fingerprint": ["something_random"],
- "user": {"id": uuid4().hex},
- },
- project_id=self.project.id,
- timestamp=before_now(minutes=minutes),
- )
- if add_events:
- self.increment(
- event,
- data["value"] + 1,
- environment=self.environment.name,
- timestamp=timezone.now() - timedelta(minutes=minutes),
- )
- self.increment(
- event,
- data["value"] + 1,
- timestamp=timezone.now() - timedelta(minutes=minutes),
- )
- if passes:
- self.assertPasses(rule, event, is_new=False)
- self.assertPasses(environment_rule, event, is_new=False)
- else:
- self.assertDoesNotPass(rule, event, is_new=False)
- self.assertDoesNotPass(environment_rule, event, is_new=False)
- def test_comparison_interval_empty_string(self):
- data = {
- "interval": "1m",
- "value": 16,
- "comparisonType": "count",
- "comparisonInterval": "",
- }
- self._run_test(data=data, minutes=1, passes=False)
- def test_one_minute_with_events(self):
- data = {"interval": "1m", "value": 6, "comparisonType": "count", "comparisonInterval": "5m"}
- self._run_test(data=data, minutes=1, passes=True, add_events=True)
- data = {
- "interval": "1m",
- "value": 16,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=1, passes=False)
- def test_one_hour_with_events(self):
- data = {"interval": "1h", "value": 6, "comparisonType": "count", "comparisonInterval": "5m"}
- self._run_test(data=data, minutes=60, passes=True, add_events=True)
- data = {
- "interval": "1h",
- "value": 16,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=60, passes=False)
- def test_one_day_with_events(self):
- data = {"interval": "1d", "value": 6, "comparisonType": "count", "comparisonInterval": "5m"}
- self._run_test(data=data, minutes=1440, passes=True, add_events=True)
- data = {
- "interval": "1d",
- "value": 16,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=1440, passes=False)
- def test_one_week_with_events(self):
- data = {"interval": "1w", "value": 6, "comparisonType": "count", "comparisonInterval": "5m"}
- self._run_test(data=data, minutes=10080, passes=True, add_events=True)
- data = {
- "interval": "1w",
- "value": 16,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=10080, passes=False)
- def test_one_minute_no_events(self):
- data = {"interval": "1m", "value": 6, "comparisonType": "count", "comparisonInterval": "5m"}
- self._run_test(data=data, minutes=1, passes=False)
- def test_one_hour_no_events(self):
- data = {"interval": "1h", "value": 6, "comparisonType": "count", "comparisonInterval": "5m"}
- self._run_test(data=data, minutes=60, passes=False)
- def test_one_day_no_events(self):
- data = {"interval": "1d", "value": 6, "comparisonType": "count", "comparisonInterval": "5m"}
- self._run_test(data=data, minutes=1440, passes=False)
- def test_one_week_no_events(self):
- data = {"interval": "1w", "value": 6, "comparisonType": "count", "comparisonInterval": "5m"}
- self._run_test(data=data, minutes=10080, passes=False)
- def test_comparison(self):
- # Test data is 4 events in the current period and 2 events in the comparison period, so
- # a 100% increase.
- event = self.add_event(
- data={
- "fingerprint": ["something_random"],
- "user": {"id": uuid4().hex},
- },
- project_id=self.project.id,
- timestamp=before_now(minutes=1),
- )
- self.increment(
- event,
- 3,
- timestamp=timezone.now() - timedelta(minutes=1),
- )
- self.increment(
- event,
- 2,
- timestamp=timezone.now() - timedelta(days=1, minutes=20),
- )
- data = {
- "interval": "1h",
- "value": 99,
- "comparisonType": "percent",
- "comparisonInterval": "1d",
- }
- rule = self.get_rule(data=data, rule=Rule(environment_id=None))
- self.assertPasses(rule, event, is_new=False)
- data = {
- "interval": "1h",
- "value": 101,
- "comparisonType": "percent",
- "comparisonInterval": "1d",
- }
- rule = self.get_rule(data=data, rule=Rule(environment_id=None))
- self.assertDoesNotPass(rule, event, is_new=False)
- def test_comparison_empty_comparison_period(self):
- # Test data is 1 event in the current period and 0 events in the comparison period. This
- # should always result in 0 and never fire.
- event = self.add_event(
- data={
- "fingerprint": ["something_random"],
- "user": {"id": uuid4().hex},
- },
- project_id=self.project.id,
- timestamp=before_now(minutes=1),
- )
- data = {
- "interval": "1h",
- "value": 0,
- "comparisonType": "percent",
- "comparisonInterval": "1d",
- }
- rule = self.get_rule(data=data, rule=Rule(environment_id=None))
- self.assertDoesNotPass(rule, event, is_new=False)
- data = {
- "interval": "1h",
- "value": 100,
- "comparisonType": "percent",
- "comparisonInterval": "1d",
- }
- rule = self.get_rule(data=data, rule=Rule(environment_id=None))
- self.assertDoesNotPass(rule, event, is_new=False)
- @patch("sentry.rules.conditions.event_frequency.BaseEventFrequencyCondition.get_rate")
- def test_is_new_issue_skips_snuba(self, mock_get_rate):
- # Looking for more than 1 event
- data = {"interval": "1m", "value": 6}
- minutes = 1
- rule = self.get_rule(data=data, rule=Rule(environment_id=None))
- environment_rule = self.get_rule(data=data, rule=Rule(environment_id=self.environment.id))
- event = self.add_event(
- data={
- "fingerprint": ["something_random"],
- "user": {"id": uuid4().hex},
- },
- project_id=self.project.id,
- timestamp=before_now(minutes=minutes),
- )
- # Issue is new and is the first event
- self.assertDoesNotPass(rule, event, is_new=True)
- self.assertDoesNotPass(environment_rule, event, is_new=True)
- assert mock_get_rate.call_count == 0
- class EventFrequencyConditionTestCase(StandardIntervalTestBase):
- __test__ = Abstract(__module__, __qualname__)
- rule_cls = EventFrequencyCondition
- def increment(self, event, count, environment=None, timestamp=None):
- timestamp = timestamp if timestamp else before_now(minutes=1)
- data = {"fingerprint": event.data["fingerprint"]}
- if environment:
- data["environment"] = environment
- for _ in range(count):
- self.add_event(
- data=data,
- project_id=self.project.id,
- timestamp=timestamp,
- )
- class EventUniqueUserFrequencyConditionTestCase(StandardIntervalTestBase):
- __test__ = Abstract(__module__, __qualname__)
- rule_cls = EventUniqueUserFrequencyCondition
- def increment(self, event, count, environment=None, timestamp=None):
- timestamp = timestamp if timestamp else before_now(minutes=1)
- data = {"fingerprint": event.data["fingerprint"]}
- if environment:
- data["environment"] = environment
- for _ in range(count):
- event_data = deepcopy(data)
- event_data["user"] = {"id": uuid4().hex}
- self.add_event(
- data=event_data,
- project_id=self.project.id,
- timestamp=timestamp,
- )
- @apply_feature_flag_on_cls("organizations:event-unique-user-frequency-condition-with-conditions")
- class EventUniqueUserFrequencyConditionWithConditionsTestCase(StandardIntervalTestBase):
- __test__ = Abstract(__module__, __qualname__)
- rule_cls = EventUniqueUserFrequencyConditionWithConditions
- def increment(self, event, count, environment=None, timestamp=None):
- timestamp = timestamp if timestamp else before_now(minutes=1)
- data = {"fingerprint": event.data["fingerprint"]}
- if environment:
- data["environment"] = environment
- for _ in range(count):
- event_data = deepcopy(data)
- event_data["user"] = {"id": uuid4().hex}
- self.add_event(
- data=event_data,
- project_id=self.project.id,
- timestamp=timestamp,
- )
- def test_comparison(self):
- # Test data is 4 events in the current period and 2 events in the comparison period, so
- # a 100% increase.
- event = self.add_event(
- data={
- "fingerprint": ["something_random"],
- "user": {"id": uuid4().hex},
- },
- project_id=self.project.id,
- timestamp=before_now(minutes=1),
- )
- self.increment(
- event,
- 3,
- timestamp=timezone.now() - timedelta(minutes=1),
- )
- self.increment(
- event,
- 2,
- timestamp=timezone.now() - timedelta(days=1, minutes=20),
- )
- data = {
- "interval": "1h",
- "value": 99,
- "comparisonType": "percent",
- "comparisonInterval": "1d",
- "id": "EventFrequencyConditionWithConditions",
- }
- rule = self.get_rule(
- data=data,
- rule=Rule(
- environment_id=None,
- project_id=self.project.id,
- data={
- "conditions": [data],
- "filter_match": "all",
- },
- ),
- )
- self.assertPasses(rule, event, is_new=False)
- data = {
- "interval": "1h",
- "value": 101,
- "comparisonType": "percent",
- "comparisonInterval": "1d",
- "id": "EventFrequencyConditionWithConditions",
- }
- rule = self.get_rule(
- data=data,
- rule=Rule(
- environment_id=None,
- project_id=self.project.id,
- data={
- "conditions": [data],
- "filter_match": "all",
- },
- ),
- )
- self.assertDoesNotPass(rule, event, is_new=False)
- def test_comparison_empty_comparison_period(self):
- # Test data is 1 event in the current period and 0 events in the comparison period. This
- # should always result in 0 and never fire.
- event = self.add_event(
- data={
- "fingerprint": ["something_random"],
- "user": {"id": uuid4().hex},
- },
- project_id=self.project.id,
- timestamp=before_now(minutes=1),
- )
- data = {
- "filter_match": "all",
- "conditions": [
- {
- "interval": "1h",
- "value": 0,
- "comparisonType": "percent",
- "comparisonInterval": "1d",
- }
- ],
- }
- rule = self.get_rule(
- data=data, rule=Rule(environment_id=None, project_id=self.project.id, data=data)
- )
- self.assertDoesNotPass(rule, event, is_new=False)
- data = {
- "filter_match": "all",
- "conditions": [
- {
- "interval": "1h",
- "value": 100,
- "comparisonType": "percent",
- "comparisonInterval": "1d",
- }
- ],
- }
- rule = self.get_rule(
- data=data, rule=Rule(environment_id=None, project_id=self.project.id, data=data)
- )
- self.assertDoesNotPass(rule, event, is_new=False)
- def _run_test(self, minutes, data, passes, add_events=False):
- if not self.environment:
- self.environment = self.create_environment(name="prod")
- data["filter_match"] = "all"
- data["conditions"] = data.get("conditions", [])
- rule = self.get_rule(
- data=data,
- rule=Rule(environment_id=None, project_id=self.project.id, data=data),
- )
- environment_rule = self.get_rule(
- data=data,
- rule=Rule(
- environment_id=self.environment.id,
- project_id=self.project.id,
- data=data,
- ),
- )
- event = self.add_event(
- data={
- "fingerprint": ["something_random"],
- "user": {"id": uuid4().hex},
- },
- project_id=self.project.id,
- timestamp=before_now(minutes=minutes),
- )
- if add_events:
- self.increment(
- event,
- data["value"] + 1,
- environment=self.environment.name,
- timestamp=timezone.now() - timedelta(minutes=minutes),
- )
- self.increment(
- event,
- data["value"] + 1,
- timestamp=timezone.now() - timedelta(minutes=minutes),
- )
- if passes:
- self.assertPasses(rule, event, is_new=False)
- self.assertPasses(environment_rule, event, is_new=False)
- else:
- self.assertDoesNotPass(rule, event, is_new=False)
- self.assertDoesNotPass(environment_rule, event, is_new=False)
- def test_convert_rule_condition_to_snuba_condition():
- # Test non-TaggedEventFilter condition
- condition = {"id": "some.other.condition"}
- assert (
- EventUniqueUserFrequencyConditionWithConditions.convert_rule_condition_to_snuba_condition(
- condition
- )
- is None
- )
- # Test TaggedEventFilter conditions
- base_condition = {
- "id": "sentry.rules.filters.tagged_event.TaggedEventFilter",
- "key": "test_key",
- "value": "test_value",
- }
- # Test equality
- eq_condition = {**base_condition, "match": MatchType.EQUAL}
- assert (
- EventUniqueUserFrequencyConditionWithConditions.convert_rule_condition_to_snuba_condition(
- eq_condition
- )
- == (
- "tags[test_key]",
- Op.EQ.value,
- "test_value",
- )
- )
- # Test inequality
- ne_condition = {**base_condition, "match": MatchType.NOT_EQUAL}
- assert (
- EventUniqueUserFrequencyConditionWithConditions.convert_rule_condition_to_snuba_condition(
- ne_condition
- )
- == (
- "tags[test_key]",
- Op.NEQ.value,
- "test_value",
- )
- )
- # Test starts with
- sw_condition = {**base_condition, "match": MatchType.STARTS_WITH}
- assert (
- EventUniqueUserFrequencyConditionWithConditions.convert_rule_condition_to_snuba_condition(
- sw_condition
- )
- == (
- "tags[test_key]",
- Op.LIKE.value,
- "test_value%",
- )
- )
- # Test not starts with
- nsw_condition = {**base_condition, "match": MatchType.NOT_STARTS_WITH}
- assert (
- EventUniqueUserFrequencyConditionWithConditions.convert_rule_condition_to_snuba_condition(
- nsw_condition
- )
- == (
- "tags[test_key]",
- Op.NOT_LIKE.value,
- "test_value%",
- )
- )
- # Test ends with
- ew_condition = {**base_condition, "match": MatchType.ENDS_WITH}
- assert (
- EventUniqueUserFrequencyConditionWithConditions.convert_rule_condition_to_snuba_condition(
- ew_condition
- )
- == (
- "tags[test_key]",
- Op.LIKE.value,
- "%test_value",
- )
- )
- # Test not ends with
- new_condition = {**base_condition, "match": MatchType.NOT_ENDS_WITH}
- assert (
- EventUniqueUserFrequencyConditionWithConditions.convert_rule_condition_to_snuba_condition(
- new_condition
- )
- == (
- "tags[test_key]",
- Op.NOT_LIKE.value,
- "%test_value",
- )
- )
- # Test contains
- co_condition = {**base_condition, "match": MatchType.CONTAINS}
- assert (
- EventUniqueUserFrequencyConditionWithConditions.convert_rule_condition_to_snuba_condition(
- co_condition
- )
- == (
- "tags[test_key]",
- Op.LIKE.value,
- "%test_value%",
- )
- )
- # Test not contains
- nc_condition = {**base_condition, "match": MatchType.NOT_CONTAINS}
- assert (
- EventUniqueUserFrequencyConditionWithConditions.convert_rule_condition_to_snuba_condition(
- nc_condition
- )
- == (
- "tags[test_key]",
- Op.NOT_LIKE.value,
- "%test_value%",
- )
- )
- # Test is set
- is_condition = {**base_condition, "match": MatchType.IS_SET}
- assert (
- EventUniqueUserFrequencyConditionWithConditions.convert_rule_condition_to_snuba_condition(
- is_condition
- )
- == (
- "tags[test_key]",
- Op.IS_NOT_NULL.value,
- None,
- )
- )
- # Test not set
- ns_condition = {**base_condition, "match": MatchType.NOT_SET}
- assert (
- EventUniqueUserFrequencyConditionWithConditions.convert_rule_condition_to_snuba_condition(
- ns_condition
- )
- == (
- "tags[test_key]",
- Op.IS_NULL.value,
- None,
- )
- )
- # Test is in
- in_condition = {
- "id": "sentry.rules.filters.tagged_event.TaggedEventFilter",
- "key": "test_key",
- "value": "test_value_1,test_value_2",
- "match": MatchType.IS_IN,
- }
- assert (
- EventUniqueUserFrequencyConditionWithConditions.convert_rule_condition_to_snuba_condition(
- in_condition
- )
- == (
- "tags[test_key]",
- Op.IN.value,
- ["test_value_1", "test_value_2"],
- )
- )
- # Test not in
- not_in_condition = {
- "id": "sentry.rules.filters.tagged_event.TaggedEventFilter",
- "key": "test_key",
- "value": "test_value_1,test_value_2",
- "match": MatchType.NOT_IN,
- }
- assert (
- EventUniqueUserFrequencyConditionWithConditions.convert_rule_condition_to_snuba_condition(
- not_in_condition
- )
- == (
- "tags[test_key]",
- Op.NOT_IN.value,
- ["test_value_1", "test_value_2"],
- )
- )
- # Test unsupported match type
- with pytest.raises(ValueError, match="Unsupported match type: unsupported"):
- EventUniqueUserFrequencyConditionWithConditions.convert_rule_condition_to_snuba_condition(
- {**base_condition, "match": "unsupported"}
- )
- class EventFrequencyPercentConditionTestCase(BaseEventFrequencyPercentTest, RuleTestCase):
- __test__ = Abstract(__module__, __qualname__)
- rule_cls = EventFrequencyPercentCondition
- def add_event(self, data, project_id, timestamp):
- raise NotImplementedError
- def _run_test(self, minutes, data, passes, add_events=False):
- if not self.environment or self.environment.name != "prod":
- self.environment = self.create_environment(name="prod")
- if not hasattr(self, "test_event"):
- self.test_event = self.add_event(
- data={
- "fingerprint": ["something_random"],
- "user": {"id": uuid4().hex},
- "environment": self.environment.name,
- },
- project_id=self.project.id,
- timestamp=before_now(minutes=minutes),
- )
- if add_events:
- self.increment(
- self.test_event,
- max(1, int(minutes / 2)) - 1,
- environment=self.environment.name,
- timestamp=timezone.now() - timedelta(minutes=minutes),
- )
- rule = self.get_rule(data=data, rule=Rule(environment_id=None))
- environment_rule = self.get_rule(data=data, rule=Rule(environment_id=self.environment.id))
- if passes:
- self.assertPasses(rule, self.test_event, is_new=False)
- self.assertPasses(environment_rule, self.test_event, is_new=False)
- else:
- self.assertDoesNotPass(rule, self.test_event)
- self.assertDoesNotPass(environment_rule, self.test_event)
- def increment(self, event, count, environment=None, timestamp=None):
- data = {
- "fingerprint": event.data["fingerprint"],
- }
- timestamp = timestamp if timestamp else before_now(minutes=1)
- if environment:
- data["environment"] = environment
- for _ in range(count):
- event_data = deepcopy(data)
- event_data["user"] = {"id": uuid4().hex}
- self.add_event(
- data=event_data,
- project_id=self.project.id,
- timestamp=timestamp,
- )
- @patch("sentry.rules.conditions.event_frequency.MIN_SESSIONS_TO_FIRE", 1)
- def test_five_minutes_with_events(self):
- self._make_sessions(60)
- data = {
- "interval": "5m",
- "value": 39,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=5, passes=True, add_events=True)
- data = {
- "interval": "5m",
- "value": 41,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=5, passes=False)
- @patch("sentry.rules.conditions.event_frequency.MIN_SESSIONS_TO_FIRE", 1)
- def test_ten_minutes_with_events(self):
- self._make_sessions(60)
- data = {
- "interval": "10m",
- "value": 49,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=10, passes=True, add_events=True)
- data = {
- "interval": "10m",
- "value": 51,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=10, passes=False)
- @patch("sentry.rules.conditions.event_frequency.MIN_SESSIONS_TO_FIRE", 1)
- def test_thirty_minutes_with_events(self):
- self._make_sessions(60)
- data = {
- "interval": "30m",
- "value": 49,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=30, passes=True, add_events=True)
- data = {
- "interval": "30m",
- "value": 51,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=30, passes=False)
- @patch("sentry.rules.conditions.event_frequency.MIN_SESSIONS_TO_FIRE", 1)
- def test_one_hour_with_events(self):
- self._make_sessions(60)
- data = {
- "interval": "1h",
- "value": 49,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=60, add_events=True, passes=True)
- data = {
- "interval": "1h",
- "value": 51,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=60, passes=False)
- @patch("sentry.rules.conditions.event_frequency.MIN_SESSIONS_TO_FIRE", 1)
- def test_five_minutes_no_events(self):
- self._make_sessions(60)
- data = {
- "interval": "5m",
- "value": 39,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=5, passes=True, add_events=True)
- @patch("sentry.rules.conditions.event_frequency.MIN_SESSIONS_TO_FIRE", 1)
- def test_ten_minutes_no_events(self):
- self._make_sessions(60)
- data = {
- "interval": "10m",
- "value": 49,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=10, passes=True, add_events=True)
- @patch("sentry.rules.conditions.event_frequency.MIN_SESSIONS_TO_FIRE", 1)
- def test_thirty_minutes_no_events(self):
- self._make_sessions(60)
- data = {
- "interval": "30m",
- "value": 49,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=30, passes=True, add_events=True)
- @patch("sentry.rules.conditions.event_frequency.MIN_SESSIONS_TO_FIRE", 1)
- def test_one_hour_no_events(self):
- self._make_sessions(60)
- data = {
- "interval": "1h",
- "value": 49,
- "comparisonType": "count",
- "comparisonInterval": "5m",
- }
- self._run_test(data=data, minutes=60, passes=False)
- @patch("sentry.rules.conditions.event_frequency.MIN_SESSIONS_TO_FIRE", 1)
- def test_comparison(self):
- self._make_sessions(10)
- # Create sessions for previous period
- self._make_sessions(10)
- # Test data is 2 events in the current period and 1 events in the comparison period.
- # Number of sessions is 20 in each period, so current period is 20% of sessions, prev
- # is 10%. Overall a 100% increase comparatively.
- event = self.add_event(
- data={"fingerprint": ["something_random"]},
- project_id=self.project.id,
- timestamp=before_now(minutes=1),
- )
- self.increment(
- event,
- 1,
- timestamp=timezone.now() - timedelta(minutes=1),
- )
- self.increment(
- event,
- 1,
- timestamp=timezone.now() - timedelta(days=1, minutes=20),
- )
- data = {
- "interval": "1h",
- "value": 99,
- "comparisonType": "percent",
- "comparisonInterval": "1d",
- }
- rule = self.get_rule(data=data, rule=Rule(environment_id=None))
- self.assertPasses(rule, event, is_new=False)
- data = {
- "interval": "1h",
- "value": 101,
- "comparisonType": "percent",
- "comparisonInterval": "1d",
- }
- rule = self.get_rule(data=data, rule=Rule(environment_id=None))
- self.assertDoesNotPass(rule, event, is_new=False)
- @freeze_time(
- (timezone.now() - timedelta(days=2)).replace(hour=12, minute=40, second=0, microsecond=0)
- )
- class ErrorIssueFrequencyConditionTestCase(ErrorEventMixin, EventFrequencyConditionTestCase):
- pass
- @freeze_time(
- (timezone.now() - timedelta(days=2)).replace(hour=12, minute=40, second=0, microsecond=0)
- )
- class PerfIssuePlatformIssueFrequencyConditionTestCase(
- PerfIssuePlatformEventMixin,
- EventFrequencyConditionTestCase,
- ):
- pass
- @freeze_time(
- (timezone.now() - timedelta(days=2)).replace(hour=12, minute=40, second=0, microsecond=0)
- )
- class ErrorIssueUniqueUserFrequencyConditionTestCase(
- ErrorEventMixin,
- EventUniqueUserFrequencyConditionTestCase,
- ):
- pass
- @freeze_time(
- (timezone.now() - timedelta(days=2)).replace(hour=12, minute=40, second=0, microsecond=0)
- )
- class ErrorIssueUniqueUserFrequencyConditionWithConditionsTestCase(
- ErrorEventMixin,
- EventUniqueUserFrequencyConditionWithConditionsTestCase,
- ):
- pass
- @freeze_time(
- (timezone.now() - timedelta(days=2)).replace(hour=12, minute=40, second=0, microsecond=0)
- )
- class PerfIssuePlatformIssueUniqueUserFrequencyConditionTestCase(
- PerfIssuePlatformEventMixin,
- EventUniqueUserFrequencyConditionTestCase,
- ):
- pass
- @freeze_time(
- (timezone.now() - timedelta(days=2)).replace(hour=12, minute=40, second=0, microsecond=0)
- )
- class ErrorIssueEventFrequencyPercentConditionTestCase(
- ErrorEventMixin, EventFrequencyPercentConditionTestCase
- ):
- pass
- @freeze_time(
- (timezone.now() - timedelta(days=2)).replace(hour=12, minute=40, second=0, microsecond=0)
- )
- class PerfIssuePlatformIssueEventFrequencyPercentConditionTestCase(
- PerfIssuePlatformEventMixin,
- EventFrequencyPercentConditionTestCase,
- ):
- pass
|