From d87bd12dbc86846fbdeef0b920d1a98162087fb4 Mon Sep 17 00:00:00 2001
From: Simon Willison <swillison@gmail.com>
Date: Tue, 30 Sep 2025 14:33:24 -0700
Subject: [PATCH] Remove obsolete mix_stderr=False

---
 tests/test_auth.py    | 2 +-
 tests/test_cli.py     | 6 +++---
 tests/test_crossdb.py | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/tests/test_auth.py b/tests/test_auth.py
index aa78a5de..e05c6a62 100644
--- a/tests/test_auth.py
+++ b/tests/test_auth.py
@@ -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)]
diff --git a/tests/test_cli.py b/tests/test_cli.py
index 260f317d..83265a82 100644
--- a/tests/test_cli.py
+++ b/tests/test_cli.py
@@ -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
diff --git a/tests/test_crossdb.py b/tests/test_crossdb.py
index 58b81f70..bc4eaf22 100644
--- a/tests/test_crossdb.py
+++ b/tests/test_crossdb.py
@@ -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,
         [