Update Python syntax for machine readability

- Treat `print()` only as a function
- Treat `del` only as a keyword
- Use modern octal notation
- Perform idiomatic set non-membership check

Signed-off-by: Robbie Harwood <rharwood@redhat.com>

Author: frozencemetery

tests/magtests.py

@@ -11,9 +11,10 @@
 import sys
 
 # check that we can import requests (for use in test scripts)
-import requests, requests_kerberos
-del(requests)
-del(requests_kerberos)
+import requests
+import requests_kerberos
+del requests
+del requests_kerberos
 
 def parse_args():
     parser = argparse.ArgumentParser(
@@ -209,7 +210,7 @@ def setup_test_certs(testdir, testenv, logfile):
     with open(opensslcnf, 'w+') as f:
         f.write(text)
 
-    print pkinit_key
+    print(pkinit_key)
     cmd = subprocess.Popen(["openssl", "genrsa", "-out", pkinit_key,
                             "2048"], stdout=logfile,
                            stderr=logfile, env=testenv,

tests/t_basic_k5_fail_second.py

@@ -23,7 +23,7 @@
     r = s.get(url)
     if r.status_code == 200:
         raise ValueError('Basic Auth: Got Success while expecting Error')
-    if not 'GSS ERROR' in r.text:
+    if 'GSS ERROR' not in r.text:
         raise ValueError('Basic Auth: Expected error variable is missing')
 
     url = 'http://%s:%s@%s/basic_auth_krb5/' % \

tests/t_spnego.py

@@ -19,5 +19,5 @@
         raise ValueError('gssapi_session not set')
 
     data = os.stat(os.environ['DELEGCCACHE'])
-    if data[ST_MODE] != 0100666:
+    if data[ST_MODE] != 0o100666:
         raise ValueError('Incorrect perm on ccache: %o' % data[ST_MODE])