Skip to content

Online instrumentation mode#117

Merged
danglotb merged 6 commits intoSTAMP-project:masterfrom
andre15silva:online-prototype
Aug 23, 2021
Merged

Online instrumentation mode#117
danglotb merged 6 commits intoSTAMP-project:masterfrom
andre15silva:online-prototype

Conversation

@andre15silva
Copy link
Copy Markdown
Contributor

@andre15silva andre15silva commented Aug 17, 2021

I'm working on this PR, which features a new mode for getting the covered test results through online instrumentation, instead of through offline instrumentation as it is the case of all modes right now.

The advantages are:

The PR's main work is finished. Tomorrow I'll finish adding tests and testing the remaining integration with flacoco and other tools.

Early feedback is appreciated though, specifically if you want to merge some test-runner modes.

@andre15silva andre15silva mentioned this pull request Aug 17, 2021
Closed
@andre15silva andre15silva force-pushed the online-prototype branch 4 times, most recently from ac3c91a to 1516ef5 Compare August 18, 2021 10:13
@andre15silva
Copy link
Copy Markdown
Contributor Author

andre15silva commented Aug 18, 2021

coveralls is failing because it doesn't find the source files of the jacoco agent.

The coveralls-maven-plugin does not have the option to ignore these errors (trautonen/coveralls-maven-plugin#113).

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Aug 18, 2021

Codecov Report

❗ No coverage uploaded for pull request base (master@ee045f4). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #117   +/-   ##
=========================================
  Coverage          ?   41.08%           
  Complexity        ?      321           
=========================================
  Files             ?       68           
  Lines             ?     2529           
  Branches          ?      189           
=========================================
  Hits              ?     1039           
  Misses            ?     1425           
  Partials          ?       65

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ee045f4...1516ef5. Read the comment docs.

@andre15silva andre15silva mentioned this pull request Aug 18, 2021
Merged
@andre15silva
Copy link
Copy Markdown
Contributor Author

andre15silva commented Aug 18, 2021

@danglotb I have cherry-picked the commits that replace the coverage platform to #118.

After that is merged, I'll rebase here and this will be ready too.

@andre15silva andre15silva mentioned this pull request Aug 18, 2021
Merged
@andre15silva
Add online jacoco runner prototype.
fe93a14 
Signed-off-by: André Silva <andre15andre@hotmail.com>
@andre15silva
Online instrumentation for JUnit5. Jacoco agent includes, excludes
a3b8675 
options.

Signed-off-by: André Silva <andre15andre@hotmail.com>
@andre15silva
Add jacocoagent.jar resource
a90e6ef 
Signed-off-by: André Silva <andre15andre@hotmail.com>
@andre15silva
Fix jacoco version
4adea1c 
Signed-off-by: André Silva <andre15andre@hotmail.com>
@andre15silva
Replace coveralls with codecov
d8c937d 
Signed-off-by: André Silva <andre15andre@hotmail.com>
@andre15silva
Add tests and documentation.
18e6ed2 
Signed-off-by: André Silva <andre15andre@hotmail.com>
@andre15silva andre15silva marked this pull request as ready for review August 23, 2021 07:01
@andre15silva
Copy link
Copy Markdown
Contributor Author

andre15silva commented Aug 23, 2021

Thank you @danglotb !

This one is ready now too.

@andre15silva
Copy link
Copy Markdown
Contributor Author

andre15silva commented Aug 23, 2021

@danglotb do you have any timeline of when you will be able to review this PR?

This PR is blocking ASSERT-KTH/flacoco#86, SpoonLabs/astor#323 and SpoonLabs/nopol#220 and I'd like to finish it them by the end of this week (when I finish my internship) 😅

@danglotb danglotb merged commit b937cb2 into STAMP-project:master Aug 23, 2021
@andre15silva
Copy link
Copy Markdown
Contributor Author

andre15silva commented Aug 24, 2021

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@danglotb danglotb danglotb approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@andre15silva @codecov-commenter @danglotb