Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Test Coverage Tracking] Added coveralls integration #8247

Closed
wants to merge 6 commits into from

Conversation

fearphage
Copy link
Contributor

Fixes #8246

This needs an admin to setup a https://coveralls.io account and update the referenced repo token in the code.

.travis.yml Outdated
- echo "MAVEN_OPTS='-Xmx2g'" > ~/.mavenrc
after_success:
- mvn clean test jacoco:report coveralls:report
Copy link
Member

@JayDi85 JayDi85 Sep 11, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it must be like that:
mvn clean jacoco:prepare-agent test jacoco:report jacoco:report-aggregate coveralls:report

Also coveralls must support multi modules project (if not then it must load report from a Mage.Verify project only -- like sonar do, see pom's comment about it).

@fearphage
Copy link
Contributor Author

[ERROR] Failed to execute goal org.eluder.coveralls:coveralls-maven-plugin:4.3.0:report (default-cli) on project mage-root: Processing of input or output data failed: Report submission to Coveralls API failed with HTTP status 422: Unprocessable Entity (Couldn't find a repository matching this job.) -> [Help 1]

@JayDi85 Did you ever click buttons on the coveralls site to add this project?

@JayDi85
Copy link
Member

JayDi85 commented Oct 4, 2021

Yes. It sends something: https://coveralls.io/builds/43239474

@fearphage
Copy link
Contributor Author

Yes. It sends something

That was from the github-actions branch (#8248). This one failed due to the error message above.

@xenohedron
Copy link
Contributor

Closing as per #8246, sonar to be run manually on occasion due to large size of project

@xenohedron xenohedron closed this Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integrate code coverage into Travis pipeline and GitHub PR workflow
3 participants