[API] OPSAPS-35644 | Add update_user() method to python API client.

Bug: OPSAPS-35644

Issue:
Need update_user() method in python API client.

Implementation:
- Add update_user() python API client.
- Update mock unit test.

Reviewer: alexm, jayesh

Tests:
- Ran updated unit test.

- Wrote a python client program to test update
operation on user.

from cm_api.api_client import ApiResource
from cm_api.endpoints.users import ApiUser

CM_HOST = "bankim-agent-1.vpc.cloudera.com"
CM_PORT = 7180
CM_USERNAME = "admin"
CM_PASSWD = "admin"

api = ApiResource(CM_HOST, CM_PORT, CM_USERNAME, CM_PASSWD, version=7)
for u in api.get_all_users():
  print u.name
  print ','.join(u.roles)
  if u.name == "guest":
    n = ApiUser(api, u.name, roles=[ 'ROLE_LIMITED' ])
    api.update_user(n)

for u in api.get_all_users():
  print u.name
  print ','.join(u.roles)

bankim@osboxes:~/scripts$ python user.py
admin
ROLE_ADMIN
guest
ROLE_USER
admin
ROLE_ADMIN
guest
ROLE_LIMITED

Author: bbhavsar

python/src/cm_api/api_client.py

@@ -218,6 +218,15 @@ def delete_user(self, username):
     """
     return users.delete_user(self, username)
 
+  def update_user(self, user):
+    """
+    Update user with the supplied new user object.
+
+    @param user: ApiUser object to be applied
+    @return: An ApiUser object
+    """
+    return users.update_user(self, user)
+
   # Events
 
   def query_events(self, query_str = None):

python/src/cm_api_tests/test_users.py

@@ -33,7 +33,7 @@ def test_grant_admin(self):
   def test_revoke_admin(self):
     res = utils.MockResource(self)
 
-    user = ApiUser(res, "alice")
+    user = ApiUser(res, "alice", roles=[ 'ROLE_ADMIN' ] )
     expected = { 'name' : 'alice', 'roles' : [ ] }
     res.expect("PUT", "/users/alice", data=expected, retdata=expected)
     updated = user.revoke_admin_role()
@@ -42,8 +42,11 @@ def test_revoke_admin(self):
   def test_update_user(self):
     res = utils.MockResource(self)
     user = ApiUser(res, "alice", roles=[ 'ROLE_LIMITED' ])
-    expected = { 'name' : 'alice', 'roles' : [ 'ROLE_LIMITED'] }
+
+    # Update user role
+    new_user = ApiUser(res, "alice", roles=[ 'ROLE_USER' ])
+    expected = { 'name' : 'alice', 'roles' : [ 'ROLE_USER'] }
     res.expect("PUT", "/users/alice", data=expected, retdata=expected)
-    updated = update_user(res, user)
-    self.assertTrue('ROLE_LIMITED' in updated.roles)
+    updated = update_user(res, new_user)
+    self.assertTrue('ROLE_USER' in updated.roles)
     self.assertEqual(1, len(updated.roles))