diff options
| author | Jan Barta <55042barta@sstebrno.eu> | 2016-06-02 09:58:52 +0200 |
|---|---|---|
| committer | Martin Basti <mbasti@redhat.com> | 2016-09-22 16:52:57 +0200 |
| commit | 36484e8672f5ee1fdc2bd57622e330ab8dbb7671 (patch) | |
| tree | 1f06a3e4aebbc0e6e7fe28e15c9e677e396932a3 /ipatests | |
| parent | 929086e0992cc32a654b4dfa435f536ecb0c665b (diff) | |
| download | freeipa-36484e8672f5ee1fdc2bd57622e330ab8dbb7671.tar.gz freeipa-36484e8672f5ee1fdc2bd57622e330ab8dbb7671.tar.xz freeipa-36484e8672f5ee1fdc2bd57622e330ab8dbb7671.zip | |
pylint: fix simplifiable-if-statement warnings
fix inefficient if statements, enable pylint check
Reviewed-By: Tomas Krizek <tkrizek@redhat.com>
Reviewed-By: Florence Blanc-Renaud <frenaud@redhat.com>
Diffstat (limited to 'ipatests')
| -rw-r--r-- | ipatests/test_integration/tasks.py | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/ipatests/test_integration/tasks.py b/ipatests/test_integration/tasks.py index b39df34a5..6b3082a78 100644 --- a/ipatests/test_integration/tasks.py +++ b/ipatests/test_integration/tasks.py @@ -172,11 +172,7 @@ def host_service_active(host, service): res = host.run_command(['systemctl', 'is-active', '--quiet', service], raiseonerr=False) - if res.returncode == 0: - return True - else: - return False - + return res.returncode == 0 def fix_apache_semaphores(master): systemd_available = master.transport.file_exists(paths.SYSTEMCTL) @@ -325,11 +321,7 @@ def master_authoritative_for_client_domain(master, client): zone = ".".join(client.hostname.split('.')[1:]) result = master.run_command(["ipa", "dnszone-show", zone], raiseonerr=False) - if result.returncode == 0: - return True - else: - return False - + return result.returncode == 0 def replica_prepare(master, replica, extra_args=(), raiseonerr=True, stdin_text=None): |