Ignore --clean-alluredir when using --collectonly

[ShurikMen]

Context

When displaying a list of tests with the --collectonly key, the last test execution result is cleared.
To prevent the result from being lost, added ignoring cleaning (--clean-alluredir) if there is a key to only output the list of tests ( --collectonly).

Checklist

• Sign Allure CLA
• Provide unit tests

[skhomuti]

Good idea! Also, it makes sense for all args that don't cause test running. --setup-plan for example. Need to find a way to check it properly. Maybe pytest give it somewhere in params or args

[ShurikMen]

Good idea! Also, it makes sense for all args that don't cause test running. --setup-plan for example. Need to find a way to check it properly. Maybe pytest give it somewhere in params or args

For --setup-plan, the situation is a little different, because if with --coollect-only tests are not run at all (immediately exiting pytest_runtestloop without starting pytest_runtest_protocol), then with --setup-plan tests are run in --setup-only mode. All tests are "executed" without full execution. Allure makes a full report on this execution.
Well, --setup-plan remains useful when you need to see how the report will look like without running tests.

[alexterent]

And what about --co ? Will it work?

[skhomuti]

@delatrie could you look at this PR pls?

[delatrie]

Hi! And sorry, it took me so long.

I like this feature!
It would be nice to have tests though. Could you, please, add an acceptance test for each package involved? Something like this should work:

def test_no_clean_if_collectonly(allure_pytest_runner: AllurePytestRunner):
    alluredir = allure_pytest_runner.pytester.path
    allure_pytest_runner.in_memory = False

    allure_pytest_runner.run_pytest(
        "def test_fn(): pass",
        cli_args=["--collectonly", "--clean-alluredir"]
    )

    assert alluredir.joinpath("test_collectonly.py").exists()

And the same goes for allure-pytest-bdd, but the fixture would be allure_pytest_bdd_runner: AllurePytestRunner.
If you have any struggles, feel free to ping me here.

[ShurikMen]

It would be nice to have tests though.

@delatrie done

[delatrie]

Great, thanks!
Extra kudos for increasing test coverage!