From f8874cdd5fa54472119a50039609bcd499dd7267 Mon Sep 17 00:00:00 2001
From: Mathieu Bultel <mbultel@redhat.com>
Date: Mon, 9 Mar 2020 14:26:04 +0100
Subject: [PATCH] Fix pep8 lint for unit tests

---
 validations_libs/tests/test_ansible.py        | 21 +++++++------------
 .../tests/test_validations_run.py             | 14 ++++++-------
 2 files changed, 15 insertions(+), 20 deletions(-)

diff --git a/validations_libs/tests/test_ansible.py b/validations_libs/tests/test_ansible.py
index cf9c200d..02a4009b 100644
--- a/validations_libs/tests/test_ansible.py
+++ b/validations_libs/tests/test_ansible.py
@@ -24,7 +24,6 @@ from validations_libs import utils
 
 class TestAnsible(TestCase):
 
-
     def setUp(self):
         super(TestAnsible, self).setUp()
         self.unlink_patch = mock.patch('os.unlink')
@@ -45,7 +44,6 @@ class TestAnsible(TestCase):
         )
         mock_exists.assert_called_with('/tmp/non-existing.yaml')
 
-
     @mock.patch('tempfile.mkdtemp', return_value='/tmp/')
     @mock.patch('os.path.exists', return_value=True)
     @mock.patch('os.makedirs')
@@ -64,17 +62,16 @@ class TestAnsible(TestCase):
                                   mock_mkdtemp):
 
         stdout_file, _playbook, _rc, _status = self.run.run('existing.yaml',
-            'localhost,',
-            '/tmp')
+                                                            'localhost,',
+                                                            '/tmp')
         self.assertEquals((_playbook, _rc, _status),
                           ('existing.yaml', 1, 'failed'))
 
-
     @mock.patch('tempfile.mkdtemp', return_value='/tmp/')
     @mock.patch('os.path.exists', return_value=True)
     @mock.patch('os.makedirs')
-    @mock.patch.object(Runner,'run',
-        return_value=fakes.fake_ansible_runner_run_return(rc=0)
+    @mock.patch.object(Runner, 'run',
+                       return_value=fakes.fake_ansible_runner_run_return(rc=0)
     )
     @mock.patch('ansible_runner.utils.dump_artifact', autospec=True,
                 return_value="/foo/inventory.yaml")
@@ -91,12 +88,11 @@ class TestAnsible(TestCase):
         self.assertEquals((_playbook, _rc, _status),
                           ('existing.yaml', 0, 'successful'))
 
-
     @mock.patch('tempfile.mkdtemp', return_value='/tmp/')
     @mock.patch('os.path.exists', return_value=True)
     @mock.patch('os.makedirs')
-    @mock.patch.object(Runner,'run',
-        return_value=fakes.fake_ansible_runner_run_return(rc=0)
+    @mock.patch.object(Runner, 'run',
+                       return_value=fakes.fake_ansible_runner_run_return(rc=0)
     )
     @mock.patch('ansible_runner.utils.dump_artifact', autospec=True,
                 return_value="/foo/inventory.yaml")
@@ -116,12 +112,11 @@ class TestAnsible(TestCase):
         self.assertEquals((_playbook, _rc, _status),
                           ('existing.yaml', 0, 'successful'))
 
-
     @mock.patch('tempfile.mkdtemp', return_value='/tmp/')
     @mock.patch('os.path.exists', return_value=True)
     @mock.patch('os.makedirs')
-    @mock.patch.object(Runner,'run',
-        return_value=fakes.fake_ansible_runner_run_return(rc=0)
+    @mock.patch.object(Runner, 'run',
+                       return_value=fakes.fake_ansible_runner_run_return(rc=0)
     )
     @mock.patch('ansible_runner.utils.dump_artifact', autospec=True,
                 return_value="/foo/inventory.yaml")
diff --git a/validations_libs/tests/test_validations_run.py b/validations_libs/tests/test_validations_run.py
index a8c1ead2..847d24f3 100644
--- a/validations_libs/tests/test_validations_run.py
+++ b/validations_libs/tests/test_validations_run.py
@@ -40,9 +40,9 @@ class TestValidatorRun(TestCase):
 
         expected_run_return = [
             {'validation': {'playbook': 'foo.yaml',
-            'rc_code': 0,
-            'status': 'successful',
-            'stdout_file': '/tmp/validation/stdout.log'}}]
+                            'rc_code': 0,
+                            'status': 'successful',
+                            'stdout_file': '/tmp/validation/stdout.log'}}]
 
         playbook = ['fake.yaml']
         inventory = 'tmp/inventory.yaml'
@@ -56,7 +56,7 @@ class TestValidatorRun(TestCase):
     @mock.patch('validations_libs.utils.parse_all_validations_on_disk')
     @mock.patch('validations_libs.ansible.Ansible.run')
     def test_validation_run_failed(self, mock_ansible_run,
-                                    mock_validation_dir):
+                                   mock_validation_dir):
         mock_validation_dir.return_value = [{
             'description': 'My Validation One Description',
             'groups': ['prep', 'pre-deployment'],
@@ -68,9 +68,9 @@ class TestValidatorRun(TestCase):
 
         expected_run_return = [
             {'validation': {'playbook': 'foo.yaml',
-            'rc_code': 0,
-            'status': 'failed',
-            'stdout_file': '/tmp/validation/stdout.log'}}]
+                            'rc_code': 0,
+                            'status': 'failed',
+                            'stdout_file': '/tmp/validation/stdout.log'}}]
 
         playbook = ['fake.yaml']
         inventory = 'tmp/inventory.yaml'