Pylint fixes

Recent pylint update is more pedantic about module level spacing. There
are no changes to code, only couple of extra spaces to make it happy.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

avocado/core/data_dir.py

@@ -208,6 +208,7 @@ class _TmpDirTracker(Borg):
             except AttributeError:
                 pass
 
+
 _tmp_tracker = _TmpDirTracker()
 
 

avocado/utils/archive.py

@@ -238,6 +238,7 @@ def uncompress(filename, path):
     with ArchiveFile.open(filename) as x:
         x.extract(path)
 
+
 # Some aliases
 create = compress
 extract = uncompress

avocado/utils/external/gdbmi_parser.py

@@ -354,6 +354,7 @@ def __private():
 
     return (GdbMiScanner(), GdbMiParser(), GdbMiInterpreter, GdbMiRecord)
 
+
 (__the_scanner, __the_parser, __the_interpreter, __the_output) = __private()
 
 

avocado/utils/service.py

@@ -785,6 +785,7 @@ def service_manager(run=process.run):
                                                     internal_parser(run))
         return _service_manager
 
+
 ServiceManager = service_manager
 
 
@@ -847,4 +848,5 @@ def specific_service_manager(service_name, run=process.run):
                                    specific_generator(run),
                                    _get_service_result_parser(run), run)
 
+
 SpecificServiceManager = specific_service_manager

contrib/job-scripts/coredump-handling/post.d/002-link-core-to-tests

@@ -66,5 +66,6 @@ def symlink_coredumps():
         except Exception:
             pass
 
+
 if __name__ == "__main__":
     symlink_coredumps()

examples/tests/doublefree_nasty.py

@@ -39,5 +39,6 @@ class DoubleFreeTest(Test):
         cmd_result = process.run(cmd)
         self.log.info(cmd_result)
 
+
 if __name__ == "__main__":
     main()

examples/tests/errortest.py

@@ -16,5 +16,6 @@ class ErrorTest(Test):
         """
         self.error('This should end with ERROR.')
 
+
 if __name__ == "__main__":
     main()

examples/tests/fail_on_exception.py

@@ -22,5 +22,6 @@ class FailOnException(avocado.Test):
         """
         raise ValueError('This raises a ValueError and should end as a FAIL')
 
+
 if __name__ == "__main__":
     avocado.main()

examples/tests/multiple_tests.py

@@ -33,5 +33,6 @@ class MultipleTests(Test):
         """
         raise Exception('This action method should never be executed.')
 
+
 if __name__ == '__main__':
     main()

examples/tests/skip_outside_setup.py

@@ -19,5 +19,6 @@ class SkipOutsideSetup(avocado.Test):
         """
         self.skip('Calling skip() outside setUp() will result in ERROR')
 
+
 if __name__ == "__main__":
     avocado.main()

examples/tests/skiponsetup.py

@@ -22,5 +22,6 @@ class SkipOnSetupTest(Test):
         """
         pass
 
+
 if __name__ == "__main__":
     main()

examples/tests/sleeptenmin.py

@@ -32,5 +32,6 @@ class SleepTenMin(Test):
             elif method == 'shell':
                 os.system("sleep %s" % length)
 
+
 if __name__ == "__main__":
     main()

examples/tests/uncaught_exception.py

@@ -16,5 +16,6 @@ class ErrorTest(Test):
         """
         raise Exception("This is a generic exception")
 
+
 if __name__ == "__main__":
     main()

examples/tests/warntest.py

