diff options
author | Martin Basti <mbasti@redhat.com> | 2016-09-22 14:12:14 +0200 |
---|---|---|
committer | Martin Babinsky <mbabinsk@redhat.com> | 2016-09-22 15:22:56 +0200 |
commit | 929086e0992cc32a654b4dfa435f536ecb0c665b (patch) | |
tree | 87246ef804df56a6e6208863181129fe049914f5 /ipatests/test_xmlrpc | |
parent | 47c808afa35f0708ca00ac8e98851c9f8d75badc (diff) | |
download | freeipa-929086e0992cc32a654b4dfa435f536ecb0c665b.tar.gz freeipa-929086e0992cc32a654b4dfa435f536ecb0c665b.tar.xz freeipa-929086e0992cc32a654b4dfa435f536ecb0c665b.zip |
Test: dont use global variable for iteration in test_cert_plugin
Iteration over global variable causes unwanted value changes outside
method
https://fedorahosted.org/freeipa/ticket/5755
Reviewed-By: Martin Babinsky <mbabinsk@redhat.com>
Diffstat (limited to 'ipatests/test_xmlrpc')
-rw-r--r-- | ipatests/test_xmlrpc/test_cert_plugin.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/ipatests/test_xmlrpc/test_cert_plugin.py b/ipatests/test_xmlrpc/test_cert_plugin.py index fb4ab582d..2598e0b09 100644 --- a/ipatests/test_xmlrpc/test_cert_plugin.py +++ b/ipatests/test_xmlrpc/test_cert_plugin.py @@ -215,7 +215,9 @@ class test_cert(XMLRPC_test): res = api.Command['service_show'](self.service_princ)['result'] # Both the old and the new certs should be listed as certificates now - certs_encoded = (base64.b64encode(cert) for cert in res['usercertificate']) + certs_encoded = ( + base64.b64encode(usercert) for usercert in res['usercertificate'] + ) assert set(certs_encoded) == set([cert, newcert]) def test_0008_cleanup(self): |