Add option to override test suite name

[fmartinsons]

Because the default might not be suitable for all usage
(full input file path without extenstion)

Signed-off-by: Frederic Martinsons frederic.martinsons@sigfox.com

[fmartinsons]

For information, please do not merge this one already (first I'll rebase and second I search how to improve xml formatting).
For the record, I work with a C test suite made using glib tests framework which output TAP (https://developer.gnome.org/glib/stable/glib-Testing.html) and I look for presenting result in junit under jenkins (thanks to https://plugins.jenkins.io/junit/).

To compare I have an xml produced by pytest framework (pytest_xml.txt

which gives this structure in jenkins:

I have also an xml (sfxlib_xml.txt produced by tap2junit from the following tap file (sfxlib_tap.txt which gives this output:

I don't understand the presences of testsuites at top of the file generated by tap2junit (while the one with pytest don't have it) and also I think the use of classname attribute may help the output (because like you can see in the screenshot, I have this ugly "(root)" at first)

[cclauss]

I am not a fan of naming the test to be something different than the root of the basename of in_file because...

Explicit is better than implicit. -- Zen of Python

Why not just extract name from in_file?

import os
os.path.splitext(os.path.basename("a/b/c.txt"))[0]  # 'c'

[fmartinsons]

I am not a fan of naming the test to be something different than the root of the basename of in_file because...

Explicit is better than implicit. -- Zen of Python

Why not just extract name from in_file?

import os
os.path.splitext(os.path.basename("a/b/c.txt"))[0]  # 'c'

I don't see that to be a implicit vs explicit problem but to let the user choose if he wants to (because there is no special reason for using the filename currently being parsed for the testsuite name, they can be totally unrelated to what the testuiste name is).
By the way this is what pytest do for junit output , see junit_suite_name).
But what you suggest can be applied for the default behavior when no name is given. What do you think ?

[fmartinsons]

Hello @cclauss , any chance to land this merge request ? Is my suggestion of changing the default value of test suite name and let the option to the user suits you ?

[fmartinsons]

Any news here ?