@@ -16,5 +16,6 @@ class WarnTest(Test):
         """
         self.log.warn("This marks test as WARN")
 
+
 if __name__ == "__main__":
     main()

examples/tests/whiteboard.py

@@ -38,5 +38,6 @@ class WhiteBoard(Test):
             result += data
         self.whiteboard = base64.encodestring(result)
 
+
 if __name__ == "__main__":
     main()

scripts/avocado

@@ -55,6 +55,7 @@ def handle_exception(*exc_info):
     # imported because we are dealing with import failures
     sys.exit(-1)
 
+
 if __name__ == '__main__':
     sys.excepthook = handle_exception
 
@@ -66,8 +67,10 @@ if os.path.isdir(os.path.join(basedir, 'avocado')):
     os.environ['PATH'] += ":" + os.path.join(basedir, 'libexec')
     sys.path.append(basedir)
 
+
 from avocado.core.app import AvocadoApp
 
+
 if __name__ == '__main__':
     # Override tmp in case it's not set in env
     for attr in ("TMP", "TEMP", "TMPDIR"):

scripts/avocado-run-testplan

@@ -42,6 +42,7 @@ class Parser(argparse.ArgumentParser):
                                 'will show a human readable report for the '
                                 'given result file'))
 
+
 RESULT_MAP = {"P": "PASS",
               "p": "PASS",
               "F": "FAIL",
@@ -167,6 +168,7 @@ class App(object):
                 print("%s: %s" % (name, out))
         return 0
 
+
 if __name__ == '__main__':
     app = App()
     sys.exit(app.run())

selftests/functional/test_argument_parsing.py

@@ -72,5 +72,6 @@ class ArgumentParsingErrorEarlyTest(unittest.TestCase):
         self.run_but_fail_before_create_job_dir('--whacky-option passtest',
                                                 exit_codes.AVOCADO_FAIL)
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/functional/test_basic.py

@@ -105,6 +105,7 @@ def probe_binary(binary):
     except utils_path.CmdNotFoundError:
         return None
 
+
 CC_BINARY = probe_binary('cc')
 ECHO_BINARY = probe_binary('echo')
 READ_BINARY = probe_binary('read')
@@ -1152,5 +1153,6 @@ class PluginsJSONTest(AbsPluginsTest, unittest.TestCase):
         shutil.rmtree(self.tmpdir)
         super(PluginsJSONTest, self).tearDown()
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/functional/test_export_variables.py

@@ -62,5 +62,6 @@ class EnvironmentVariablesTest(unittest.TestCase):
         self.script.remove()
         shutil.rmtree(self.tmpdir)
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/functional/test_gdb.py

@@ -36,5 +36,6 @@ class GDBPluginTest(unittest.TestCase):
     def tearDown(self):
         shutil.rmtree(self.tmpdir)
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/functional/test_interrupt.py

@@ -182,5 +182,6 @@ class InterruptTest(unittest.TestCase):
     def tearDown(self):
         shutil.rmtree(self.tmpdir)
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/functional/test_job_timeout.py

@@ -168,5 +168,6 @@ class JobTimeOutTest(unittest.TestCase):
         self.py.remove()
         shutil.rmtree(self.tmpdir)
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/functional/test_multiplex.py

@@ -138,5 +138,6 @@ class MultiplexTests(unittest.TestCase):
     def tearDown(self):
         shutil.rmtree(self.tmpdir)
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/functional/test_output_check.py

@@ -118,5 +118,6 @@ class RunnerSimpleTest(unittest.TestCase):
         self.output_script.remove()
         shutil.rmtree(self.tmpdir)
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/functional/test_standalone.py

@@ -75,5 +75,6 @@ class StandaloneTests(unittest.TestCase):
         expected_rc = exit_codes.AVOCADO_TESTS_FAIL
         self.run_and_check(cmd_line, expected_rc, 'errortest')
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/functional/test_sysinfo.py

@@ -70,5 +70,6 @@ class SysInfoTest(unittest.TestCase):
     def tearDown(self):
         shutil.rmtree(self.tmpdir)
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/functional/test_thirdparty_bugs.py

@@ -28,5 +28,6 @@ class TestThirdPartyBugs(unittest.TestCase):
         except download.urllib2.URLError as details:
             raise unittest.SkipTest(details)
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/functional/test_unittest_compat.py

@@ -89,5 +89,6 @@ class UnittestCompat(unittest.TestCase):
         self.assertIn('This test is supposed to error', result.stderr)
         self.assertIn('FAILED (errors=1)', result.stderr)
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/run

@@ -26,6 +26,7 @@ def test_suite():
                                        top_level_dir=basedir))
     return suite
 
+
 if __name__ == '__main__':
     runner = unittest.TextTestRunner(failfast=not os.environ.get("SELF_CHECK_CONTINUOUS"))
     result = runner.run(test_suite())

selftests/unit/test_astring.py

@@ -39,5 +39,6 @@ class AstringTest(unittest.TestCase):
                       "avocado 123")
         self.assertEqual(astring.tabular_output(matrix), str_matrix)
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/unit/test_datadir.py

@@ -110,5 +110,6 @@ class DataDirTest(unittest.TestCase):
         if hasattr(self, 'alt_mapping'):
             shutil.rmtree(self.alt_mapping['base_dir'])
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/unit/test_dispatcher.py

@@ -23,5 +23,6 @@ class DispatcherTest(unittest.TestCase):
             self.assertEqual(names, sorted(names))
             self.assertEqual(ext_names, sorted(ext_names))
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/unit/test_job.py

@@ -119,5 +119,6 @@ class JobTest(unittest.TestCase):
         self.assertEqual(myjob.unique_id[::-1],
                          open(os.path.join(myjob.logdir, "reversed_id")).read())
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/unit/test_remote.py

@@ -182,5 +182,6 @@ class RemoteTestRunnerSetup(unittest.TestCase):
         self.runner.setup()
         flexmock_teardown()
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/unit/test_restclient_response.py

@@ -54,5 +54,6 @@ class ResultResponseTest(unittest.TestCase):
                           response.ResultResponse,
                           self.BAD_DATA_RESULTS)
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/unit/test_settings.py

@@ -101,5 +101,6 @@ class SettingsTest(unittest.TestCase):
     def tearDown(self):
         os.unlink(self.config_file.name)
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/unit/test_sysinfo.py

@@ -93,5 +93,6 @@ class SysinfoTest(unittest.TestCase):
     def tearDown(self):
         shutil.rmtree(self.tmpdir)
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/unit/test_test.py

@@ -249,5 +249,6 @@ class SkipTest(unittest.TestCase):
             except Exception:
                 pass
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/unit/test_tree.py

@@ -216,5 +216,6 @@ class TestPathParent(unittest.TestCase):
     def test_false_direct_parent(self):
         self.assertNotEqual(tree.path_parent('/os/linux'), '/')
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/unit/test_utils_asset.py

@@ -100,5 +100,6 @@ class TestAsset(unittest.TestCase):
     def tearDown(self):
         shutil.rmtree(self.basedir)
 
+
 if __name__ == "__main__":
     unittest.main()

selftests/unit/test_utils_filelock.py

@@ -36,5 +36,6 @@ class TestFileLock(unittest.TestCase):
     def tearDown(self):
         shutil.rmtree(self.basedir)
 
+
 if __name__ == "__main__":
     unittest.main()

selftests/unit/test_utils_process.py

@@ -218,5 +218,6 @@ class MiscProcessTests(unittest.TestCase):
         res = process.binary_from_shell_cmd("FOO=bar ./bin var=value")
         self.assertEqual("./bin", res)
 
+
 if __name__ == "__main__":
     unittest.main()

selftests/unit/test_utils_stacktrace.py

@@ -80,5 +80,6 @@ class TestUnpickableObject(unittest.TestCase):
                ListWithUnpickableAttribute(ListWithUnpickableAttribute(
                    [InClassUnpickable()]))])
 
+
 if __name__ == '__main__':
     unittest.main()

selftests/unit/test_vm.py

@@ -67,5 +67,6 @@ class VMTestRunnerSetup(unittest.TestCase):
         self.runner.tear_down()
         flexmock_teardown()
 
+
 if __name__ == '__main__':
     unittest.main()

setup.py

@@ -102,6 +102,7 @@ def get_long_description():
         req_contents = req.read()
     return req_contents
 
+
 if __name__ == '__main__':
     setup(name='avocado-framework',
           version=VERSION,