added_UT's

mahendras25 · Apr 29, 2021 · 7a149ee · 7a149ee
1 parent 357983b
commit 7a149ee
Show file tree

Hide file tree

Showing 5 changed files with 296 additions and 32 deletions.
diff --git a/examples/oneview_user_facts.yml b/examples/oneview_user_facts.yml
@@ -51,7 +51,7 @@
         role: "{{ role }}"
       delegate_to: localhost
 
-    - debug: var=users
+    - debug: var=role
 
     - name: Gather facts about lists of user's roles
       oneview_user_facts:
@@ -62,3 +62,4 @@
       delegate_to: localhost
 
     - debug: var=users
+    - debug: var=user_roles
diff --git a/library/oneview_user.py b/library/oneview_user.py
@@ -238,7 +238,7 @@ class UserModule(OneViewModule):
     MSG_USERNAME_MISSING = 'userName field is missing.'
     MSG_USERNAME_DOES_NOT_EXIT = 'userName doesn\'t exist.'
     MSG_ROLELIST_MISSING = 'role_list field is missing.'
-    MSG_PASSWORD_MISSING =  'either oldPassword or newPassword field is missing.'
+    MSG_PASSWORD_MISSING = 'either oldPassword or newPassword field is missing.'
     MSG_ROLENAME_MISSING = 'roleName field is missing.'
     MSG_USERLIST_MISSING = 'users_list field is missing.'
     MSG_FULLNAME_MISSING = 'fullName field is missing.'
@@ -253,7 +253,7 @@ def __init__(self):
                                                 'set_password']))
 
         super().__init__(additional_arg_spec=additional_arg_spec,
-                                         validate_etag_support=True)
+                         validate_etag_support=True)
 
         self.resource_client = self.oneview_client.users
 

diff --git a/library/oneview_user_facts.py b/library/oneview_user_facts.py
@@ -107,7 +107,7 @@ def __init__(self):
             options=dict(required=False, type='list')
         )
 
-        super(UserFactsModule, self).__init__(additional_arg_spec=argument_spec)
+        super().__init__(additional_arg_spec=argument_spec)
         self.set_resource_object(self.oneview_client.users)
 
     def execute_module(self):
@@ -124,8 +124,7 @@ def execute_module(self):
         if self.options.get('getUserRoles'):
             ansible_facts['user_roles'] = self.resource_client.get_role_associated_with_userName(self.module.params['userName'])
 
-
-        return dict(changed=False, ansible_facts=dict(users=ansible_facts))
+        return dict(changed=False, ansible_facts=ansible_facts)
 
 
 def main():
@@ -134,4 +133,3 @@ def main():
 
 if __name__ == '__main__':
     main()
-