Remove obsolete mix_stderr=False

This commit is contained in:
Simon Willison 2025-09-30 14:33:24 -07:00
commit d87bd12dbc
3 changed files with 5 additions and 5 deletions

View file

@ -309,7 +309,7 @@ async def test_auth_with_dstok_token(ds_client, scenario, should_work):
@pytest.mark.parametrize("expires", (None, 1000, -1000))
def test_cli_create_token(event_loop, app_client, expires):
secret = app_client.ds._secret
runner = CliRunner(mix_stderr=False)
runner = CliRunner()
args = ["create-token", "--secret", secret, "test"]
if expires:
args += ["--expires-after", str(expires)]

View file

@ -218,7 +218,7 @@ def test_version():
@pytest.mark.parametrize("invalid_port", ["-1", "0.5", "dog", "65536"])
def test_serve_invalid_ports(invalid_port):
runner = CliRunner(mix_stderr=False)
runner = CliRunner()
result = runner.invoke(cli, ["--port", invalid_port])
assert result.exit_code == 2
assert "Invalid value for '-p'" in result.stderr
@ -304,7 +304,7 @@ def test_plugin_s_overwrite():
def test_setting_type_validation():
runner = CliRunner(mix_stderr=False)
runner = CliRunner()
result = runner.invoke(cli, ["--setting", "default_page_size", "dog"])
assert result.exit_code == 2
assert '"settings.default_page_size" should be an integer' in result.stderr
@ -333,7 +333,7 @@ def test_setting_default_allow_sql(default_allow_sql):
def test_sql_errors_logged_to_stderr():
runner = CliRunner(mix_stderr=False)
runner = CliRunner()
result = runner.invoke(cli, ["--get", "/_memory/-/query.json?sql=select+blah"])
assert result.exit_code == 1
assert "sql = 'select blah', params = {}: no such column: blah\n" in result.stderr

View file

@ -45,7 +45,7 @@ def test_crossdb_warning_if_too_many_databases(tmp_path_factory):
conn = sqlite3.connect(path)
conn.execute("vacuum")
dbs.append(path)
runner = CliRunner(mix_stderr=False)
runner = CliRunner()
result = runner.invoke(
cli,
[