mirror of
https://github.com/open-jarvis/OpenJarvis.git
synced 2026-07-29 18:40:38 +00:00
cc1e14f1c3
- Engine discovery (#73): get_engine() now falls back to any healthy engine when the explicitly-requested key fails, instead of returning None. Fixes LM Studio (and other non-default engines) not being found when deploying agents. - FTS5 case sensitivity & scoring (#67): Add explicit unicode61 tokenizer to the FTS5 virtual table for case-insensitive search. Replace ambiguous `rank` column with explicit `bm25()` call with column weights (id=0, content=1, source=0.5) to produce correct positive scores. Includes auto-migration for existing databases. - Interactive engine selection (#72): `jarvis init` now detects running engines and presents an interactive picker. Also accepts `--engine` flag to skip the prompt. Engine choice flows through to config generation. - Windows desktop support (#68): Add RAM detection via wmic, Windows binary paths (LOCALAPPDATA, ProgramFiles, cargo), port cleanup via netstat+taskkill, and USERPROFILE fallback for HOME env var. Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
136 lines
4.8 KiB
Python
136 lines
4.8 KiB
Python
"""Tests for ``jarvis init`` next-steps guidance."""
|
|
|
|
from __future__ import annotations
|
|
|
|
from pathlib import Path
|
|
from unittest import mock
|
|
|
|
from click.testing import CliRunner
|
|
|
|
from openjarvis.cli import cli
|
|
from openjarvis.cli.init_cmd import _next_steps_text
|
|
|
|
|
|
class TestInitShowsNextSteps:
|
|
def test_init_shows_next_steps(self, tmp_path: Path) -> None:
|
|
"""Init command prints next-steps panel after writing config."""
|
|
config_dir = tmp_path / ".openjarvis"
|
|
config_path = config_dir / "config.toml"
|
|
with (
|
|
mock.patch(
|
|
"openjarvis.cli.init_cmd.DEFAULT_CONFIG_DIR", config_dir
|
|
),
|
|
mock.patch(
|
|
"openjarvis.cli.init_cmd.DEFAULT_CONFIG_PATH", config_path
|
|
),
|
|
):
|
|
result = CliRunner().invoke(cli, ["init", "--engine", "llamacpp"])
|
|
assert result.exit_code == 0
|
|
assert "Getting Started" in result.output
|
|
assert "jarvis ask" in result.output
|
|
assert "jarvis doctor" in result.output
|
|
|
|
def test_init_output_shows_toml_sections_literally(self, tmp_path: Path) -> None:
|
|
"""Init output should render TOML section headers like [engine] literally."""
|
|
config_dir = tmp_path / ".openjarvis"
|
|
config_path = config_dir / "config.toml"
|
|
with (
|
|
mock.patch(
|
|
"openjarvis.cli.init_cmd.DEFAULT_CONFIG_DIR", config_dir
|
|
),
|
|
mock.patch(
|
|
"openjarvis.cli.init_cmd.DEFAULT_CONFIG_PATH", config_path
|
|
),
|
|
):
|
|
result = CliRunner().invoke(cli, ["init", "--engine", "llamacpp"])
|
|
assert result.exit_code == 0
|
|
assert "[engine]" in result.output
|
|
assert "[intelligence]" in result.output
|
|
|
|
|
|
class TestNextStepsOllama:
|
|
def test_next_steps_ollama(self) -> None:
|
|
text = _next_steps_text("ollama")
|
|
assert "ollama serve" in text
|
|
assert "ollama pull" in text
|
|
assert "jarvis ask" in text
|
|
assert "jarvis doctor" in text
|
|
|
|
def test_next_steps_ollama_with_model(self) -> None:
|
|
text = _next_steps_text("ollama", "qwen3.5:14b")
|
|
assert "ollama pull qwen3.5:14b" in text
|
|
|
|
def test_next_steps_ollama_default_model(self) -> None:
|
|
text = _next_steps_text("ollama")
|
|
assert "ollama pull qwen3.5:3b" in text
|
|
|
|
|
|
class TestNextStepsVllm:
|
|
def test_next_steps_vllm(self) -> None:
|
|
text = _next_steps_text("vllm")
|
|
assert "pip install vllm" in text
|
|
assert "vllm serve" in text
|
|
assert "jarvis ask" in text
|
|
assert "jarvis doctor" in text
|
|
|
|
|
|
class TestNextStepsLlamacpp:
|
|
def test_next_steps_llamacpp(self) -> None:
|
|
text = _next_steps_text("llamacpp")
|
|
assert "brew install llama.cpp" in text
|
|
assert "llama-server" in text
|
|
assert "jarvis ask" in text
|
|
assert "jarvis doctor" in text
|
|
|
|
|
|
class TestNextStepsMlx:
|
|
def test_next_steps_mlx(self) -> None:
|
|
text = _next_steps_text("mlx")
|
|
assert "pip install mlx-lm" in text
|
|
assert "mlx_lm.server" in text
|
|
assert "jarvis ask" in text
|
|
assert "jarvis doctor" in text
|
|
|
|
|
|
class TestMinimalConfig:
|
|
def test_init_generates_minimal_by_default(self, tmp_path: Path) -> None:
|
|
"""Default jarvis init produces a short config."""
|
|
config_dir = tmp_path / ".openjarvis"
|
|
config_path = config_dir / "config.toml"
|
|
with (
|
|
mock.patch(
|
|
"openjarvis.cli.init_cmd.DEFAULT_CONFIG_DIR", config_dir
|
|
),
|
|
mock.patch(
|
|
"openjarvis.cli.init_cmd.DEFAULT_CONFIG_PATH", config_path
|
|
),
|
|
):
|
|
result = CliRunner().invoke(cli, ["init", "--engine", "ollama"])
|
|
assert result.exit_code == 0
|
|
content = config_path.read_text()
|
|
# Minimal config should be short
|
|
lines = [ln for ln in content.splitlines() if ln.strip()]
|
|
assert len(lines) <= 30
|
|
# Should have the reference hint
|
|
assert "jarvis init --full" in content
|
|
|
|
def test_init_full_generates_verbose_config(self, tmp_path: Path) -> None:
|
|
"""jarvis init --full produces the full reference config."""
|
|
config_dir = tmp_path / ".openjarvis"
|
|
config_path = config_dir / "config.toml"
|
|
with (
|
|
mock.patch(
|
|
"openjarvis.cli.init_cmd.DEFAULT_CONFIG_DIR", config_dir
|
|
),
|
|
mock.patch(
|
|
"openjarvis.cli.init_cmd.DEFAULT_CONFIG_PATH", config_path
|
|
),
|
|
):
|
|
result = CliRunner().invoke(cli, ["init", "--full", "--engine", "ollama"])
|
|
assert result.exit_code == 0
|
|
content = config_path.read_text()
|
|
# Full config should have many sections
|
|
assert "[engine.ollama]" in content
|
|
assert "[server]" in content
|
|
assert "[security]" in content
|