diff options
| author | Martin Basti <mbasti@redhat.com> | 2016-09-22 14:39:16 +0200 |
|---|---|---|
| committer | Martin Basti <mbasti@redhat.com> | 2016-09-23 09:23:41 +0200 |
| commit | 9b68d2a1f858854bc3cf2d6024f3fd3d79c2f696 (patch) | |
| tree | 05b54fce4f950dc16051e2bf081f33698cd2a203 /ipatests/test_xmlrpc | |
| parent | 4d7d53c9664c9e68d7c6cda1a65cae7551423df7 (diff) | |
| download | freeipa-9b68d2a1f858854bc3cf2d6024f3fd3d79c2f696.tar.gz freeipa-9b68d2a1f858854bc3cf2d6024f3fd3d79c2f696.tar.xz freeipa-9b68d2a1f858854bc3cf2d6024f3fd3d79c2f696.zip | |
Pylint: enable global-variable-not-assigned check
the global keyword should be used only when variable from outside is
assigned inside, otherwise it has no effect and just confuses developers
Reviewed-By: Tomas Krizek <tkrizek@redhat.com>
Diffstat (limited to 'ipatests/test_xmlrpc')
| -rw-r--r-- | ipatests/test_xmlrpc/test_cert_plugin.py | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/ipatests/test_xmlrpc/test_cert_plugin.py b/ipatests/test_xmlrpc/test_cert_plugin.py index 2598e0b09..ab09d0aa4 100644 --- a/ipatests/test_xmlrpc/test_cert_plugin.py +++ b/ipatests/test_xmlrpc/test_cert_plugin.py @@ -166,8 +166,6 @@ class test_cert(XMLRPC_test): """ Verify that service-show has the right certificate using service-show. """ - global cert - res = api.Command['service_show'](self.service_princ)['result'] assert base64.b64encode(res['usercertificate'][0]) == cert @@ -175,8 +173,6 @@ class test_cert(XMLRPC_test): """ Verify that service-find has the right certificate using service-find. """ - global cert - # Assume there is only one service res = api.Command['service_find'](self.service_princ)['result'] assert base64.b64encode(res[0]['usercertificate'][0]) == cert @@ -210,8 +206,6 @@ class test_cert(XMLRPC_test): """ Verify the new certificate with service-show. """ - global cert, newcert - res = api.Command['service_show'](self.service_princ)['result'] # Both the old and the new certs should be listed as certificates now |