Browse Source

chore(capman): add tenant_ids to unmerge test (#48225)

Just updating some tests to be in line with snuba allocation policy
requirements. no functional changes

Allows https://github.com/getsentry/snuba/pull/4098 to be merged

This should be the last one (for errors and transactions at least)
volokluev 1 year ago
parent
commit
11afe16b3f
1 changed files with 4 additions and 0 deletions
  1. 4 0
      tests/snuba/tasks/test_unmerge.py

+ 4 - 0
tests/snuba/tasks/test_unmerge.py

@@ -438,6 +438,7 @@ class UnmergeTestCase(TestCase, SnubaTestCase):
             now - timedelta(seconds=rollup_duration),
             time_from_now(17),
             rollup_duration,
+            tenant_ids={"referrer": "r", "organization_id": 1234},
         )
 
         environment_time_series = tsdb.get_distinct_counts_series(
@@ -447,6 +448,7 @@ class UnmergeTestCase(TestCase, SnubaTestCase):
             time_from_now(17),
             rollup_duration,
             environment_id=production_environment.id,
+            tenant_ids={"referrer": "r", "organization_id": 1234},
         )
 
         def collect_by_user_tag(aggregate, event):
@@ -512,6 +514,7 @@ class UnmergeTestCase(TestCase, SnubaTestCase):
                 now - timedelta(seconds=rollup_duration),
                 time_from_now(17),
                 rollup_duration,
+                tenant_ids={"referrer": "r", "organization_id": 1234},
             )
         )
 
@@ -546,6 +549,7 @@ class UnmergeTestCase(TestCase, SnubaTestCase):
                 now - timedelta(seconds=rollup_duration),
                 time_from_now(17),
                 rollup_duration,
+                tenant_ids={"referrer": "r", "organization_id": 1234},
             )
         )