Skip to content
Snippets Groups Projects

Added unittest code and CI setup

Merged Carlos Mastalli requested to merge unittest into master

In this PR I do:

  • addition of loading unit-test code that depends on pinocchio,
  • definition of Dockerfile and CI configuration, and
  • better description of installation procedure and pipeline status inside Readme.

As you can see there is missing something here, it's failing the CI job. @gsaurel could you add the additional information here?

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    • df28a261 - [unittest] Removed useless imported module

    Compare with previous version

  • Also, I think we can find a better way to code test_all.py. But as there is only one test file now, I think we should just remove it, and directly launch the test file instead.

  • added 1 commit

    • b621edd5 - [unittest] Formatting these files

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Carlos Mastalli resolved all discussions

    resolved all discussions

  • added 1 commit

    • b4d4f4dc - [unittest] Removed test_all since it's overkill right now

    Compare with previous version

  • @gsaurel I removed test_all and instead CI jobs call test_load. I think we can merge it after solving the issue with the CI jobs.

    Edited by Carlos Mastalli
  • @cmastall the buildfarm can now handle to install pinocchio only for the tests. But tell me, which pinocchio version has to be used ? 1.3 or 2.0 ?

  • I think both are OK. Just uses the version that makes your life easier :)

  • I tried with 1.3 without success, as

    • RobotWrapper API has changed (easy to workaround)
    • loadRotorParameters is not yet available.

    I think we can wait for 2.0 to land in the binary repository.

    Edited by Guilhem Saurel
  • It's good to me!

    Edited by Carlos Mastalli
  • Carlos Mastalli added 2 commits

    added 2 commits

    • 0cd13863 - [organization] Moving urdfs and srdfs to robots folder in order to mimic previous organizations
    • f71b126d - [unittest] Updated the path according to new organization

    Compare with previous version

  • mentioned in issue #4 (closed)

  • Carlos Mastalli added 19 commits

    added 19 commits

    • f71b126d...8f22bb30 - 6 commits from branch master
    • 8f922511 - [unittest] Added loading test for Talos, Talos arm and HyQ
    • c7b23e6e - [unittest] Defined the Dockerfile with the dependencies
    • 1d19baf4 - [unittest] Configure the CI jobs
    • 89ca8587 - [git] Added ignore rule
    • a295336e - [readme] Added status of pipeline
    • ad078a19 - [readme] Added information for installation
    • 00a1ef59 - [unittest] Removed useless imported module
    • d65e828b - [unittest] Formatting these files
    • f74920b9 - [unittest] Sorting modules
    • a6001ff9 - [unittest] Using eu0c
    • 639aadd9 - [unittest] Updated the path according to new organization
    • 14c2aca5 - [unittest] Removed test_all since it's overkill right now
    • da6012b8 - [unittest] Updated according to changes in the organization

    Compare with previous version

  • Guilhem Saurel mentioned in merge request !4 (merged)

    mentioned in merge request !4 (merged)

  • Carlos Mastalli added 13 commits

    added 13 commits

    • 330099c7 - [unittest] Added loading test for Talos, Talos arm and HyQ
    • f26a1801 - [unittest] Defined the Dockerfile with the dependencies
    • 8ac7b290 - [unittest] Configure the CI jobs
    • f205e94f - [git] Added ignore rule
    • 71e9965d - [readme] Added status of pipeline
    • 1c43916e - [readme] Added information for installation
    • c7a5bae7 - Merge branch 'master' into unittest
    • 9c6a8ee4 - [Test] Standardization
    • bf359cd6 - Merge branch 'unittest' of gepgitlab.laas.fr:gepetto/example-robot-data into unittest
    • 9aeb4e3e - [Tests] we test the files in the current repository, not the ones that are installed
    • 4f67fa6f - [Tests] use getModelPath to allow build/test in different directories
    • e42751ec - remove Dockerfile
    • 747ac089 - Merge branch 'unittest' into 'unittest'

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading