diff options
author | Dos Moonen <darsstar@gmail.com> | 2023-04-10 11:05:34 +0200 |
---|---|---|
committer | Dos Moonen <darsstar@gmail.com> | 2023-04-10 11:05:34 +0200 |
commit | ebcc368a517995d341c2b1a5d1dc5fb12437081e (patch) | |
tree | 8eaa400f4e25552b4a98ea0dc27fd03c9d4e512b /tests | |
parent | aebc0c5fc321141ede837c80572427ab7b795c3f (diff) | |
download | pip-ebcc368a517995d341c2b1a5d1dc5fb12437081e.tar.gz |
Alter test_prompt_for_keyring_if_needed to make sure we test the default is `auto`.
Diffstat (limited to 'tests')
-rw-r--r-- | tests/functional/test_install_config.py | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/tests/functional/test_install_config.py b/tests/functional/test_install_config.py index 563b5604a..9f8a80677 100644 --- a/tests/functional/test_install_config.py +++ b/tests/functional/test_install_config.py @@ -370,7 +370,7 @@ def auth_needed(request: pytest.FixtureRequest) -> bool: return request.param -@pytest.fixture(params=("disabled", "import", "subprocess", "auto")) +@pytest.fixture(params=(None, "disabled", "import", "subprocess", "auto")) def keyring_provider(request: pytest.FixtureRequest) -> str: return request.param @@ -389,17 +389,20 @@ def flags( keyring_provider_implementation: str, ) -> List[str]: if ( - keyring_provider != "auto" + keyring_provider not in [None, "auto"] and keyring_provider_implementation != keyring_provider ): pytest.skip() - flags = ["--keyring-provider", keyring_provider] + flags = [] + if keyring_provider is not None: + flags.append("--keyring-provider") + flags.append(keyring_provider) if not interactive: flags.append("--no-input") if auth_needed: if keyring_provider_implementation == "disabled" or ( - not interactive and keyring_provider == "auto" + not interactive and keyring_provider in [None, "auto"] ): request.applymarker(pytest.mark.xfail()) return flags @@ -441,7 +444,10 @@ def test_prompt_for_keyring_if_needed( virtualenv = virtualenv_factory(workspace.joinpath("venv")) script = script_factory(workspace.joinpath("venv"), virtualenv, environ=environ) - if keyring_provider != "auto" or keyring_provider_implementation != "subprocess": + if ( + keyring_provider not in [None, "auto"] + or keyring_provider_implementation != "subprocess" + ): script.pip( "install", "keyring", |