Skip to content

Commit

Permalink
test(flaky): Remove unused metrics consumer from test (#3741)
Browse files Browse the repository at this point in the history
No metrics are asserted in the test, no need to setup metric emission
for the test. This also means the emitted usage metric doesn't cause the
test to fail.
  • Loading branch information
Dav1dde committed Jun 18, 2024
1 parent 7205cce commit 437a479
Showing 1 changed file with 2 additions and 12 deletions.
14 changes: 2 additions & 12 deletions tests/integration/test_spans.py
Original file line number Diff line number Diff line change
Expand Up @@ -1331,20 +1331,10 @@ def test_span_reject_invalid_timestamps(


def test_span_ingestion_with_performance_scores(
mini_sentry, relay_with_processing, spans_consumer, metrics_consumer
mini_sentry, relay_with_processing, spans_consumer
):
spans_consumer = spans_consumer()
metrics_consumer = metrics_consumer()
relay = relay_with_processing(
options={
"aggregator": {
"bucket_interval": 1,
"initial_delay": 0,
"debounce_delay": 0,
"shift_key": "none",
}
}
)
relay = relay_with_processing()

project_id = 42
project_config = mini_sentry.add_full_project_config(project_id)
Expand Down

0 comments on commit 437a479

Please sign in to comment.