Correct test_update_person() and update_person() per API docs

**The Cisco Spark API docs state:**
“Include all details for the person. This action expects all user
details to be present in the request. A common approach is to first GET
the person's details, make changes, then PUT both the changed and
unchanged values.”

Update the test case to reflect this recommendation.

Given the simple, stateless and atomic wrapping of the API calls that
this package (ciscosparkapi) provides; don’t modify the package code to
provide this functionality.  Such stateful automations can be made in
the more feature-rich ciscosparksdk package.

Author: cmlccie

tests/api/test_people.py

@@ -45,7 +45,14 @@ def create_person(api, emails, **person_attributes):
 
 
 def update_person(api, person, **person_attributes):
-    return api.people.update(person.id, **person_attributes)
+    # Get a copy of the person's current attributes
+    new_attributes = person._json.copy()
+
+    # Merge in attribute updates
+    for attribute, value in person_attributes.items():
+        new_attributes[attribute] = value
+
+    return api.people.update(person.id, **new_attributes)
 
 
 def delete_person(api, person):
@@ -150,24 +157,16 @@ def test_create_person(self, test_people):
         person = test_people["not_a_member"]
         assert is_valid_person(person)
 
-    # TODO: Investigate update person API not working
-    # def test_update_person(self, api, temp_person, roles_dict, licenses_dict,
-    #                        get_new_email_address):
-    #     # Note:  Not testing updating orgId
-    #     updated_attributes = {
-    #         "emails": [get_new_email_address()],
-    #         "displayName": temp_person.displayName + " Updated",
-    #         "firstName": temp_person.firstName + " Updated",
-    #         "lastName": temp_person.lastName + " Updated",
-    #         "avatar": TEST_FILE_URL,
-    #         "roles": [roles_dict["Read-only administrator"].id],
-    #         "licenses": [licenses_dict["Messaging"].id,
-    #                      licenses_dict["Meeting 25 party"].id],
-    #     }
-    #     updated_person = update_person(api, temp_person, **updated_attributes)
-    #     assert is_valid_person(updated_person)
-    #     for attribute, value in updated_attributes:
-    #         assert getattr(updated_person, attribute, default=None) == value
+    def test_update_person(self, api, temp_person):
+        update_attributes = {
+            "displayName": temp_person.displayName + " Updated",
+            "firstName": temp_person.firstName + " Updated",
+            "lastName": temp_person.lastName + " Updated",
+        }
+        updated_person = update_person(api, temp_person, **update_attributes)
+        assert is_valid_person(updated_person)
+        for attribute, value in update_attributes.items():
+            assert getattr(updated_person, attribute) == value
 
     def test_get_my_details(self, me):
         assert is_valid_person(me)