Skip to content

special characters are not escaped in UT_XUNIT_REPORTER #499

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

Closed
MickeVr opened this issue Oct 23, 2017 · 0 comments
Closed

special characters are not escaped in UT_XUNIT_REPORTER #499

MickeVr opened this issue Oct 23, 2017 · 0 comments
Assignees
Labels
Milestone

Comments

@MickeVr
Copy link

MickeVr commented Oct 23, 2017

I am using utPLSQL-sql-cli to let the UT_XUNIT_REPORTER create a file with the results of the utplsql3 unit tests. My Jenkins pipeline project should pick up the generated xml, but is giving an arror after I have added a test with a description that contains an "<" character, e.g.
-- %test(Test a case where A <> B)

Jenkins reports the following error:
Failed to read test report file C:\Jenkins\workspace\A\utPLSQL-sql-cli\unit_test_output.xml org.dom4j.DocumentException: Error on line 1246 of document : The value of attribute "name" associated with an element type "testcase" must not contain the '<' character.

I would expect that I can use special XML characters in the name of a test, and that the reporter will escape them.

@jgebal jgebal added this to the v3.1.0 milestone Oct 23, 2017
@jgebal jgebal added the bug label Oct 23, 2017
jgebal added a commit that referenced this issue Oct 24, 2017
Resolves #499

Unified reporting of failures across reporters.
With that change, expectation results are only kept in `ut_test` for failed expectations.
XUnit reporter now reports also the line of the test where an expectation has failed.
Resolves #487
@jgebal jgebal self-assigned this Oct 24, 2017
@ghost ghost removed the in progress label Oct 24, 2017
@jgebal jgebal modified the milestones: v3.1.0, v3.0.4 Nov 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants