diff options
author | Arthur Zamarin <arthurzam@gentoo.org> | 2022-08-24 20:08:07 +0300 |
---|---|---|
committer | Arthur Zamarin <arthurzam@gentoo.org> | 2022-08-24 20:58:54 +0300 |
commit | be80c6813c4ae056e667a6065816f73471446b86 (patch) | |
tree | 974fac701f97ca2a6c7a219d1d2be53bba5911d1 | |
parent | remove cython _posix.pyx (diff) | |
download | snakeoil-be80c6813c4ae056e667a6065816f73471446b86.tar.gz snakeoil-be80c6813c4ae056e667a6065816f73471446b86.tar.bz2 snakeoil-be80c6813c4ae056e667a6065816f73471446b86.zip |
remove mentions of _readdir C extension
Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
-rw-r--r-- | src/snakeoil/osutils/__init__.py | 8 | ||||
-rw-r--r-- | tests/test_osutils.py | 51 |
2 files changed, 14 insertions, 45 deletions
diff --git a/src/snakeoil/osutils/__init__.py b/src/snakeoil/osutils/__init__.py index d4c888ba..1dc7561e 100644 --- a/src/snakeoil/osutils/__init__.py +++ b/src/snakeoil/osutils/__init__.py @@ -44,13 +44,7 @@ import os import stat import sys -# No name '_readdir' in module osutils -# pylint: disable=E0611 - -try: - from . import _readdir as module -except ImportError: - from . import native_readdir as module +from . import native_readdir as module # delay this... it's a 1ms hit, and not a lot of the consumers # force utf8 codepaths yet. diff --git a/tests/test_osutils.py b/tests/test_osutils.py index aea0ddab..e56d3ab3 100644 --- a/tests/test_osutils.py +++ b/tests/test_osutils.py @@ -13,13 +13,10 @@ from snakeoil.contexts import Namespace from snakeoil.fileutils import touch, write_file from snakeoil.osutils import native_readdir, supported_systems, sizeof_fmt from snakeoil.osutils.mount import MNT_DETACH, MS_BIND, mount, umount -from snakeoil.test import mk_cpy_loadable_testcase class ReaddirCommon: - module = native_readdir - @pytest.fixture def subdir(self, tmp_path): subdir = tmp_path / 'dir' @@ -36,27 +33,27 @@ class ReaddirCommon: class TestNativeListDir(ReaddirCommon): def test_listdir(self, tmp_path, subdir): - assert sorted(self.module.listdir(tmp_path)) == ['dir', 'fifo', 'file'] - assert self.module.listdir(subdir) == [] + assert set(native_readdir.listdir(tmp_path)) == {'dir', 'fifo', 'file'} + assert native_readdir.listdir(subdir) == [] def test_listdir_dirs(self, tmp_path, subdir): - assert self.module.listdir_dirs(tmp_path) == ['dir'] - assert self.module.listdir_dirs(subdir) == [] + assert native_readdir.listdir_dirs(tmp_path) == ['dir'] + assert native_readdir.listdir_dirs(subdir) == [] def test_listdir_files(self, tmp_path, subdir): - assert self.module.listdir_files(tmp_path) == ['file'] - assert self.module.listdir_dirs(subdir) == [] + assert native_readdir.listdir_files(tmp_path) == ['file'] + assert native_readdir.listdir_dirs(subdir) == [] def test_missing(self, tmp_path, subdir): return self._test_missing(tmp_path, ( - self.module.listdir, - self.module.listdir_dirs, - self.module.listdir_files, + native_readdir.listdir, + native_readdir.listdir_dirs, + native_readdir.listdir_files, )) def test_dangling_sym(self, tmp_path, subdir): (tmp_path / "monkeys").symlink_to("foon") - assert self.module.listdir_files(tmp_path) == ['file'] + assert native_readdir.listdir_files(tmp_path) == ['file'] class TestNativeReaddir(ReaddirCommon): @@ -73,29 +70,11 @@ class TestNativeReaddir(ReaddirCommon): ("monkeys", "symlink"), ("sym", "symlink"), } - assert set(self.module.readdir(tmp_path)) == expected - assert self.module.readdir(subdir) == [] + assert set(native_readdir.readdir(tmp_path)) == expected + assert native_readdir.readdir(subdir) == [] def test_missing(self, tmp_path): - return self._test_missing(tmp_path, (self.module.readdir,)) - - -try: - # No name "readdir" in module osutils - # pylint: disable=E0611 - from snakeoil.osutils import _readdir -except ImportError: - _readdir = None - - -@pytest.mark.skipif(_readdir is None, reason="extension isn't compiled") -class TestCPyListDir(TestNativeListDir): - module = _readdir - - -@pytest.mark.skipif(_readdir is None, reason="extension isn't compiled") -class TestCPyReaddir(TestNativeReaddir): - module = _readdir + return self._test_missing(tmp_path, (native_readdir.readdir,)) class TestEnsureDirs: @@ -434,10 +413,6 @@ class TestMount: pytest.skip('No permission to use user and mount namespace') -Test_cpy_readdir_loaded = mk_cpy_loadable_testcase( - "snakeoil.osutils._readdir", "snakeoil.osutils", "listdir", "listdir") - - class TestSizeofFmt: expected = { 0: ("0.0 B", "0.0 B"), |