cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

aci_aaa_user: Fix changing user description

Browse source 
This fixes a reported problem with the aci_aaa_user module.
This commit is contained in:
Dag Wieers 2019-01-29 02:34:27 +01:00 committed by Toshio Kuratomi
parent c372d95c69
commit 0d5f083a52
3 changed files with 13 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
changelogs/fragments/51406-aci_aaa_user-description.yaml Normal file
View file

@ -0,0 +1,2 @@
bugfixes:
- aci_aaa_user - Fix setting user description (https://github.com/ansible/ansible/issues/51406)

1
lib/ansible/modules/network/aci/aci_aaa_user.py
View file

@ -319,6 +319,7 @@ def main():
class_config=dict( class_config=dict(
accountStatus=enabled, accountStatus=enabled,
clearPwdHistory=clear_password_history, clearPwdHistory=clear_password_history,
descr=description,
email=email, email=email,
expiration=expiration, expiration=expiration,
expires=expires, expires=expires,

10
test/integration/targets/aci_aaa_user/tasks/main.yml
View file

@ -66,8 +66,10 @@
that: that:
- cm_add_user is changed - cm_add_user is changed
- nm_add_user is changed - nm_add_user is changed
- nm_add_user.current.0.aaaUser.attributes.descr == 'Ansible test user'
- cm_add_user_again is not changed - cm_add_user_again is not changed
- nm_add_user_again is not changed - nm_add_user_again is not changed
- nm_add_user_again.current.0.aaaUser.attributes.descr == 'Ansible test user'
# MODIFY USER # MODIFY USER
@ -107,8 +109,10 @@
that: that:
- cm_modify_user is changed - cm_modify_user is changed
- nm_modify_user is changed - nm_modify_user is changed
- nm_modify_user.current.0.aaaUser.attributes.descr == 'Ansible test user for integration tests'
- cm_modify_user_again is not changed - cm_modify_user_again is not changed
- nm_modify_user_again is not changed - nm_modify_user_again is not changed
- nm_modify_user_again.current.0.aaaUser.attributes.descr == 'Ansible test user for integration tests'
# QUERY ALL USERS # QUERY ALL USERS
@ -157,6 +161,12 @@
- cm_query_user is not changed - cm_query_user is not changed
- nm_query_user is not changed - nm_query_user is not changed
- cm_query_user == nm_query_user - cm_query_user == nm_query_user
- nm_query_user.current.0.aaaUser.attributes.accountStatus == 'active'
- nm_query_user.current.0.aaaUser.attributes.descr == 'Ansible test user for integration tests'
- nm_query_user.current.0.aaaUser.attributes.email == 'aci-ansible@ansible.lan'
- nm_query_user.current.0.aaaUser.attributes.expiration == '2123-12-12T00:00:00.000+00:00'
- nm_query_user.current.0.aaaUser.attributes.expires == 'yes'
- nm_query_user.current.0.aaaUser.attributes.phone == '2-345-555-678'
# REMOVE USER # REMOVE USER