From 36484e8672f5ee1fdc2bd57622e330ab8dbb7671 Mon Sep 17 00:00:00 2001 From: Jan Barta <55042barta@sstebrno.eu> Date: Thu, 2 Jun 2016 09:58:52 +0200 Subject: pylint: fix simplifiable-if-statement warnings fix inefficient if statements, enable pylint check Reviewed-By: Tomas Krizek Reviewed-By: Florence Blanc-Renaud --- ipatests/test_integration/tasks.py | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) (limited to 'ipatests') 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): -- cgit