search

eclipsesource / jshint-eclipse

JSHint integration for the Eclipse IDE
http://github.eclipsesource.com/jshint-eclipse/
146 stars 31 forks source link

Enable unit tests in Travis CI #67

Closed ralfstx closed 11 years ago

ralfstx commented 11 years ago

The Travis CI build currently doesn't run the tests. Some tweaks may be needed for tycho-surefire. Not sure if the UI tests would run there anyway.

paulvi commented 11 years ago

Travis runs build in 2 steps

mvn install -DskipTests=true -B
mvn test -B

EclipseSource: Build Status Nodeclipse: Build Status

I tried an idea just to remove tycho-surefire config.

Well, in all cases maven doesn't see there are tests to execute.

No quick ideas more, needs to look carefully at those tests.

ralfstx commented 11 years ago

That worked - Travis now runs all tests including UI tests! Tycho-surefire tests are executed as in the integration-test phase because they require OSGi, so the bundles must be packaged already.

Thanks again for starting this effort, Paul! That's a real improvement.

Fixed with commit 6162e0ab2432818ba476af57ac9b0a8ab75adf63.

paulvi commented 11 years ago

Unit tests without CI is something half-done.

Secondly, I want this nice project to grow happily.

ralfstx commented 11 years ago

I really appreciate this. That gave me the push to do some work on jshint-eclipse again. It's time for a 0.9.9.