Issue 182 (Enhance XSLTCoverageTraceListener)

[cmarchand]

A proposal to solve Issue 182
I could maybe remove the @authors list in javadoc

[cmarchand]

Could someone give a test under Windows ?
Install Maven, then

mvn test 

It should display a "Build success" message

[AirQuick]

I will, but after handling #188.

[AirQuick]

mvn test worked fine on Windows. I'm going to check the other things.

[AirQuick]

I confirmed that the coverage XML was now written to a file specified by com.jenitennison.xsl.tests.coverage.output. This is the first time I see the coverage XML is made well-formed.😅 Thanks for the improvement.

I committed some changes of xspec.sh and xspec.bat in your branch. They set the new system properties without trying to normalize their values at all. It is very much appreciated if any platform or environment differences are absorbed within XSLTCoverageTraceListener.

[AirQuick]

So,

• Can you update XSLTCoverageTraceListener.class, too?

• With the updated xspec.sh and xspec.bat, I've observed these values at the beginning of enter(), for example:

computedGenerateTestUtilsName=C:\Users\User\Desktop\xspec\bin\..\src\compiler\generate-tests-utils.xsl
computedIgnoreDir=C:\Users\User\Desktop\xspec\tutorial\xspec
systemId=file:/C:/Users/User/Desktop/xspec/src/compiler/generate-tests-utils.xsl

computedGenerateTestUtilsName=../src/compiler/generate-tests-utils.xsl
computedIgnoreDir=../tutorial/xspec
systemId=file:/home/user/test/xspec/test/../tutorial/xspec/escape-for-regex.xsl

To get them work, the following tweaks or something like them would be necessary, perhaps? (Just my quick thought, and I scarcely know Java. So please do what you think is the best.)

• com.jenitennison.xsl.tests.coverage.generate-tests-utils: File(value).toURI().normalize()
• com.jenitennison.xsl.tests.coverage.ignore-dir: File(value + File.separator).toURI().normalize()
• systemId: URI(value).normalize()