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

add support for Cloverage #50

Open
esuomi opened this issue Jun 25, 2018 · 1 comment
Open

add support for Cloverage #50

esuomi opened this issue Jun 25, 2018 · 1 comment

Comments

@esuomi
Copy link

esuomi commented Jun 25, 2018

This is a continuation to #20 but with updated details:

@Deraen
Copy link
Contributor

Deraen commented Jun 25, 2018

Another example here: https://github.com/metosin/bat-test/blob/master/src/metosin/bat_test/cloverage.clj

The difference between this and circleci.test is that,

With circleci.test one runs cloverage lein plugin which then calls into circleci.test to run the tests while Cloverage instrumentations are active. I'm not sure how well this approach works together with e.g. eftest report options. It could be hard to access plugin settings in the multimethod.

With bat-test the user runs bat-test like normal but with Cloverage option enabled, and in this bat-test takes care of enabling instrumentations and writing the output.

My approach requires duplicating some code from Cloverage, and it might make sense to add this API to Cloverage if another library wants to use this.

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

No branches or pull requests

2 participants