diff options
author | Simo Sorce <simo@redhat.com> | 2017-01-13 15:49:18 -0500 |
---|---|---|
committer | Simo Sorce <simo@redhat.com> | 2017-01-13 15:50:06 -0500 |
commit | 56d2a3119c4713fbfabf98b0afc0882d64324166 (patch) | |
tree | ab714cd42e19449c869df28feef74ee87ed4aacf | |
parent | 6f68b3a140172376d4b239bea11b48ba74b47786 (diff) | |
download | gss-proxy-56d2a3119c4713fbfabf98b0afc0882d64324166.tar.gz gss-proxy-56d2a3119c4713fbfabf98b0afc0882d64324166.tar.xz gss-proxy-56d2a3119c4713fbfabf98b0afc0882d64324166.zip |
Fix print_return logic
Signed-off-by: Simo Sorce <simo@redhat.com>
-rw-r--r-- | proxy/tests/testlib.py | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/proxy/tests/testlib.py b/proxy/tests/testlib.py index 01ffdc8..aa2c914 100644 --- a/proxy/tests/testlib.py +++ b/proxy/tests/testlib.py @@ -49,11 +49,12 @@ def print_warning(key, text, io=sys.stderr): print_keyed("other", key, text, io) def print_return(ret, name, expected_failure): - if ret != 0 and not expected_failure: - print_failure("SUCCESS" if ret == 0 else "FAILED", + if ((ret == 0 and expected_failure == False) or + (ret != 0 and expected_failure == True)): + print_success("SUCCESS" if ret == 0 else "FAILED", "%s test returned %s" % (name, str(ret))) else: - print_success("SUCCESS" if ret == 0 else "FAILED", + print_failure("SUCCESS" if ret == 0 else "FAILED", "%s test returned %s" % (name, str(ret))) WRAP_HOSTNAME = "kdc.gssproxy.dev" |