Dependencies: Permit installation of click 8.3

Author: amotl

CHANGES.md

@@ -2,6 +2,8 @@
 
 ## Unreleased
 
+- Dependencies: Permitted installation of click 8.3
+
 ## 2026/03/16 v0.0.46
 - I/O: API improvements: `ctk {load,save} table` became `ctk {load,save}`
   Also, the `--cluster-url` option becomes optional. That's a much more

cratedb_toolkit/cfr/cli.py

@@ -109,7 +109,7 @@ def job_statistics(ctx: click.Context):
 @click.option(
     "--anonymize",
     type=str,
-    is_flag=False,
+    is_flag=True,
     flag_value="decoder_dictionary.json",  # Use this value when flag is used without value
     default=None,  # No anonymization by default
     help="Path to the decoder dictionary file for anonymizing SQL statements",

pyproject.toml

@@ -93,7 +93,7 @@ dependencies = [
   "attrs<27",
   "boltons<26",
   "cattrs<27",
-  "click<8.2",
+  "click<8.4",
   "click-aliases>=1.0.4,<2",
   "colorama<1",
   "colorlog",

tests/cfr/test_systable.py

@@ -1,4 +1,5 @@
 # ruff: noqa: E402
+import importlib
 import json
 import os.path
 import re
@@ -7,6 +8,7 @@
 import tarfile
 
 import pytest
+from verlib2 import Version
 
 pymongo = pytest.importorskip("polars", reason="Skipping tests because polars is not installed")
 
@@ -29,15 +31,25 @@ def filenames(path: Path):
     return sorted([item.name for item in path.iterdir()])
 
 
-def test_cfr_sys_export_success(cratedb, tmp_path, caplog):
+@pytest.fixture(scope="session")
+def click_kwargs():
+    """
+    Click 8.2 no longer understands `mix_stderr`.
+    """
+    kwargs = {}
+    click_version = importlib.metadata.version("click")
+    if Version(click_version) < Version("8.2"):
+        kwargs = {"mix_stderr": False}
+    return kwargs
+
+
+def test_cfr_sys_export_success(cratedb, click_kwargs, tmp_path, caplog):
     """
     Verify `ctk cfr sys-export` works.
     """
 
     # Invoke command.
-    runner = CliRunner(
-        env={"CRATEDB_CLUSTER_URL": cratedb.database.dburi, "CFR_TARGET": str(tmp_path)}, mix_stderr=False
-    )
+    runner = CliRunner(env={"CRATEDB_CLUSTER_URL": cratedb.database.dburi, "CFR_TARGET": str(tmp_path)}, **click_kwargs)
     result = runner.invoke(
         cli,
         args="--debug sys-export",
@@ -60,17 +72,15 @@ def test_cfr_sys_export_success(cratedb, tmp_path, caplog):
     assert len(data_files) >= 10
 
 
-def test_cfr_sys_export_to_archive_file(cratedb, tmp_path, caplog):
+def test_cfr_sys_export_to_archive_file(cratedb, click_kwargs, tmp_path, caplog):
     """
     Verify `ctk cfr sys-export some-file.tgz` works.
     """
 
     target = os.path.join(tmp_path, "cluster-data.tgz")
 
     # Invoke command.
-    runner = CliRunner(
-        env={"CRATEDB_CLUSTER_URL": cratedb.database.dburi, "CFR_TARGET": str(tmp_path)}, mix_stderr=False
-    )
+    runner = CliRunner(env={"CRATEDB_CLUSTER_URL": cratedb.database.dburi, "CFR_TARGET": str(tmp_path)}, **click_kwargs)
     result = runner.invoke(
         cli,
         args=f"--debug sys-export {target}",
@@ -100,13 +110,13 @@ def test_cfr_sys_export_to_archive_file(cratedb, tmp_path, caplog):
     assert len(data_files) >= 10
 
 
-def test_cfr_sys_export_failure(cratedb, tmp_path, caplog):
+def test_cfr_sys_export_failure(cratedb, click_kwargs, tmp_path, caplog):
     """
     Verify `ctk cfr sys-export` failure.
     """
 
     # Invoke command.
-    runner = CliRunner(env={"CRATEDB_CLUSTER_URL": "crate://foo.bar/", "CFR_TARGET": str(tmp_path)}, mix_stderr=False)
+    runner = CliRunner(env={"CRATEDB_CLUSTER_URL": "crate://foo.bar/", "CFR_TARGET": str(tmp_path)}, **click_kwargs)
     result = runner.invoke(
         cli,
         args="--debug sys-export",
@@ -119,10 +129,8 @@ def test_cfr_sys_export_failure(cratedb, tmp_path, caplog):
     assert result.output == ""
 
 
-def test_cfr_sys_export_ensure_table_name_is_quoted(cratedb, tmp_path, caplog):
-    runner = CliRunner(
-        env={"CRATEDB_CLUSTER_URL": cratedb.database.dburi, "CFR_TARGET": str(tmp_path)}, mix_stderr=False
-    )
+def test_cfr_sys_export_ensure_table_name_is_quoted(cratedb, click_kwargs, tmp_path, caplog):
+    runner = CliRunner(env={"CRATEDB_CLUSTER_URL": cratedb.database.dburi, "CFR_TARGET": str(tmp_path)}, **click_kwargs)
     result = runner.invoke(
         cli,
         args="--debug sys-export",
@@ -178,9 +186,7 @@ def test_cfr_sys_import_success(cratedb, tmp_path, caplog):
     shutil.copy(sys_operations_data, data_path)
 
     # Invoke command.
-    runner = CliRunner(
-        env={"CRATEDB_CLUSTER_URL": cratedb.database.dburi, "CFR_SOURCE": str(tmp_path)}, mix_stderr=False
-    )
+    runner = CliRunner(env={"CRATEDB_CLUSTER_URL": cratedb.database.dburi, "CFR_SOURCE": str(tmp_path)})
     result = runner.invoke(
         cli,
         args="--debug sys-import",
@@ -200,13 +206,13 @@ def test_cfr_sys_import_success(cratedb, tmp_path, caplog):
     assert cratedb.database.count_records("sys-operations") == 1
 
 
-def test_cfr_sys_import_failure(cratedb, tmp_path, caplog):
+def test_cfr_sys_import_failure(cratedb, click_kwargs, tmp_path, caplog):
     """
     Verify `ctk cfr sys-import` failure.
     """
 
     # Invoke command.
-    runner = CliRunner(env={"CRATEDB_CLUSTER_URL": "crate://foo.bar/", "CFR_SOURCE": str(tmp_path)}, mix_stderr=False)
+    runner = CliRunner(env={"CRATEDB_CLUSTER_URL": "crate://foo.bar/", "CFR_SOURCE": str(tmp_path)}, **click_kwargs)
     result = runner.invoke(
         cli,
         args="--debug sys-import",

tests/settings/test_cli.py

@@ -9,7 +9,7 @@ def test_settings_list(cratedb, caplog):
     """
 
     # Invoke command.
-    runner = CliRunner(env={"CRATEDB_CLUSTER_URL": cratedb.database.dburi}, mix_stderr=False)
+    runner = CliRunner(env={"CRATEDB_CLUSTER_URL": cratedb.database.dburi})
     result = runner.invoke(
         cli,
         args="list",
@@ -26,7 +26,7 @@ def test_settings_compare(cratedb, caplog):
     """
 
     # Invoke command.
-    runner = CliRunner(env={"CRATEDB_CLUSTER_URL": cratedb.database.dburi}, mix_stderr=False)
+    runner = CliRunner(env={"CRATEDB_CLUSTER_URL": cratedb.database.dburi})
     result = runner.invoke(
         cli,
         args="compare --no-color",