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

Plugin not including coverage report for classes in inline grails plugins #44

Open
asuraphel opened this issue Sep 17, 2015 · 5 comments

Comments

@asuraphel
Copy link

We have split up our application into smaller plugins and most of our code is in those plugins. However grails-code-coverage doesn't seem to report on the plugin classes. Is this a bug or unsupported functionality?

@beckje01
Copy link
Owner

You would use the the coverage tool in each of your plugins builds directly.

@asuraphel
Copy link
Author

@beckje01 It's just one app with a few plugins. Did you mean we should put the grails-code-coverage plugin inside all plugin's BuildConfig.groovy?

@asuraphel asuraphel changed the title Plugin not including coverage report for custom grails classes Plugin not including coverage report for classes in inline grails plugins Sep 18, 2015
@beckje01
Copy link
Owner

Ahh I didn't know it was inline plugins, in that case you could just add the extra source roots. Check out the sourceInclusions in the readme.

@asuraphel
Copy link
Author

@beckje01 I tried doing:

coverage {
    sourceInclusions = ['../inline-plugin-name/grails-app/services']
}

However, the service classes inside that plugin are not showing on the coverage report. Am I missing something?

@asuraphel
Copy link
Author

@beckje01 Superficially going through the code I see that classes are instrumented from classDir and the plugin's class files are inside work/plugin-classes. I think source inclusions feature will not work. Don't you think so?

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