prasadtalasila / BITS-Darshini

A Modular, Concurrent and Customizable Protocol Analyzer
GNU General Public License v3.0
5 stars 12 forks source link

code coverage on travis #139

Closed prasadtalasila closed 6 years ago

prasadtalasila commented 6 years ago

The coveralls plugin can help in pushing the code coverage report to coveralls. We can try to use the same.

There is also codacy plugin if coveralls plug in does not work.

prasadtalasila commented 6 years ago

update: our pom.xml file already has the necessary configuration for coveralls plugin with jacoco code coverage tool. We just need to follow the suggested configuration to see if the code coverage is reported to code climate.

Alternative

The new code climate test reporter has support for Java. Please see the example configuration for tavis-ci

prasadtalasila commented 6 years ago

The codeclimate test reporter ID token has been placed in travis CI settings of the project. The token can be accessed using the environment variable $CC_TEST_REPORTER_ID.

prasadtalasila commented 6 years ago

We found codecov to be a better code coverage reporter. Let's integrate that plug-in for code coverage.

prasadtalasila commented 6 years ago

@kapilkalra04 I merged the existing PR #147. There is still some old code in pom.xml. See the following code snippet.

                <plugin>
                    <groupId>org.jacoco</groupId>
                    <artifactId>jacoco-maven-plugin</artifactId>
                    <version>0.7.9</version>
                    <executions>
                        <execution>
                            <id>prepare-agent</id>
                            <goals>
                                <goal>prepare-agent</goal>
                            </goals>
                        </execution>
                    </executions>
                </plugin>

This part can be safely removed now. Please fork my dev branch again, and update the pom.xml and send a PR again.

prasadtalasila commented 6 years ago

PR #147 solves this issue.