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 feature to exclude files from coverage analysis #68

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Added feature to exclude files from coverage analysis #68

wants to merge 7 commits into from

Conversation

Johnlon
Copy link

@Johnlon Johnlon commented Sep 8, 2013

New system property provides a comma separated list of regex's for the paths to ignore in coverage.
Files matching these patterns are excluded from the final coverage reports.

<scct.excluded.paths.regex>src/main/scala/.*Excluded.*/morepath,other/excluded/path/regex</scct.excluded.paths.regex>

I have tested against my own project and it works for me.

Any matching files are dropped entirely from the reports,

However, I have not been able to run any of the tests in the scct package as I keep getting an errors, presumably to do with my runtime env like the one below.

test-fail: Scala compiler is probably not finding scala jars (catch-22).
Fix paths in InstrumentationSpec:createSettings.

 example
	<configuration>
					<systemPropertyVariables>
						<scct.debug>false</scct.debug>
						<scct.project.name>${project.name}</scct.project.name>
						<scct.coverage.file>${project.build.outputDirectory}/coverage.data</scct.coverage.file>
						<scct.report.dir>${project.build.directory}/coverage-report</scct.report.dir>
						<scct.source.dir>${project.build.sourceDirectory}</scct.source.dir>
						<scct.excluded.paths.regex>src/main/scala/.*Excluded,other/excluded/path/regex</scct.excluded.paths.regex>
					</systemPropertyVariables>
				</configuration>
@Johnlon Johnlon mentioned this pull request Sep 8, 2013
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.

2 participants