Open requested to merge experimental/maven_test into master
Created by: muffato
The Java tests are currently run twice: in
before_install and in
scripts/dev/travis_run_tests.sh. This causes two issues:
- extra Travis time, which is really precious and should not be wasted
- The test build will immediately fail and only report the Java error. There could be other errors worth being reported part of the build
Only run the tests in
scripts/dev/travis_run_tests.sh (like all the other tests).
Extra Travis time when the only issue is that the Java wrapper doesn't pass its own tests. But the same could be said of any components. It would make sense to test (and fail early) the critical components, but I don't think the Java wrapper is currently critical.
Have you added/modified unit tests to test the changes?
Have you run the entire test suite and no regression was detected?