add CLI arg value if not present (--junitxml)

[choppsv1]

I need to be able to add a dynamic default value for --junitxml, if it is not present based on other ini an cil args present. Currently I am using --addopts --junitxml=/tmp/default.xml in the an pytest.ini file.

I have another ini/cli option I add, rundir, which sets up where the run will happen and logs are placed, and so I'd like to dynamically set the junitxml, if unset to inside rundir/default.xml.

The place to do this seemed to be in my pytest_configure() hook in conftest.py where I process the rundir value. However I can't figure out how to add junit to the config at that point. Indeed is it too late to set junitxml in any case?

How can I manage to do this?

[nicoddemus]

Hi @choppsv1,

The internal JUnitXML plugin also checks the options in pytest_configure:

pytest/src/_pytest/junitxml.py

Lines 426 to 427 in 2439729

	def pytest_configure(config: Config) -> None:
	xmlpath = config.option.xmlpath

Given that external plugins execute first, and then internal ones, your pytest_configure() should be able to set config.option.xmlpath before junitxml plugin gets to it. Perhaps you are setting config.option.junitxml (after the command-line argument)?

[RonnyPfannschmidt]

i recall its possible with a @pytest.hookimpl(tryfirst=True) to set/force that value

[choppsv1]

Hi @choppsv1,

The internal JUnitXML plugin also checks the options in pytest_configure:

pytest/src/_pytest/junitxml.py

Lines 426 to 427 in 2439729

	def pytest_configure(config: Config) -> None:
	xmlpath = config.option.xmlpath

Given that external plugins execute first, and then internal ones, your pytest_configure() should be able to set config.option.xmlpath before junitxml plugin gets to it. Perhaps you are setting config.option.junitxml (after the command-line argument)?

This worked great, thanks!

    rundir = config.getini("rundir")
    if not rundir:
        rundir = "/tmp/topotests"
    if not config.getoption("--junitxml"):
        config.option.xmlpath = os.path.join(rundir, "topotests.xml")

[nicoddemus]

Cool. Can you mark my reply as "answer"? This helps others find it in the future. 👍