summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPetr Viktorin <pviktori@redhat.com>2013-12-06 16:39:06 +0100
committerPetr Viktorin <pviktori@redhat.com>2013-12-10 15:42:52 +0100
commit5640049f7aa132ab73099db856b352c3413489bb (patch)
tree9c05b7eb7a617b3cd9fd1c7bb236799e63a28e8b
parente2625b66e2d46d402e58282d875da480676e6282 (diff)
downloadfreeipa.git-5640049f7aa132ab73099db856b352c3413489bb.tar.gz
freeipa.git-5640049f7aa132ab73099db856b352c3413489bb.tar.xz
freeipa.git-5640049f7aa132ab73099db856b352c3413489bb.zip
test_webui: Allow False values in configuration for no_ca, no_dns, has_trusts
The driver only checked if the corresponding value was in the config, so no_dns: False had the same effect as no_dns: True Change the check to take the value into consideration. This makes false-y values like False (from YAML) and empty string (from environment) work as if the value was not specified.
-rw-r--r--ipatests/test_webui/ui_driver.py6
1 files changed, 3 insertions, 3 deletions
diff --git a/ipatests/test_webui/ui_driver.py b/ipatests/test_webui/ui_driver.py
index 8603aa70..f58294af 100644
--- a/ipatests/test_webui/ui_driver.py
+++ b/ipatests/test_webui/ui_driver.py
@@ -241,19 +241,19 @@ class UI_driver(object):
"""
FreeIPA server was installed with CA.
"""
- return 'no_ca' not in self.config
+ return not self.config.get('no_ca')
def has_dns(self):
"""
FreeIPA server was installed with DNS.
"""
- return 'no_dns' not in self.config
+ return not self.config.get('no_dns')
def has_trusts(self):
"""
FreeIPA server was installed with Trusts.
"""
- return 'has_trusts' in self.config
+ return self.config.get('has_trusts')
def has_active_request(self):
"""