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

Added ability to run instrumentation tests before unit tests and merge all tests' reports into single jacoco coverage report #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nekdenis
Copy link

No description provided.

@codecov-io
Copy link

Current coverage is 88.89%

Merging #21 into master will increase coverage by -9.41%

  1. File ...AndroidPlugin.groovy was modified. more
    • Misses 0
    • Partials +2
    • Hits -2
@@           master        #21   diff @@
========================================
  Files           3          3          
  Lines          59         72    +13   
  Methods         0          0          
  Branches        4          7     +3   
========================================
+ Hits           58         64     +6   
- Misses          1          4     +3   
- Partials        0          4     +4   

Powered by Codecov. Last updated by 12dc827...c6ccff7

@WarrenFaith
Copy link

@nekdenis do I understand this correctly that your changes enables also that reports from different target devices are merged as well?
If that is the case it would be what I really really need.

If it is not the case, do you think you can change your PR to support that as well?

Some context: Currently the last fetched report from our test device farm is the one used for the reports in SonarQube and I failed to set up the merging of it.

@mguntli
Copy link

mguntli commented Jan 4, 2017

Hi
When is it planned to integrate this into the master branch?

@suarezjulian
Copy link

Would love to see this on the next release

@bmoliveira
Copy link

There is any plan to merge this and a new version release for this?

@martiwi
Copy link

martiwi commented Mar 19, 2019

Hi here, I see that this PR is opened for a very long time now, is there any plan to have it merged any time soon?

@Jeehut
Copy link

Jeehut commented Nov 30, 2019

Any updates here @arturdm, would love to use this as well! 👍

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.

8 participants