From 7dad32842451e3f01309d7ca61eccc874f3f9eef Mon Sep 17 00:00:00 2001 From: "Brian C. Lane" Date: Tue, 8 Jan 2019 11:38:26 -0800 Subject: [PATCH] Remove unneeded else from for/else loop. It confuses pylint (cherry picked from commit 2950f2641bdf8faa8e18c857698597a06a6057a3) Related: rhbz#1664485 --- tests/pylorax/test_recipes.py | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/tests/pylorax/test_recipes.py b/tests/pylorax/test_recipes.py index dcc22a83..9156934e 100644 --- a/tests/pylorax/test_recipes.py +++ b/tests/pylorax/test_recipes.py @@ -375,24 +375,21 @@ class CustomizationsTests(unittest.TestCase): for user in ks.handler.user.userList: if user.name == username: return user - else: - return None + return None @staticmethod def _find_sshkey(ks, username): for key in ks.handler.sshkey.sshUserList: if key.username == username: return key - else: - return None + return None @staticmethod def _find_group(ks, groupname): for group in ks.handler.group.groupList: if group.name == groupname: return group - else: - return None + return None def test_hostname(self): blueprint_data = """name = "test-hostname"