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

XUnit reporter does not fill the classname tag due to missing lower conversion #659

Closed
nrogachev opened this Issue May 4, 2018 · 0 comments

Comments

Projects
None yet
3 participants
@nrogachev
Copy link

nrogachev commented May 4, 2018

Looks like that after upgrade to 3.1.1 new XUnit reporter (ut_junit_reporter) supports only suites where test case names (procedure names) are strictly lower case.
The reason seems to be that somewhere in suit builder conversion to lower case happens only partially.

My test package looks like this:

CREATE OR REPLACE PACKAGE TestPkg1
AS

-- %suite(Some desc)
-- %suitepath(de.suit1)

-- %test(My Test 1)
PROCEDURE Test1;

END TestPkg1;

I've added some debugging in the reporter type in ut_junit_reporter.print_test_elements in the very beginning:
raise_application_error(-20666,a_test.path||'/'||a_test.name);

and see the following:
de.suit1.testpkg1.Test1/test1

Subsequently the get_path fails to find a path and classname stays unfilled. And without classname the whole report becomes unusable.

If the procedure is renamed to test1 than everything works. But in 3.0.4 there was no such restriction.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.