Merge remote-tracking branch 'clebergnu/selftests_reorg_v2'

Signed-off-by: Cleber Rosa <[email protected]>
avocado-framework · Mar 31, 2021 · 78e6c68 · 78e6c68
2 parents 933c920 + e07373d
commit 78e6c68
Show file tree

Hide file tree

Showing 50 changed files with 38 additions and 37 deletions.
diff --git a/examples/jobs/nrunner.py b/examples/jobs/nrunner.py
@@ -13,7 +13,7 @@
     'nrunner.status_server_listen': status_server,
     'nrunner.status_server_uri': status_server,
     'run.references': [
-        'selftests/unit/test_resolver.py',
+        'selftests/unit/plugin/test_resolver.py',
         'selftests/functional/test_argument_parsing.py',
         '/bin/true',
     ],

diff --git a/selftests/check.py b/selftests/check.py
@@ -498,9 +498,9 @@ def create_suites(args):
     # Run all static checks, unit and functional tests
     # ========================================================================
     config_check = {
-        'run.references': (glob.glob('selftests/jobs/*') +
-                           glob.glob('selftests/unit/*.py') +
-                           glob.glob('selftests/functional/*.py')),
+        'run.references': ['selftests/jobs/',
+                           'selftests/unit/',
+                           'selftests/functional/'],
         'run.test_runner': 'nrunner',
         'run.ignore_missing_references': True
     }

diff --git a/selftests/functional/test_plugin_assets.py → selftests/functional/plugin/test_assets.py b/selftests/functional/test_plugin_assets.py → selftests/functional/plugin/test_assets.py
@@ -11,8 +11,8 @@
 from avocado.core import exit_codes
 from avocado.utils import process
 
