Mailjet: handle bare event when webhook "group events" not enabled

Fixes #106

Author: medmunds

anymail/webhooks/mailjet.py

@@ -16,6 +16,14 @@ class MailjetTrackingWebhookView(AnymailBaseWebhookView):
 
     def parse_events(self, request):
         esp_events = json.loads(request.body.decode('utf-8'))
+        # Mailjet webhook docs say the payload is "a JSON array of event objects,"
+        # but that's not true if "group events" isn't enabled in webhook config...
+        try:
+            esp_events[0]  # is this really an array of events?
+        except IndexError:
+            pass  # yep (and it's empty?!)
+        except KeyError:
+            esp_events = [esp_events]  # nope, it's a single, bare event
         return [self.esp_to_anymail_event(esp_event) for esp_event in esp_events]
 
     # https://dev.mailjet.com/guides/#events

tests/test_mailjet_webhooks.py

@@ -224,3 +224,32 @@ def test_bounced_greylist_event(self):
         self.assertEqual(event.message_id, "12345678901234567")
         self.assertEqual(event.recipient, "[email protected]")
         self.assertEqual(event.reject_reason, "other")
+
+    def test_non_grouped_event(self):
+        # If you don't enable "group events" on a webhook, Mailjet sends a single bare event
+        # (not a list of one event, despite what the docs say).
+        raw_event = {
+            "event": "sent",
+            "time": 1498093527,
+            "MessageID": 12345678901234567,
+            "email": "[email protected]",
+            "mj_campaign_id": 1234567890,
+            "mj_contact_id": 9876543210,
+            "customcampaign": "",
+            "mj_message_id": "12345678901234567",
+            "smtp_reply": "sent (250 2.0.0 OK 1498093527 a67bc12345def.22 - gsmtp)",
+            "Payload": "",
+        }
+        response = self.client.post('/anymail/mailjet/tracking/',
+                                    content_type='application/json', data=json.dumps(raw_event))
+        self.assertEqual(response.status_code, 200)
+        kwargs = self.assert_handler_called_once_with(self.tracking_handler, sender=MailjetTrackingWebhookView,
+                                                      event=ANY, esp_name='Mailjet')
+        event = kwargs['event']
+        self.assertIsInstance(event, AnymailTrackingEvent)
+        self.assertEqual(event.event_type, "delivered")
+        self.assertEqual(event.timestamp, datetime(2017, 6, 22, 1, 5, 27, tzinfo=utc))
+        self.assertEqual(event.esp_event, raw_event)
+        self.assertEqual(event.mta_response, "sent (250 2.0.0 OK 1498093527 a67bc12345def.22 - gsmtp)")
+        self.assertEqual(event.message_id, "12345678901234567")  # converted to str (matching backend status)
+        self.assertEqual(event.recipient, "[email protected]")