Skip to content

Commit 684832a

Browse files
parametrize test
1 parent 6bf8f11 commit 684832a

1 file changed

Lines changed: 8 additions & 4 deletions

File tree

tests/integrations/django/test_basic.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1180,8 +1180,9 @@ def test_csrf(sentry_init, client):
11801180

11811181

11821182
@pytest.mark.skipif(DJANGO_VERSION < (2, 0), reason="Requires Django > 2.0")
1183+
@pytest.mark.parametrize("middleware_spans", [False, True])
11831184
def test_custom_urlconf_middleware(
1184-
settings, sentry_init, client, capture_events, render_span_tree
1185+
settings, sentry_init, client, capture_events, render_span_tree, middleware_spans
11851186
):
11861187
"""
11871188
Some middlewares (for instance in django-tenants) overwrite request.urlconf.
@@ -1193,7 +1194,8 @@ def test_custom_urlconf_middleware(
11931194
client.application.load_middleware()
11941195

11951196
sentry_init(
1196-
integrations=[DjangoIntegration(middleware_spans=True)], traces_sample_rate=1.0
1197+
integrations=[DjangoIntegration(middleware_spans=middleware_spans)],
1198+
traces_sample_rate=1.0,
11971199
)
11981200
events = capture_events()
11991201

@@ -1203,7 +1205,8 @@ def test_custom_urlconf_middleware(
12031205

12041206
event = events.pop(0)
12051207
assert event["transaction"] == "/custom/ok"
1206-
assert "custom_urlconf_middleware" in render_span_tree(event)
1208+
if middleware_spans:
1209+
assert "custom_urlconf_middleware" in render_span_tree(event)
12071210

12081211
_content, status, _headers = unpack_werkzeug_response(client.get("/custom/exc"))
12091212
assert status.lower() == "500 internal server error"
@@ -1212,7 +1215,8 @@ def test_custom_urlconf_middleware(
12121215
assert error_event["transaction"] == "/custom/exc"
12131216
assert error_event["exception"]["values"][-1]["mechanism"]["type"] == "django"
12141217
assert transaction_event["transaction"] == "/custom/exc"
1215-
assert "custom_urlconf_middleware" in render_span_tree(transaction_event)
1218+
if middleware_spans:
1219+
assert "custom_urlconf_middleware" in render_span_tree(transaction_event)
12161220

12171221
settings.MIDDLEWARE.pop(0)
12181222

0 commit comments

Comments
 (0)