-from .. import (AVOCADO, TestCaseTmpDir, get_temporary_config,
-                skipUnlessPathExists)
+from ... import (AVOCADO, TestCaseTmpDir, get_temporary_config,
+                 skipUnlessPathExists)
 
 TEST_TEMPLATE = r"""
 from avocado import Test

diff --git a/selftests/functional/test_plugin_diff.py → selftests/functional/plugin/test_diff.py b/selftests/functional/test_plugin_diff.py → selftests/functional/plugin/test_diff.py
@@ -6,7 +6,7 @@
 from avocado.core import exit_codes
 from avocado.utils import process, script
 
-from .. import AVOCADO, BASEDIR, TestCaseTmpDir
+from ... import AVOCADO, BASEDIR, TestCaseTmpDir
 
 
 class DiffTests(TestCaseTmpDir):

diff --git a/...ests/functional/test_plugin_jobscripts.py → ...ests/functional/plugin/test_jobscripts.py b/...ests/functional/test_plugin_jobscripts.py → ...ests/functional/plugin/test_jobscripts.py
@@ -4,7 +4,7 @@
 from avocado.core import exit_codes
 from avocado.utils import process, script
 
-from .. import AVOCADO, TestCaseTmpDir
+from ... import AVOCADO, TestCaseTmpDir
 
 SCRIPT_PRE_TOUCH = """#!/bin/sh -e
 touch %s"""

diff --git a/selftests/functional/test_plugin_vmimage.py → selftests/functional/plugin/test_vmimage.py b/selftests/functional/test_plugin_vmimage.py → selftests/functional/plugin/test_vmimage.py
@@ -5,7 +5,7 @@
 from avocado.core import exit_codes
 from avocado.utils import path, process
 
-from .. import AVOCADO, get_temporary_config
+from ... import AVOCADO, get_temporary_config
 
 
 def missing_binary(binary):

diff --git a/selftests/unit/test_plugin_assets.py → selftests/unit/plugin/test_assets.py b/selftests/unit/test_plugin_assets.py → selftests/unit/plugin/test_assets.py
diff --git a/selftests/unit/test_jsonresult.py → selftests/unit/plugin/test_jsonresult.py b/selftests/unit/test_jsonresult.py → selftests/unit/plugin/test_jsonresult.py
@@ -7,7 +7,7 @@
 from avocado.core.result import Result
 from avocado.plugins import jsonresult
 
-from .. import TestCaseTmpDir, setup_avocado_loggers
+from ... import TestCaseTmpDir, setup_avocado_loggers
 
 setup_avocado_loggers()
 

diff --git a/selftests/unit/test_legacy_replay.py → selftests/unit/plugin/test_legacy_replay.py b/selftests/unit/test_legacy_replay.py → selftests/unit/plugin/test_legacy_replay.py
@@ -5,7 +5,7 @@
 from avocado.core import test
 from avocado.plugins.legacy import replay as replay_legacy
 
-from .. import setup_avocado_loggers, temp_dir_prefix
+from ... import setup_avocado_loggers, temp_dir_prefix
 
 setup_avocado_loggers()
 

diff --git a/selftests/unit/test_resolver.py → selftests/unit/plugin/test_resolver.py b/selftests/unit/test_resolver.py → selftests/unit/plugin/test_resolver.py
@@ -6,7 +6,7 @@
                                        ExecTestResolver)
 from avocado.utils import script
 
-from .. import BASEDIR
+from ... import BASEDIR
 
 
 class ReferenceResolution(unittest.TestCase):

diff --git a/selftests/unit/test_spawner.py → selftests/unit/plugin/test_spawner.py b/selftests/unit/test_spawner.py → selftests/unit/plugin/test_spawner.py
diff --git a/selftests/unit/test_plugin_vmimage.py → selftests/unit/plugin/test_vmimage.py b/selftests/unit/test_plugin_vmimage.py → selftests/unit/plugin/test_vmimage.py
@@ -8,9 +8,9 @@
 from avocado.plugins import vmimage as vmimage_plugin
 from avocado.utils import vmimage as vmimage_util
 
-from .. import skipOnLevelsInferiorThan, temp_dir_prefix
-from ..functional.test_plugin_vmimage import (create_metadata_file,
-                                              missing_binary)
+from ... import skipOnLevelsInferiorThan, temp_dir_prefix
+from ...functional.plugin.test_vmimage import (create_metadata_file,
+                                               missing_binary)
 
 #: extracted from https://dl.fedoraproject.org/pub/fedora/linux/releases/
 FEDORA_PAGE = """<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">

diff --git a/selftests/unit/test_xunit.py → selftests/unit/plugin/test_xunit.py b/selftests/unit/test_xunit.py → selftests/unit/plugin/test_xunit.py
@@ -8,7 +8,7 @@
 from avocado.core.result import Result
 from avocado.plugins import xunit
 
-from .. import setup_avocado_loggers, temp_dir_prefix
+from ... import setup_avocado_loggers, temp_dir_prefix
 
 try:
     import xmlschema
@@ -93,9 +93,10 @@ def test_add_success(self):
         self.assertEqual(len(els), 1)
         self.assertEqual(els[0].attributes['time'].value, '678.237')
 
-        junit_xsd = os.path.abspath(os.path.join(os.path.dirname(__file__),
-                                                 os.path.pardir, ".data",
-                                                 'jenkins-junit.xsd'))
+        junit_xsd = os.path.abspath(os.path.join(
+            os.path.dirname(os.path.dirname(__file__)),
+            os.path.pardir, ".data",
+            'jenkins-junit.xsd'))
         xml_schema = xmlschema.XMLSchema(junit_xsd)
         self.assertTrue(xml_schema.is_valid(xunit_output))
 

diff --git a/selftests/unit/test_utils_astring.py → selftests/unit/utils/test_astring.py b/selftests/unit/test_utils_astring.py → selftests/unit/utils/test_astring.py
diff --git a/selftests/unit/test_utils_cloudinit.py → selftests/unit/utils/test_cloudinit.py b/selftests/unit/test_utils_cloudinit.py → selftests/unit/utils/test_cloudinit.py
@@ -6,7 +6,7 @@
 
 from avocado.utils import cloudinit, data_factory, iso9660
 
