uqbar-project / arena

MMVC framework
http://arena.uqbar-project.org/
4 stars 2 forks source link

Add code coverage to travis with coveralls #17

Closed javierfernandes closed 8 years ago

javierfernandes commented 9 years ago

See wollok project that already has this implemented

fdodino commented 8 years ago

Copiar de: https://github.com/uqbar-project/wollok/blob/master/.travis.yml

fdodino commented 8 years ago

Cosas para hacer 1) Importar en el pom de arena jacoco y coveralls

            <!-- code coverage -->
            <plugin>
                <groupId>org.jacoco</groupId>
                <artifactId>jacoco-maven-plugin</artifactId>
                <version>0.7.5.201505241946</version>
                <executions>
                    <execution>
                        <id>prepare-agent</id>
                        <goals>
                            <goal>prepare-agent</goal>
                        </goals>
                    </execution>
                </executions>
            </plugin>
            <plugin>
                <groupId>org.eluder.coveralls</groupId>
                <artifactId>coveralls-maven-plugin</artifactId>
                <version>4.1.0</version>

                <configuration>
                    <coberturaReports>
                        <coberturaReport>
                            ${basedir}/target/coverage.xml
                        </coberturaReport>
                    </coberturaReports>
                    <sourceEncoding>UTF-8</sourceEncoding>
                    <serviceName>travis-ci</serviceName>
                    <serviceJobId>${env.TRAVIS_JOB_ID} </serviceJobId>
                </configuration>
            </plugin>
            <!--  code coverage -->

2) Agregar en travis el script (copiar de .travis.yml)

script: 
- mvn -DTRAVIS_JOB_ID=$TRAVIS_JOB_ID clean install jacoco:report coveralls:report

3) Entrar a http://coveralls.io con el usuario de github de uno y habilitar el repo

4) Incorporar el badge al README (entrar nuevamente a http://coveralls.io y navegar hasta el repo para luego seleccionar badge your repo)

fdodino commented 8 years ago

Primera parte lista. Falta linkear con tests de Scala porque hay un test coverage del 0,44% :)

fdodino commented 8 years ago

Ok con lo de Scala, de hecho hay dos clases que suben un poco el escaso test coverage que hay... https://coveralls.io/builds/4609523?all=true Marco el issue como done, y a futuro hay que tratar de subir el % de test coverage

fdodino commented 8 years ago

Ajá, no está tomando los tests de java, por ejemplo en arena-core, no me tomó los tests que le hice a los bootstraps, habrá que averiguar qué pasa.

fdodino commented 8 years ago

Falsa alarma, no había tomado el último commit.