Update comments and docstrings

Author: cmlccie

ciscosparkapi/__init__.py

@@ -13,7 +13,9 @@ def __init__(self, access_token, base_url=None, timeout=None):
         session_args = {}
         if base_url:  session_args['base_url'] = base_url
         if timeout:  session_args['timeout'] = timeout
-        # Create API session
+        # Create API session - All of the API calls associated with a
+        # CiscoSparkAPI object will leverage a single RESTful 'session'
+        # connecting to the Cisco Spark cloud.
         self.session = RestSession(access_token, **session_args)
         # Setup Spark API wrappers
         self.rooms = RoomsAPI(self.session)

ciscosparkapi/api/rooms.py

@@ -1,4 +1,4 @@
-"""Spark - Rooms API - wrapper classes."""
+"""Cisco Spark rooms API wrapper classes."""
 
 
 from ciscosparkapi.exceptions import ciscosparkapiException
@@ -8,7 +8,7 @@
 
 
 class Room(SparkData):
-    """Spark Room-object wrapper class."""
+    """Spark room-object wrapper class."""
 
     def __init__(self, json):
         super(Room, self).__init__(json)
@@ -39,11 +39,18 @@ def created(self):
 
     @property
     def teamId(self):
+        """Return the room teamId, if it exists, otherwise return None.
+
+        teamId is an 'optional' attribute that only exists for Spark rooms that
+        are associated with a Spark Team.  To simplify use, rather than
+        requiring use of try/catch statements or hasattr() calls, we simply
+        return None if a room does not have a teamId attribute.
+        """
         return self._json.get(u'teamId', None)
 
 
 class RoomsAPI(object):
-    """Spark Rooms API request wrapper."""
+    """Cisco Spark rooms API request wrapper class."""
 
     def __init__(self, session):
         assert isinstance(session, RestSession)