-from .. import setup_avocado_loggers, temp_dir_prefix
+from ... import setup_avocado_loggers, temp_dir_prefix
 
 setup_avocado_loggers()
 

diff --git a/selftests/unit/test_utils_cpu.py → selftests/unit/utils/test_cpu.py b/selftests/unit/test_utils_cpu.py → selftests/unit/utils/test_cpu.py
diff --git a/selftests/unit/test_utils_cpu.py.data/armv7 → selftests/unit/utils/test_cpu.py.data/armv7 b/selftests/unit/test_utils_cpu.py.data/armv7 → selftests/unit/utils/test_cpu.py.data/armv7
diff --git a/selftests/unit/test_utils_cpu.py.data/armv8 → selftests/unit/utils/test_cpu.py.data/armv8 b/selftests/unit/test_utils_cpu.py.data/armv8 → selftests/unit/utils/test_cpu.py.data/armv8
diff --git a/selftests/unit/test_utils_cpu.py.data/i386 → selftests/unit/utils/test_cpu.py.data/i386 b/selftests/unit/test_utils_cpu.py.data/i386 → selftests/unit/utils/test_cpu.py.data/i386
diff --git a/selftests/unit/test_utils_cpu.py.data/power8 → selftests/unit/utils/test_cpu.py.data/power8 b/selftests/unit/test_utils_cpu.py.data/power8 → selftests/unit/utils/test_cpu.py.data/power8
diff --git a/selftests/unit/test_utils_cpu.py.data/power9 → selftests/unit/utils/test_cpu.py.data/power9 b/selftests/unit/test_utils_cpu.py.data/power9 → selftests/unit/utils/test_cpu.py.data/power9
diff --git a/selftests/unit/test_utils_cpu.py.data/risc_v → selftests/unit/utils/test_cpu.py.data/risc_v b/selftests/unit/test_utils_cpu.py.data/risc_v → selftests/unit/utils/test_cpu.py.data/risc_v
diff --git a/selftests/unit/test_utils_cpu.py.data/s390x → selftests/unit/utils/test_cpu.py.data/s390x b/selftests/unit/test_utils_cpu.py.data/s390x → selftests/unit/utils/test_cpu.py.data/s390x
diff --git a/...tests/unit/test_utils_cpu.py.data/s390x_2 → ...tests/unit/utils/test_cpu.py.data/s390x_2 b/...tests/unit/test_utils_cpu.py.data/s390x_2 → ...tests/unit/utils/test_cpu.py.data/s390x_2
diff --git a/...tests/unit/test_utils_cpu.py.data/s390x_3 → ...tests/unit/utils/test_cpu.py.data/s390x_3 b/...tests/unit/test_utils_cpu.py.data/s390x_3 → ...tests/unit/utils/test_cpu.py.data/s390x_3
diff --git a/selftests/unit/test_utils_cpu.py.data/x86_64 → selftests/unit/utils/test_cpu.py.data/x86_64 b/selftests/unit/test_utils_cpu.py.data/x86_64 → selftests/unit/utils/test_cpu.py.data/x86_64
diff --git a/selftests/unit/test_utils_datadrainer.py → selftests/unit/utils/test_datadrainer.py b/selftests/unit/test_utils_datadrainer.py → selftests/unit/utils/test_datadrainer.py
diff --git a/selftests/unit/test_utils_diff_validator.py → selftests/unit/utils/test_diff_validator.py b/selftests/unit/test_utils_diff_validator.py → selftests/unit/utils/test_diff_validator.py
@@ -6,7 +6,7 @@
 
 from avocado.utils import diff_validator
 
-from .. import temp_dir_prefix
+from ... import temp_dir_prefix
 
 
 class ChangeValidationTest(unittest.TestCase):

