Removed Message().load(...) and added an error case test.

Author: Arjen

messagebird/client.py

@@ -90,7 +90,7 @@ def message_create(self, originator, recipients, body, params=None):
 
   def message_delete(self, id):
     """Delete a message from the dashboard."""
-    return Message().load(self.request('messages/' + str(id), 'DELETE'))
+    self.request_plain_text('messages/' + str(id), 'DELETE')
 
   def voice_message(self, id):
     "Retrieve the information of a specific voice message."

tests/test_message.py

@@ -1,5 +1,5 @@
 import unittest
-from messagebird import Client
+from messagebird import Client, ErrorException
 
 try:
     from unittest.mock import Mock
@@ -36,3 +36,12 @@ def test_message_delete(self):
         Client('', http_client).message_delete('message-id')
 
         http_client.request.assert_called_once_with('messages/message-id', 'DELETE', None)
+
+    def test_message_delete_invalid(self):
+        http_client = Mock()
+        http_client.request.return_value = '{"errors": [{"code": 20, "description": "message not found", "parameter": null}]}'
+
+        with self.assertRaises(ErrorException):
+            Client('', http_client).message_delete('non-existent-message-id')
+
+        http_client.request.assert_called_once_with('messages/non-existent-message-id', 'DELETE', None)