Enable python tests to run in CI

apojomovsky commented 6 years ago

Likely by running the tools/run_tests.sh, which would run both, the C++ and Python tests. Blocked by #275

apojomovsky commented 6 years ago

Currently facing an issue when attempting to run python tests in CI:

ERROR: test.simulation_runner_py_test (unittest.loader.ModuleImportFailure)

----------------------------------------------------------------------

ImportError: Failed to import test module: test.simulation_runner_py_test

Traceback (most recent call last):

  File "/usr/lib/python2.7/unittest/loader.py", line 254, in _find_tests

    module = self._get_module_from_name(name)

  File "/usr/lib/python2.7/unittest/loader.py", line 232, in _get_module_from_name

    __import__(name)

  File "/home/ubuntu/workspace/_delphyne_test_do_not_merge-RFH3SOKHTEDOIJRE3PTYX6AHJ4JU5EZRCFTTKQLJRXNAEIEOJV5Q/backend/test/simulation_runner_py_test.py", line 9, in <module>

    from python_bindings import (

ImportError: ../../../install/lib/libdrake.so: cannot open shared object file: No such file or directory

After having checked and re-checked the PYTHONPATH and LD_LIBRARY_PATH multiple times, I realized the issue is related to #277 (addressed in drake's #8308 ). I would like to ask for suggestions in the way to proceed with this issue, since currently I see three options:

Hack our current Jenkins' workspace to match the relative-paths just as libdrake.so currently expects (as we currently do in our development workspaces, where it just works).
Target a forked version of drake, and point to a cherry-picked/hacked version to get the fixes of the PR mentioned above.
We might want to wait for #275 and then address this in a much more straightforward way.

Any suggestions would be highly appreciated.

cc @clalancette @basicNew

clalancette commented 6 years ago

I think we really want to get #275 resolved. It is blocking quite a number of things from moving forward. @apojomovsky, my suggestion is to spend some time debugging the current ign-rendering build system, and seeing if we can get a quick fix to make it work. Assuming we can get that, we can move forward with Drake and unblock a lot of stuff. Does that make sense to you, and do you mind doing that?

basicNew commented 6 years ago

Totally agree on getting #275 fixed. Moving the conversation to that issue.

apojomovsky commented 6 years ago

@clalancette , it makes a lot of sense. And of course, I'd gladly address the ign-rendering task.

maliput / delphyne

Enable python tests to run in CI #295