diff --git a/selftests/unit/test_utils_disk.py → selftests/unit/utils/test_disk.py b/selftests/unit/test_utils_disk.py → selftests/unit/utils/test_disk.py
diff --git a/selftests/unit/test_utils_filelock.py → selftests/unit/utils/test_filelock.py b/selftests/unit/test_utils_filelock.py → selftests/unit/utils/test_filelock.py
@@ -4,7 +4,7 @@
 
 from avocado.utils.filelock import AlreadyLocked, FileLock
 
-from .. import temp_dir_prefix
+from ... import temp_dir_prefix
 
 
 class TestFileLock(unittest.TestCase):

diff --git a/selftests/unit/test_utils_genio.py → selftests/unit/utils/test_genio.py b/selftests/unit/test_utils_genio.py → selftests/unit/utils/test_genio.py
@@ -6,7 +6,7 @@
 
 from avocado.utils import genio
 
-from .. import setup_avocado_loggers, temp_dir_prefix
+from ... import setup_avocado_loggers, temp_dir_prefix
 
 setup_avocado_loggers()
 

diff --git a/selftests/unit/test_utils_iso9660.py → selftests/unit/utils/test_iso9660.py b/selftests/unit/test_utils_iso9660.py → selftests/unit/utils/test_iso9660.py
@@ -7,17 +7,17 @@
 
 from avocado.utils import iso9660, process
 
-from .. import setup_avocado_loggers, temp_dir_prefix
+from ... import setup_avocado_loggers, temp_dir_prefix
 
 setup_avocado_loggers()
 
 
 class Capabilities(unittest.TestCase):
 
     def setUp(self):
-        self.iso_path = os.path.abspath(os.path.join(os.path.dirname(__file__),
-                                                     os.path.pardir, ".data",
-                                                     "sample.iso"))
+        self.iso_path = os.path.abspath(
+            os.path.join(os.path.dirname(os.path.dirname(__file__)),
+                         os.path.pardir, ".data", "sample.iso"))
 
     @unittest.mock.patch('avocado.utils.iso9660.has_pycdlib', return_value=True)
     def test_capabilities_pycdlib(self, has_pycdlib_mocked):
