Strip cache population as it is not necessary

Issues with Pinocchio unit tests lie elsewhere and are now documented.

See #6

Author: stephane-caron

CHANGELOG.md

@@ -24,10 +24,6 @@ All notable changes to this project will be documented in this file.
 - Match cache path with package name when it helps PyBullet
 - Remove ``MESHES_PATH`` attribute
 
-### Fixed
-
-- CI: Populate cache before running unit tests
-
 ## [0.3.0] - 2022/09/12
 
 ### Added

robot_descriptions/__init__.py

@@ -20,7 +20,3 @@
 """
 
 __version__ = "0.4.0rc1"
-
-from ._cache import populate_cache
-
-__all__ = ["populate_cache"]

robot_descriptions/_cache.py

@@ -119,15 +119,3 @@ def clone_to_cache(description_name: str) -> str:
     clone = clone_to_directory(repository.url, target_dir)
     clone.git.checkout(repository.commit)
     return str(clone.working_dir)
-
-
-def populate_cache() -> None:
-    """
-    Clone all repositories to cache.
-
-    Notes:
-        This function can be used for unit testing, or before another program
-        that wants to make sure robot descriptions are imported right away.
-    """
-    for description_name in REPOSITORIES:
-        clone_to_cache(description_name)

tests/test_cache.py

@@ -30,7 +30,7 @@
 from robot_descriptions._repositories import REPOSITORIES
 
 
-class TestGit(unittest.TestCase):
+class TestCache(unittest.TestCase):
 
     """
     Test fixture for git-related functions.

tox.ini

@@ -27,7 +27,6 @@ commands =
     flake8 robot_descriptions
     pylint robot_descriptions --exit-zero --rcfile={toxinidir}/tox.ini
     mypy robot_descriptions --ignore-missing-imports
-    python -c "from robot_descriptions import populate_cache; populate_cache()"
     coverage erase
     coverage run -m unittest discover
     coverage report --include="robot_descriptions/*"