fix: isolate tests from real KB directories via mocking

Author: KylinMountain

tests/test_add_command.py

@@ -37,8 +37,9 @@ def test_finds_openkb_dir(self, tmp_path, monkeypatch):
 
     def test_returns_none_if_no_openkb(self, tmp_path, monkeypatch):
         monkeypatch.chdir(tmp_path)
-        result = _find_kb_dir()
-        assert result is None
+        with patch("openkb.cli.load_global_config", return_value={}):
+            result = _find_kb_dir()
+            assert result is None
 
 
 class TestAddCommand:

tests/test_cli.py

@@ -1,4 +1,6 @@
 import json
+from unittest.mock import patch
+
 import pytest
 from click.testing import CliRunner
 
@@ -8,7 +10,8 @@
 
 def test_init_creates_structure(tmp_path):
     runner = CliRunner()
-    with runner.isolated_filesystem(temp_dir=tmp_path):
+    with runner.isolated_filesystem(temp_dir=tmp_path), \
+         patch("openkb.cli.register_kb"):
         result = runner.invoke(cli, ["init"])
         assert result.exit_code == 0
 
@@ -42,7 +45,8 @@ def test_init_creates_structure(tmp_path):
 
 def test_init_schema_content(tmp_path):
     runner = CliRunner()
-    with runner.isolated_filesystem(temp_dir=tmp_path):
+    with runner.isolated_filesystem(temp_dir=tmp_path), \
+         patch("openkb.cli.register_kb"):
         result = runner.invoke(cli, ["init"])
         assert result.exit_code == 0
 
@@ -53,7 +57,8 @@ def test_init_schema_content(tmp_path):
 
 def test_init_already_exists(tmp_path):
     runner = CliRunner()
-    with runner.isolated_filesystem(temp_dir=tmp_path):
+    with runner.isolated_filesystem(temp_dir=tmp_path), \
+         patch("openkb.cli.register_kb"):
         # First run should succeed
         result = runner.invoke(cli, ["init"])
         assert result.exit_code == 0

tests/test_list_status.py

@@ -32,7 +32,8 @@ def _setup_kb(tmp_path: Path) -> Path:
 class TestListCommand:
     def test_list_no_kb(self, tmp_path):
         runner = CliRunner()
-        with runner.isolated_filesystem(temp_dir=tmp_path):
+        with runner.isolated_filesystem(temp_dir=tmp_path), \
+             patch("openkb.cli._find_kb_dir", return_value=None):
             result = runner.invoke(cli, ["list"])
             assert "No knowledge base found" in result.output
 
@@ -91,7 +92,8 @@ def test_list_no_concepts_section_when_empty(self, tmp_path):
 class TestStatusCommand:
     def test_status_no_kb(self, tmp_path):
         runner = CliRunner()
-        with runner.isolated_filesystem(temp_dir=tmp_path):
+        with runner.isolated_filesystem(temp_dir=tmp_path), \
+             patch("openkb.cli._find_kb_dir", return_value=None):
             result = runner.invoke(cli, ["status"])
             assert "No knowledge base found" in result.output