Skip to content

Commit 7597448

Browse files
committed
Mailtrap webhook tests: remove unnecessary json.dumps
1 parent 1da6971 commit 7597448

File tree

1 file changed

+10
-11
lines changed

1 file changed

+10
-11
lines changed

tests/test_mailtrap_webhooks.py

+10-11
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
import json
21
from datetime import datetime, timezone
32
from unittest.mock import ANY
43

@@ -16,7 +15,7 @@ def call_webhook(self):
1615
return self.client.post(
1716
"/anymail/mailtrap/tracking/",
1817
content_type="application/json",
19-
data=json.dumps({}),
18+
data={},
2019
)
2120

2221
# Actual tests are in WebhookBasicAuthTestCase
@@ -42,7 +41,7 @@ def test_sent_event(self):
4241
response = self.client.post(
4342
"/anymail/mailtrap/tracking/",
4443
content_type="application/json",
45-
data=json.dumps(payload),
44+
data=payload,
4645
)
4746
self.assertEqual(response.status_code, 200)
4847
kwargs = self.assert_handler_called_once_with(
@@ -87,7 +86,7 @@ def test_open_event(self):
8786
response = self.client.post(
8887
"/anymail/mailtrap/tracking/",
8988
content_type="application/json",
90-
data=json.dumps(payload),
89+
data=payload,
9190
)
9291
self.assertEqual(response.status_code, 200)
9392
kwargs = self.assert_handler_called_once_with(
@@ -129,7 +128,7 @@ def test_click_event(self):
129128
response = self.client.post(
130129
"/anymail/mailtrap/tracking/",
131130
content_type="application/json",
132-
data=json.dumps(payload),
131+
data=payload,
133132
)
134133
self.assertEqual(response.status_code, 200)
135134
kwargs = self.assert_handler_called_once_with(
@@ -174,7 +173,7 @@ def test_bounce_event(self):
174173
response = self.client.post(
175174
"/anymail/mailtrap/tracking/",
176175
content_type="application/json",
177-
data=json.dumps(payload),
176+
data=payload,
178177
)
179178
self.assertEqual(response.status_code, 200)
180179
kwargs = self.assert_handler_called_once_with(
@@ -218,7 +217,7 @@ def test_soft_bounce_event(self):
218217
response = self.client.post(
219218
"/anymail/mailtrap/tracking/",
220219
content_type="application/json",
221-
data=json.dumps(payload),
220+
data=payload,
222221
)
223222
self.assertEqual(response.status_code, 200)
224223
kwargs = self.assert_handler_called_once_with(
@@ -256,7 +255,7 @@ def test_spam_event(self):
256255
response = self.client.post(
257256
"/anymail/mailtrap/tracking/",
258257
content_type="application/json",
259-
data=json.dumps(payload),
258+
data=payload,
260259
)
261260
self.assertEqual(response.status_code, 200)
262261
kwargs = self.assert_handler_called_once_with(
@@ -291,7 +290,7 @@ def test_unsubscribe_event(self):
291290
response = self.client.post(
292291
"/anymail/mailtrap/tracking/",
293292
content_type="application/json",
294-
data=json.dumps(payload),
293+
data=payload,
295294
)
296295
self.assertEqual(response.status_code, 200)
297296
kwargs = self.assert_handler_called_once_with(
@@ -327,7 +326,7 @@ def test_suspension_event(self):
327326
response = self.client.post(
328327
"/anymail/mailtrap/tracking/",
329328
content_type="application/json",
330-
data=json.dumps(payload),
329+
data=payload,
331330
)
332331
self.assertEqual(response.status_code, 200)
333332
kwargs = self.assert_handler_called_once_with(
@@ -359,7 +358,7 @@ def test_reject_event(self):
359358
response = self.client.post(
360359
"/anymail/mailtrap/tracking/",
361360
content_type="application/json",
362-
data=json.dumps(payload),
361+
data=payload,
363362
)
364363
self.assertEqual(response.status_code, 200)
365364
kwargs = self.assert_handler_called_once_with(

0 commit comments

Comments
 (0)