@@ -49,9 +49,9 @@ class BaseIso9660:
     """
 
     def setUp(self):
-        self.iso_path = os.path.abspath(os.path.join(os.path.dirname(__file__),
-                                                     os.path.pardir, ".data",
-                                                     "sample.iso"))
+        self.iso_path = os.path.abspath(
+            os.path.join(os.path.dirname(os.path.dirname(__file__)),
+                         os.path.pardir, ".data", "sample.iso"))
         self.iso = None
         prefix = temp_dir_prefix(__name__, self, 'setUp')
         self.tmpdir = tempfile.TemporaryDirectory(prefix=prefix)

diff --git a/selftests/unit/test_utils_kernel.py → selftests/unit/utils/test_kernel.py b/selftests/unit/test_utils_kernel.py → selftests/unit/utils/test_kernel.py
@@ -2,7 +2,7 @@
 
 from avocado.utils.kernel import KernelBuild
 
-from .. import setup_avocado_loggers
+from ... import setup_avocado_loggers
 
 setup_avocado_loggers()
 

diff --git a/selftests/unit/test_utils_linux_modules.py → selftests/unit/utils/test_linux_modules.py b/selftests/unit/test_utils_linux_modules.py → selftests/unit/utils/test_linux_modules.py
diff --git a/...it/test_utils_linux_modules.py.data/lsmod → ...it/utils/test_linux_modules.py.data/lsmod b/...it/test_utils_linux_modules.py.data/lsmod → ...it/utils/test_linux_modules.py.data/lsmod
diff --git a/..._utils_linux_modules.py.data/proc_modules → ...s/test_linux_modules.py.data/proc_modules b/..._utils_linux_modules.py.data/proc_modules → ...s/test_linux_modules.py.data/proc_modules
diff --git a/selftests/unit/test_utils_memory.py → selftests/unit/utils/test_memory.py b/selftests/unit/test_utils_memory.py → selftests/unit/utils/test_memory.py
diff --git a/selftests/unit/test_utils_network.py → selftests/unit/utils/test_network.py b/selftests/unit/test_utils_network.py → selftests/unit/utils/test_network.py
diff --git a/selftests/unit/test_utils_output.py → selftests/unit/utils/test_output.py b/selftests/unit/test_utils_output.py → selftests/unit/utils/test_output.py
diff --git a/selftests/unit/test_utils_partition.py → selftests/unit/utils/test_partition.py b/selftests/unit/test_utils_partition.py → selftests/unit/utils/test_partition.py
@@ -13,7 +13,7 @@
 from avocado.utils import path as utils_path
 from avocado.utils import process, wait
 
-from .. import temp_dir_prefix
+from ... import temp_dir_prefix
 
 
 def missing_binary(binary):

diff --git a/selftests/unit/test_utils_path.py → selftests/unit/utils/test_path.py b/selftests/unit/test_utils_path.py → selftests/unit/utils/test_path.py
diff --git a/selftests/unit/test_utils_pci.py → selftests/unit/utils/test_pci.py b/selftests/unit/test_utils_pci.py → selftests/unit/utils/test_pci.py
diff --git a/selftests/unit/test_utils_pmem.py → selftests/unit/utils/test_pmem.py b/selftests/unit/test_utils_pmem.py → selftests/unit/utils/test_pmem.py
diff --git a/selftests/unit/test_utils_process.py → selftests/unit/utils/test_process.py b/selftests/unit/test_utils_process.py → selftests/unit/utils/test_process.py
@@ -7,8 +7,8 @@
 
 from avocado.utils import path, process, script
 
-from .. import (setup_avocado_loggers, skipOnLevelsInferiorThan,
-                skipUnlessPathExists)
+from ... import (setup_avocado_loggers, skipOnLevelsInferiorThan,
+                 skipUnlessPathExists)
 
 setup_avocado_loggers()
 

diff --git a/selftests/unit/test_utils_script.py → selftests/unit/utils/test_script.py b/selftests/unit/test_utils_script.py → selftests/unit/utils/test_script.py
diff --git a/selftests/unit/test_utils_service.py → selftests/unit/utils/test_service.py b/selftests/unit/test_utils_service.py → selftests/unit/utils/test_service.py
@@ -20,7 +20,7 @@
 
 from avocado.utils import service
 
-from .. import setup_avocado_loggers
+from ... import setup_avocado_loggers
 
 setup_avocado_loggers()
 

diff --git a/...tests/unit/test_utils_software_manager.py → ...tests/unit/utils/test_software_manager.py b/...tests/unit/test_utils_software_manager.py → ...tests/unit/utils/test_software_manager.py
@@ -3,7 +3,7 @@
 
 from avocado.utils import distro, software_manager
 
-from .. import setup_avocado_loggers
+from ... import setup_avocado_loggers
 
 setup_avocado_loggers()
 

diff --git a/selftests/unit/test_utils_ssh.py → selftests/unit/utils/test_ssh.py b/selftests/unit/test_utils_ssh.py → selftests/unit/utils/test_ssh.py
diff --git a/selftests/unit/test_utils_stacktrace.py → selftests/unit/utils/test_stacktrace.py b/selftests/unit/test_utils_stacktrace.py → selftests/unit/utils/test_stacktrace.py
diff --git a/selftests/unit/test_utils_vmimage.py → selftests/unit/utils/test_vmimage.py b/selftests/unit/test_utils_vmimage.py → selftests/unit/utils/test_vmimage.py
@@ -3,7 +3,7 @@
 
 from avocado.utils import vmimage
 
-from .. import setup_avocado_loggers
+from ... import setup_avocado_loggers
 
 setup_avocado_loggers()