test: More tests for django post body

untitaker · untitaker · commit 2b0c68bcbfcb · 2018-10-11T21:44:30.000+02:00
diff --git a/tests/integrations/django/myapp/urls.py b/tests/integrations/django/myapp/urls.py
@@ -28,6 +28,7 @@
     path("message", views.message, name="message"),
     path("mylogin", views.mylogin, name="mylogin"),
     path("classbased", views.ClassBasedView.as_view(), name="classbased"),
+    path("post-echo", views.post_echo, name="post_echo"),
 ]
 
 handler500 = views.handler500
diff --git a/tests/integrations/django/myapp/views.py b/tests/integrations/django/myapp/views.py
@@ -32,3 +32,8 @@ class ClassBasedView(ListView):
     def head(self, *args, **kwargs):
         sentry_sdk.capture_message("hi")
         return HttpResponse("")
+
+
+def post_echo(request):
+    sentry_sdk.capture_message("hi")
+    return HttpResponse(request.body)
diff --git a/tests/integrations/django/test_basic.py b/tests/integrations/django/test_basic.py
@@ -185,3 +185,33 @@ def test_transaction_style(
 
     event, = events
     assert event["transaction"] == expected_transaction
+
+
+def test_request_body(sentry_init, client, capture_events):
+    sentry_init(integrations=[DjangoIntegration()])
+    events = capture_events()
+    content, status, headers = client.post(
+        reverse("post_echo"), data=b"heyooo", content_type="text/plain"
+    )
+    assert status.lower() == "200 ok"
+    assert b"".join(content) == b"heyooo"
+
+    event, = events
+
+    assert event["message"] == "hi"
+    assert event["request"]["data"] == ""
+    assert event[""]["request"]["data"][""] == {"len": 6, "rem": [["!raw", "x", 0, 6]]}
+
+    del events[:]
+
+    content, status, headers = client.post(
+        reverse("post_echo"), data=b'{"hey": 42}', content_type="application/json"
+    )
+    assert status.lower() == "200 ok"
+    assert b"".join(content) == b'{"hey": 42}'
+
+    event, = events
+
+    assert event["message"] == "hi"
+    assert event["request"]["data"] == {"hey": 42}
+    assert "" not in event

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@`
`28`	`28`	`path("message", views.message, name="message"),`
`29`	`29`	`path("mylogin", views.mylogin, name="mylogin"),`
`30`	`30`	`path("classbased", views.ClassBasedView.as_view(), name="classbased"),`
	`31`	`+ path("post-echo", views.post_echo, name="post_echo"),`
`31`	`32`	`]`
`32`	`33`
`33`	`34`	`handler500 = views.handler500`