scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-11-12 08:42:18 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 207

test(cli): drop the browser launcher test that started a subprocess

Browse source
This commit is contained in:
mateo-berri 2026-10-16 14:45:06 -07:00
parent f883b06457
commit 8c90ab6534
1 changed files with 1 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
tests/unit/proxy/client/cli/test_auto_sign_in.py
View file

@ -10,7 +10,7 @@ from click.testing import CliRunner, Result
from litellm.proxy.client.cli import cli
from litellm.proxy.client.cli.commands.auth import CliTokenData, get_stored_api_key, save_token
from litellm.proxy.client.cli.commands.auto_sign_in import open_browser_off_stdout, pkce_browser_sign_in
from litellm.proxy.client.cli.commands.auto_sign_in import pkce_browser_sign_in
from litellm.proxy.client.cli.commands.pkce_login import PkceCredential, PkceFailure, pkce_token_record
BASE_URL: Final = "https://proxy.example.com"
@ -215,19 +215,6 @@ def test_a_browser_the_user_picked_starts_the_sign_in_and_names_how_long_it_wait
]
def test_the_browser_launcher_keeps_stdout_clean_and_never_rings_the_terminal_bell(
capfd: pytest.CaptureFixture[str], monkeypatch: pytest.MonkeyPatch, tmp_path: Path
):
chatty_browser = tmp_path / "chatty-browser"
chatty_browser.write_text('#!/bin/sh\necho "Opening in existing browser session."\n')
chatty_browser.chmod(0o755)
monkeypatch.setenv("BROWSER", str(chatty_browser))
open_browser_off_stdout(f"{BASE_URL}/v1/cli/authorize?state=fake")
assert capfd.readouterr() == ("", "Opening in existing browser session.\n")
@pytest.mark.parametrize(
("command", "method", "path", "body"),
[