Documentation/errors: provide a forward-path for the --junitxml deprecation

[Liz4v]

With the deprecation of junitxml, we are only provided with a way to silence the warning, but no clear path to act on it in a way that isn't just kicking the can down the road. The deprecation warning should include a future-ready solution as well even if that's just a link to the external plugin or whatever.

[Liz4v]

Ah… I've finally figure out that the warning is because I don't have this value set at all! I guess explicitly saying so in the error message would help?

[Zac-HD]

Looks like the warning is defined here:

pytest/src/_pytest/deprecated.py

Lines 38 to 41 in d1eb89d

	JUNIT_XML_DEFAULT_FAMILY = PytestDeprecationWarning(
	"The 'junit_family' default value will change to 'xunit2' in pytest 6.0.\n"
	"Add 'junit_family=legacy' to your pytest.ini file to silence this warning and make your suite compatible."
	)

and used here:

pytest/src/_pytest/junitxml.py

Lines 434 to 437 in d1eb89d

	junit_family = config.getini("junit_family")
	if not junit_family:
	_issue_warning_captured(deprecated.JUNIT_XML_DEFAULT_FAMILY, config.hook, 2)
	junit_family = "xunit1"

I think it would be sufficient to expand the message:

JUNIT_XML_DEFAULT_FAMILY = PytestDeprecationWarning( 
    "You have not set the 'junit_family' config value, which currently defaults to 'xunit1'.\n" 
    "The 'junit_family' default value will change to 'xunit2' in pytest 6.0.\n" 
    "Add 'junit_family=legacy' (or another value) to your pytest.ini file to "
    "silence this warning and make your suite compatible." 
) 

[fperrin]

Hi @Zac-HD ,

I think it would be sufficient to expand the message:

JUNIT_XML_DEFAULT_FAMILY = PytestDeprecationWarning( 
    "You have not set the 'junit_family' config value, which currently defaults to 'xunit1'.\n" 
    "The 'junit_family' default value will change to 'xunit2' in pytest 6.0.\n" 
    "Add 'junit_family=legacy' (or another value) to your pytest.ini file to "
    "silence this warning and make your suite compatible." 
) 

I'm not sure I follow the meaning of the message. What does "legacy (or another value)" mean? Shouldn't the recommendation be to explicitly set the current default? Ie. the last sentence whould read : « Add 'junit_family=xunit1' to your pytest.ini file to keep the current format in future versions of pytest and silence this warning ».

[nicoddemus]

@fperrin I agree 👍

[Zac-HD]

Also agree, I'd just made the minimal edit without thinking too much 😅

[nicoddemus]

Thanks everyone: #6343 👍

[fperrin]

Thanks @nicoddemus