Skip to content

Commit

Permalink
Disable yourdfpy in loader fixture directly
Browse files Browse the repository at this point in the history
  • Loading branch information
stephane-caron committed Dec 8, 2023
1 parent acabd40 commit cdbe0e9
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 13 deletions.
26 changes: 16 additions & 10 deletions tests/test_loaders.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
import unittest

import pybullet

from robot_descriptions.loaders.idyntree import (
load_robot_description as load_idyntree,
)
Expand All @@ -34,9 +35,12 @@
from robot_descriptions.loaders.robomeshcat import (
load_robot_description as load_robomeshcat,
)
from robot_descriptions.loaders.yourdfpy import (
load_robot_description as load_yourdfpy,
)

# Disabled because of https://github.com/clemense/yourdfpy/issues/49
# See https://github.com/robot-descriptions/robot_descriptions.py/issues/43
# from robot_descriptions.loaders.yourdfpy import (
# load_robot_description as load_yourdfpy,
# )


class TestLoaders(unittest.TestCase):
Expand Down Expand Up @@ -76,13 +80,15 @@ def test_robomeshcat(self):
)
)

def test_yourdfpy(self):
self.assertIsNotNone(
load_yourdfpy(
"upkie_description",
commit=self.upkie_description_commit,
)
)
# Disabled because of https://github.com/clemense/yourdfpy/issues/49
# See https://github.com/robot-descriptions/robot_descriptions.py/issues/43
# def test_yourdfpy(self):
# self.assertIsNotNone(
# load_yourdfpy(
# "upkie_description",
# commit=self.upkie_description_commit,
# )
# )

def test_idyntree(self):
self.assertIsNotNone(
Expand Down
4 changes: 1 addition & 3 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,7 @@ deps =
pin >=2.6.14
pybullet >=3.2.5
robomeshcat >= 1.0.4
# Disabled "yourdfpy" loader because of https://github.com/clemense/yourdfpy/issues/49
# Tracked in https://github.com/robot-descriptions/robot_descriptions.py/issues/43
# yourdfpy >=0.0.53
yourdfpy >=0.0.53
commands =
coverage erase
coverage run -m unittest tests
Expand Down

0 comments on commit cdbe0e9

Please sign in to comment.