core: pre-commit

Author: simonprovost

tests/conftest.py

@@ -1,7 +1,7 @@
+import contextlib
 import sys
 import types
-import contextlib
-from typing import Iterable
+from collections.abc import Iterable
 
 import pytest
 
@@ -11,6 +11,7 @@
 def _make_fn(name: str):
     def _fn(*args, **kwargs):
         return {"__tool_name__": name}
+
     _fn.__name__ = name
     return _fn
 
@@ -53,4 +54,4 @@ def install_fake_jupyter_server(tool_names: Iterable[str]):
 
 @pytest.fixture(scope="session")
 def known_tools():
-    return load_known_tools()["known_tools"]
+    return load_known_tools()["known_tools"]

tests/test_config_loader.py

@@ -1,9 +1,10 @@
 from mcpstack_jupyter.tools.jupyter.utils.config_loader import (
-    load_known_tools,
-    load_env_defaults,
     load_cli_defaults,
+    load_env_defaults,
+    load_known_tools,
 )
 
+
 def test_known_tools_and_read_only_lists():
     data = load_known_tools()
     known = data["known_tools"]
@@ -13,9 +14,10 @@ def test_known_tools_and_read_only_lists():
     assert "read_all_cells" in known
     assert set(ro).issubset(set(known))
 
+
 def test_env_and_cli_defaults_present():
     env = load_env_defaults()
     cli = load_cli_defaults()
     assert "provider" in env
     assert "document_url" in env
-    assert "output_filename" in cli
+    assert "output_filename" in cli

tests/test_jupyter_bindings.py

@@ -41,14 +41,23 @@ def test_missing_upstream_tool_raises_attribute_error(known_tools):
 def test_env_mapping_defaults_and_overrides(monkeypatch, known_tools):
     defaults = load_env_defaults()
     with install_fake_jupyter_server(known_tools) as server:
-        for k in ["PROVIDER","DOCUMENT_URL","DOCUMENT_ID","DOCUMENT_TOKEN",
-                  "RUNTIME_URL","RUNTIME_ID","RUNTIME_TOKEN"]:
+        for k in [
+            "PROVIDER",
+            "DOCUMENT_URL",
+            "DOCUMENT_ID",
+            "DOCUMENT_TOKEN",
+            "RUNTIME_URL",
+            "RUNTIME_ID",
+            "RUNTIME_TOKEN",
+        ]:
             monkeypatch.delenv(k, raising=False)
         tool = Jupyter()
         tool.initialize()
 
         assert server.PROVIDER == defaults.get("provider", "jupyter")
-        assert server.DOCUMENT_URL == defaults.get("document_url", "http://127.0.0.1:8888")
+        assert server.DOCUMENT_URL == defaults.get(
+            "document_url", "http://127.0.0.1:8888"
+        )
         assert server.DOCUMENT_ID == defaults.get("document_id", "notebook.ipynb")
         assert server.RUNTIME_URL == defaults.get("runtime_url") or server.DOCUMENT_URL
 
@@ -76,4 +85,4 @@ def test_env_mapping_defaults_and_overrides(monkeypatch, known_tools):
 def test_to_from_dict_roundtrip():
     src = {"include": ["read_all_cells", "get_notebook_info"]}
     tool = Jupyter.from_dict(src)
-    assert tool.to_dict() == src
+    assert tool.to_dict() == src

tests/test_status_cli.py

@@ -19,4 +19,4 @@ def test_status_verbose_lists_availability(known_tools, capsys):
         JupyterCLI.status(verbose=True)
         out = capsys.readouterr().out
         assert "Upstream Tool Availability" in out
-        assert re.search(r"\byes\b", out, re.IGNORECASE)
+        assert re.search(r"\byes\b", out, re.IGNORECASE)