-
Notifications
You must be signed in to change notification settings - Fork 31
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
feat(xslt): coverage reporter for SonarQube #1005
base: master
Are you sure you want to change the base?
Conversation
Thanks for restarting the pr, @VTristan
|
<xsl:variable name="context" as="node()*" select="trace/m"/> | ||
|
||
<coverage version="1"> | ||
<xsl:for-each select="$context[not(contains(@u,'xspec-utils.xsl'))]"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now that #1014 is merged into the master, I think you can remove the predicate.
<xsl:for-each select="$context[not(contains(@u,'xspec-utils.xsl'))]"> | |
<xsl:for-each select="$context"> |
If you merge the latest master into this branch and run set COVERAGE_REPORTER_XSL=
set COVERAGE_HTML=
bin\xspec.bat -c test\end-to-end\cases\coverage-no-hit.xspec then the generated On the other hand, if you run set COVERAGE_REPORTER_XSL=src\reporter\coverage-sonar-report.xsl
set COVERAGE_HTML=coverage-sonarqube.xml
bin\xspec.bat -c test\end-to-end\cases\coverage-no-hit.xspec then the generated <coverage version="1"></coverage> where you can't identify the two involved-but-untested stylesheets. Is that fine? |
Proposal for issue #877
I propose to answer this issue by adding a new stylesheet
coverage-sonar-report.xsl
and by factoringcoverage-report.xsl.
The new report generated, corresponds to the generic format of SonarQube.
Just change the environment variables.
e.g on windows:
set "COVERAGE_REPORTER_XSL=%XSPEC_HOME%\src\reporter\coverage-sonar-report.xsl"
set "COVERAGE_HTML=%TEST_DIR%\%TARGET_FILE_NAME%-sonar-coverage.xml"
(I did a second fork, to offer a cleaner and more up-to-date PR).