Boost server/ui test coverage: extract testable logic, add 35 new tests

- Extract build_filter_js() from explorer.py for testable JS generation
- Extract get_tool_map() from mcp_console.py for testable tool lookup
- Add tests for _on_page_change, _on_drill_down, _nav_to navigation helpers
- Add tests for _get_tool_fn, get_tool_map, and edge cases in parse/coerce
- Cover components.py line 98-99 (start_line without end_line branch)
- Add test_ui_init.py verifying setup_ui is importable and callable
- Result: components 100%, theme 100%, explorer 33%, mcp_console 38%, total 42%
- Remaining uncovered lines are pure NiceGUI rendering (ui.card, ui.dialog, etc.)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: aksOps

src/osscodeiq/server/ui/explorer.py

@@ -65,25 +65,45 @@ def navigate_to(self, index: int) -> None:
 # Search filter JavaScript (client-side, no server round-trip)
 # ---------------------------------------------------------------------------
 
-_SEARCH_JS = """
-(function(query) {
-    const cards = document.querySelectorAll('.explorer-card');
+_SEARCH_JS_TEMPLATE = """
+(function(query) {{
+    const cards = document.querySelectorAll('{container}');
     const lower = query.toLowerCase();
-    cards.forEach(function(card) {
+    cards.forEach(function(card) {{
         const text = card.textContent.toLowerCase();
-        if (!lower || text.includes(lower)) {
+        if (!lower || text.includes(lower)) {{
             card.style.opacity = '1';
             card.style.pointerEvents = 'auto';
             card.style.display = '';
-        } else {
+        }} else {{
             card.style.opacity = '0.15';
             card.style.pointerEvents = 'none';
-        }
-    });
-})("{query}")
+        }}
+    }});
+}})("{query}")
 """
 
 
+def build_filter_js(query: str, container_selector: str = ".explorer-card") -> str:
+    """Build a JavaScript snippet that filters cards by text content.
+
+    Parameters
+    ----------
+    query:
+        The search string to filter by. Double-quotes are escaped.
+    container_selector:
+        CSS selector for the card elements to filter.
+
+    Returns
+    -------
+    A self-executing JavaScript string.
+    """
+    safe_query = query.replace("\\", "\\\\").replace('"', '\\"')
+    return _SEARCH_JS_TEMPLATE.format(
+        container=container_selector, query=safe_query
+    )
+
+
 # ---------------------------------------------------------------------------
 # Detail modal
 # ---------------------------------------------------------------------------
@@ -332,7 +352,7 @@ def content() -> None:
         search_input.on(
             "update:model-value",
             lambda e: ui.run_javascript(
-                _SEARCH_JS.replace("{query}", str(e.args or "").replace('"', '\\"'))
+                build_filter_js(str(e.args or ""))
             ),
         )
 

src/osscodeiq/server/ui/mcp_console.py

@@ -69,15 +69,21 @@ def parse_mcp_command(raw: str) -> tuple[str, dict[str, Any]]:
 # ── MCP tool lookup table ──────────────────────────────────────────────────
 
 
-def _get_tool_fn(name: str):
-    """Import and return the MCP tool function by *name*, or None."""
+def get_tool_map() -> dict[str, Any]:
+    """Build and return the MCP tool name -> function mapping.
+
+    This is separated from ``_get_tool_fn`` so it can be tested without a
+    NiceGUI context.  The import is deferred so the module can be loaded
+    without the full server stack at import time.
+    """
     from osscodeiq.server.mcp_server import (  # noqa: C0415
         find_callers,
         find_component_by_file,
         find_consumers,
         find_cycles,
         find_dependencies,
         find_dependents,
+        find_producers,
         find_related_endpoints,
         find_shortest_path,
         generate_flow,
@@ -91,7 +97,7 @@ def _get_tool_fn(name: str):
         trace_impact,
     )
 
-    _TOOL_MAP: dict[str, Any] = {
+    return {
         "get_stats": get_stats,
         "query_nodes": query_nodes,
         "query_edges": query_edges,
@@ -100,7 +106,7 @@ def _get_tool_fn(name: str):
         "find_cycles": find_cycles,
         "find_shortest_path": find_shortest_path,
         "find_consumers": find_consumers,
-        "find_producers": find_producers,  # noqa: F841 — not importable separately
+        "find_producers": find_producers,
         "find_callers": find_callers,
         "find_dependencies": find_dependencies,
         "find_dependents": find_dependents,
@@ -112,12 +118,10 @@ def _get_tool_fn(name: str):
         "read_file": read_file,
     }
 
-    # find_producers is imported via find_consumers' sibling — fix the map
-    from osscodeiq.server.mcp_server import find_producers  # noqa: C0415
 
-    _TOOL_MAP["find_producers"] = find_producers
-
-    return _TOOL_MAP.get(name)
+def _get_tool_fn(name: str):
+    """Import and return the MCP tool function by *name*, or None."""
+    return get_tool_map().get(name)
 
 
 # ── Console builder ────────────────────────────────────────────────────────

tests/server/test_ui_components.py

@@ -42,6 +42,11 @@ def test_unknown_kind_gets_defaults(self) -> None:
         assert result["icon"] == "circle"
         assert result["color"].startswith("#")
 
+    def test_missing_count_defaults_zero(self) -> None:
+        kind_info = {"kind": "endpoint"}
+        result = build_kind_card_data(kind_info)
+        assert result["count"] == 0
+
 
 class TestBuildNodeCardData:
     def test_basic_transform(self) -> None:
@@ -93,6 +98,23 @@ def test_missing_optional_fields(self) -> None:
         assert result["module"] is None
         assert result["properties"] == {}
 
+    def test_subtitle_empty_when_no_details(self) -> None:
+        node_info = {
+            "id": "mod:x.py:module:x",
+            "name": "x",
+        }
+        result = build_node_card_data(node_info)
+        assert result["subtitle"] == ""
+
+    def test_edge_count_zero(self) -> None:
+        node_info = {
+            "id": "cls:a.py:class:A",
+            "name": "A",
+            "edge_count": 0,
+        }
+        result = build_node_card_data(node_info)
+        assert "0 edges" in result["subtitle"]
+
 
 class TestBuildDetailData:
     def test_basic_transform(self) -> None:
@@ -172,6 +194,41 @@ def test_location_includes_line_numbers(self) -> None:
         assert "5" in loc_value
         assert "50" in loc_value
 
+    def test_location_with_start_line_only(self) -> None:
+        """Cover the branch where start_line is set but end_line is None (lines 98-99)."""
+        detail = {
+            "id": "cls:app.py:class:Bar",
+            "name": "Bar",
+            "kind": "class",
+            "file_path": "app.py",
+            "start_line": 42,
+            # end_line deliberately omitted
+            "properties": {},
+            "edges_out": [],
+            "edges_in": [],
+        }
+        result = build_detail_data(detail)
+        location_props = [p for p in result["properties"] if p[0] == "Location"]
+        assert len(location_props) == 1
+        loc_value = location_props[0][1]
+        assert loc_value == "app.py:42"
+
+    def test_location_with_file_path_only(self) -> None:
+        """Cover the branch where file_path is set but no line numbers."""
+        detail = {
+            "id": "mod:lib.py:module:lib",
+            "name": "lib",
+            "kind": "module",
+            "file_path": "lib.py",
+            "properties": {},
+            "edges_out": [],
+            "edges_in": [],
+        }
+        result = build_detail_data(detail)
+        location_props = [p for p in result["properties"] if p[0] == "Location"]
+        assert len(location_props) == 1
+        assert location_props[0][1] == "lib.py"
+
     def test_empty_edges(self) -> None:
         detail = {
             "id": "mod:x.py:module:x",
@@ -199,3 +256,14 @@ def test_missing_optional_fields(self) -> None:
         prop_keys = [p[0] for p in result["properties"]]
         # FQN, Module, Layer may be absent but should not error
         assert isinstance(result["properties"], list)
+
+    def test_missing_edges_defaults_empty(self) -> None:
+        detail = {
+            "id": "mod:y.py:module:y",
+            "name": "y",
+            "kind": "module",
+            "properties": {},
+        }
+        result = build_detail_data(detail)
+        assert result["edges_out"] == []
+        assert result["edges_in"] == []