Move get_files tests to separate file

tests/common.py

@@ -1,113 +0,0 @@
-from pathlib import Path
-from my.common import get_files
-
-import pytest # type: ignore
-
-
-def test_single_file():
-    '''
-    Regular file path is just returned as is.
-    '''
-
-    "Exception if it doesn't exist"
-    with pytest.raises(Exception):
-        get_files('/tmp/hpi_test/file.ext')
-
-
-    create('/tmp/hpi_test/file.ext')
-
-    '''
-    Couple of things:
-    1. Return type is a tuple, it's friendlier for hashing/caching
-    2. It always return pathlib.Path instead of plain strings
-    '''
-    assert get_files('/tmp/hpi_test/file.ext') == (
-        Path('/tmp/hpi_test/file.ext'),
-    )
-
-
-def test_multiple_files():
-    '''
-    If you pass a directory/multiple directories, it flattens the contents
-    '''
-    create('/tmp/hpi_test/dir1/')
-    create('/tmp/hpi_test/dir1/zzz')
-    create('/tmp/hpi_test/dir1/yyy')
-    # create('/tmp/hpi_test/dir1/whatever/') # TODO not sure about this... should really allow extra dirs
-    create('/tmp/hpi_test/dir2/')
-    create('/tmp/hpi_test/dir2/mmm')
-    create('/tmp/hpi_test/dir2/nnn')
-    create('/tmp/hpi_test/dir3/')
-    create('/tmp/hpi_test/dir3/ttt')
-
-    assert get_files([
-        Path('/tmp/hpi_test/dir3'), # it takes in Path as well as str
-        '/tmp/hpi_test/dir1',
-    ]) == (
-        # the paths are always returned in sorted order (unless you pass sort=False)
-        Path('/tmp/hpi_test/dir1/yyy'),
-        Path('/tmp/hpi_test/dir1/zzz'),
-        Path('/tmp/hpi_test/dir3/ttt'),
-    )
-
-
-def test_explicit_glob():
-    '''
-    You can pass a glob to restrict the extensions
-    '''
-
-    create('/tmp/hpi_test/file_3.zip')
-    create('/tmp/hpi_test/file_2.zip')
-    create('/tmp/hpi_test/ignoreme')
-    create('/tmp/hpi_test/file.zip')
-
-    # todo walrus operator would be great here...
-    expected = (
-        Path('/tmp/hpi_test/file_2.zip'),
-        Path('/tmp/hpi_test/file_3.zip'),
-    )
-    assert get_files('/tmp/hpi_test', 'file_*.zip') == expected
-
-    "named argument should work too"
-    assert get_files('/tmp/hpi_test', glob='file_*.zip') == expected
-
-
-def test_implicit_blog():
-    '''
-    Asterisc in the path results in globing too.
-    '''
-    # todo hopefully that makes sense? dunno why would anyone actually rely on asteriscs in names..
-    # this is very convenient in configs, so people don't have to use some special types
-
-    create('/tmp/hpi_test/123/')
-    create('/tmp/hpi_test/123/dummy')
-    create('/tmp/hpi_test/123/file.zip')
-    create('/tmp/hpi_test/456/')
-    create('/tmp/hpi_test/456/dummy')
-    create('/tmp/hpi_test/456/file.zip')
-
-    assert get_files(['/tmp/hpi_test/*/*.zip']) == (
-        Path('/tmp/hpi_test/123/file.zip'),
-        Path('/tmp/hpi_test/456/file.zip'),
-    )
-
-# TODO not sure if should uniquify if the filenames end up same?
-# TODO not sure about the symlinks? and hidden files?
-
-test_path = Path('/tmp/hpi_test')
-def setup():
-    teardown()
-    test_path.mkdir()
-
-
-def teardown():
-    import shutil
-    if test_path.is_dir():
-        shutil.rmtree(test_path)
-
-
-def create(f: str) -> None:
-    if f.endswith('/'):
-        Path(f).mkdir()
-    else:
-        Path(f).touch()