Ensure TypeErrors are being used in place of AssertionErrors for invalid parameter types

cmlccie · cmlccie · commit abd095f8397e · 2018-02-24T17:00:03.000-05:00
diff --git a/ciscosparkapi/api/messages.py b/ciscosparkapi/api/messages.py
@@ -127,17 +127,17 @@ def __init__(self, session):
                 API calls to the Cisco Spark service.
 
         Raises:
-            AssertionError: If the parameter types are incorrect.
+            TypeError: If the parameter types are incorrect.
 
         """
-        assert isinstance(session, RestSession)
+        check_type(session, RestSession, may_be_none=False)
         super(MessagesAPI, self).__init__()
         self._session = session
 
     @generator_container
     def list(self, roomId, mentionedPeople=None, before=None,
              beforeMessage=None, max=None, **request_parameters):
-        """Lists all messages in a room.
+        """Lists messages in a room.
 
         Each message will include content attachments if present.
 
@@ -199,7 +199,7 @@ def list(self, roomId, mentionedPeople=None, before=None,
 
     def create(self, roomId=None, toPersonId=None, toPersonEmail=None,
                text=None, markdown=None, files=None, **request_parameters):
-        """Posts a message, and optionally a attachment, to a room.
+        """Post a message, and optionally a attachment, to a room.
 
         The files parameter is a list, which accepts multiple values to allow
         for future expansion, but currently only one file may be included with
diff --git a/ciscosparkapi/api/people.py b/ciscosparkapi/api/people.py
@@ -369,7 +369,7 @@ def delete(self, personId):
             personId(basestring): The ID of the person to be deleted.
 
         Raises:
-            AssertionError: If the parameter types are incorrect.
+            TypeError: If the parameter types are incorrect.
             SparkApiError: If the Cisco Spark cloud returns an error.
 
         """
