Change how the id parameter is being handled and include the related test

Author: jbogarin

ciscosparkapi/api/people.py

@@ -177,15 +177,14 @@ def list(self, email=None, displayName=None, orgId=None, id=None, max=None, **qu
         params = {}
         if id:
             params["id"] = id
-        else:
-            if email:
+        elif email:
                 params['email'] = email
-            elif displayName:
-                params['displayName'] = displayName
-            if orgId:
-                params["orgId"] = orgId
-            if max:
-                params['max'] = max
+        elif displayName:
+            params['displayName'] = displayName
+        if orgId:
+            params["orgId"] = orgId
+        if max:
+            params['max'] = max
         # Process query_param keyword arguments
         if query_params:
             params.update(query_params)

tests/api/test_people.py

@@ -180,6 +180,12 @@ def test_list_people_by_display_name(self, api, test_people):
         assert len(list_of_people) >= 1
         assert are_valid_people(list_of_people)
 
+    def test_list_people_by_id(self, api, test_people):
+        id = test_people["not_a_member"].id
+        list_of_people = list_people(api, id=id)
+        assert len(list_of_people) >= 1
+        assert are_valid_people(list_of_people)
+
     def test_list_people_with_paging(self, api, test_people,
                                      additional_group_room_memberships):
         page_size = 1