diff options
author | Petr Viktorin <pviktori@redhat.com> | 2013-12-06 16:39:06 +0100 |
---|---|---|
committer | Petr Viktorin <pviktori@redhat.com> | 2013-12-10 15:42:52 +0100 |
commit | 5640049f7aa132ab73099db856b352c3413489bb (patch) | |
tree | 9c05b7eb7a617b3cd9fd1c7bb236799e63a28e8b | |
parent | e2625b66e2d46d402e58282d875da480676e6282 (diff) | |
download | freeipa.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.py | 6 |
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): """ |