WIP: Separate testing for coverage and loaders

Author: stephane-caron

.github/workflows/test.yml

@@ -8,17 +8,9 @@ on:
     workflow_dispatch:
 
 jobs:
-    test:
-        name: "${{ matrix.os }}, python-${{ matrix.python-version }}"
-        runs-on: ${{ matrix.os }}
-
-        env:
-            USING_COVERAGE: "3.8"
-
-        strategy:
-            matrix:
-                os: [ubuntu-latest]
-                python-version: ["3.7", "3.8", "3.9", "3.10"]
+    coverage:
+        name: "Coverage"
+        runs-on: ubuntu-latest
 
         steps:
             - name: "Checkout sources"
@@ -27,22 +19,67 @@ jobs:
             - name: "Set up Python"
               uses: actions/setup-python@v4
               with:
-                  python-version: "${{ matrix.python-version }}"
+                  python-version: "3.8"
 
             - name: "Install dependencies"
               run: |
-                  python -m pip install --upgrade pip
-                  python -m pip install coverage coveralls flake8 flit mccabe mypy pylint pytest tox tox-gh-actions
+                  python -m pip install -U pip tox tox-gh-actions
 
-            - name: "Run tox targets for ${{ matrix.python-version }}"
+            - name: "Check coverage"
               run: |
-                  tox
-              env:
-                  PLATFORM: ${{ matrix.os }}
+                  tox -e coverage
 
             - name: "Coveralls"
               env:
                   GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
               run: |
                   coveralls --service=github
-              if: ${{ runner.os == 'Linux' && contains(env.USING_COVERAGE, matrix.python-version) }}
+
+    lint:
+        name: "Code style"
+        runs-on: ubuntu-latest
+
+        steps:
+            - name: "Checkout sources"
+              uses: actions/checkout@v3
+
+            - name: "Set up Python"
+              uses: actions/setup-python@v4
+              with:
+                  python-version: "3.8"
+
+            - name: "Install dependencies"
+              run: |
+                  python -m pip install -U pip tox tox-gh-actions
+
+            - name: "Check code format"
+              run: |
+                  tox -e lint
+
+    loaders:
+        name: "Loader: ${{ matrix.loader }}"
+        runs-on: ubuntu-latest
+
+        strategy:
+            matrix:
+                python-version: ["3.8"]
+                loader: ["mujoco", "pinocchio", "pybullet", "robomeshcat", "yourdfpy"]
+
+        steps:
+            - name: "Checkout sources"
+              uses: actions/checkout@v3
+
+            - name: "Set up Python"
+              uses: actions/setup-python@v4
+              with:
+                  python-version: "${{ matrix.python-version }}"
+
+            - name: "Install dependencies"
+              run: |
+                  python -m pip install -U pip tox tox-gh-actions
+
+            - name: "Run loader tests"
+              run: |
+                  tox -e loaders
+              env:
+                  LOADER: ${{ matrix.loader }}

tests/__init__.py

@@ -1 +1,26 @@
-# Make sure Python treats the test directory as a package.
+#!/usr/bin/env python
+# -*- coding: utf-8 -*-
+#
+# Copyright 2022 Stéphane Caron
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+from .test_cache import TestCache
+from .test_descriptions import TestDescriptions
+from .test_loaders import TestLoaders
+
+__all__ = [
+    "TestCache",
+    "TestDescriptions",
+    "TestLoaders",
+]

tests/loaders/test_mujoco.py

@@ -19,8 +19,8 @@
 import unittest
 
 from robot_descriptions._descriptions import DESCRIPTIONS
-from robot_descriptions.loaders.mujoco import load_robot_description
 
+from robot_descriptions.loaders.mujoco import load_robot_description
 
 class TestMuJoCo(unittest.TestCase):
 
@@ -50,7 +50,6 @@ def test(self):
 
         return test
 
-
 for name, description in DESCRIPTIONS.items():
     if description.has_mjcf:
         setattr(