From 67528deb4c570a44b91fdc628853df5fbd1cb051 Mon Sep 17 00:00:00 2001 From: chengyongru <2755839590@qq.com> Date: Tue, 17 Mar 2026 22:20:55 +0800 Subject: [PATCH] fix(tests): use --no-interactive for non-interactive onboard tests Tests for non-interactive onboard mode now explicitly use --no-interactive flag since the default changed to interactive mode. Co-Authored-By: Claude Opus 4.6 --- tests/test_commands.py | 10 +++++----- tests/test_config_migration.py | 4 ++-- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/tests/test_commands.py b/tests/test_commands.py index f140d1f..d374d0c 100644 --- a/tests/test_commands.py +++ b/tests/test_commands.py @@ -61,7 +61,7 @@ def test_onboard_fresh_install(mock_paths): """No existing config — should create from scratch.""" config_file, workspace_dir, mock_ws = mock_paths - result = runner.invoke(app, ["onboard"]) + result = runner.invoke(app, ["onboard", "--no-interactive"]) assert result.exit_code == 0 assert "Created config" in result.stdout @@ -79,7 +79,7 @@ def test_onboard_existing_config_refresh(mock_paths): config_file, workspace_dir, _ = mock_paths config_file.write_text('{"existing": true}') - result = runner.invoke(app, ["onboard"], input="n\n") + result = runner.invoke(app, ["onboard", "--no-interactive"], input="n\n") assert result.exit_code == 0 assert "Config already exists" in result.stdout @@ -93,7 +93,7 @@ def test_onboard_existing_config_overwrite(mock_paths): config_file, workspace_dir, _ = mock_paths config_file.write_text('{"existing": true}') - result = runner.invoke(app, ["onboard"], input="y\n") + result = runner.invoke(app, ["onboard", "--no-interactive"], input="y\n") assert result.exit_code == 0 assert "Config already exists" in result.stdout @@ -107,7 +107,7 @@ def test_onboard_existing_workspace_safe_create(mock_paths): workspace_dir.mkdir(parents=True) config_file.write_text("{}") - result = runner.invoke(app, ["onboard"], input="n\n") + result = runner.invoke(app, ["onboard", "--no-interactive"], input="n\n") assert result.exit_code == 0 assert "Created workspace" not in result.stdout @@ -141,7 +141,7 @@ def test_onboard_uses_explicit_config_and_workspace_paths(tmp_path, monkeypatch) result = runner.invoke( app, - ["onboard", "--config", str(config_path), "--workspace", str(workspace_path)], + ["onboard", "--config", str(config_path), "--workspace", str(workspace_path), "--no-interactive"], ) assert result.exit_code == 0 diff --git a/tests/test_config_migration.py b/tests/test_config_migration.py index 7728c26..28e0feb 100644 --- a/tests/test_config_migration.py +++ b/tests/test_config_migration.py @@ -75,7 +75,7 @@ def test_onboard_refresh_rewrites_legacy_config_template(tmp_path, monkeypatch) from typer.testing import CliRunner from nanobot.cli.commands import app runner = CliRunner() - result = runner.invoke(app, ["onboard"], input="n\n") + result = runner.invoke(app, ["onboard", "--no-interactive"], input="n\n") assert result.exit_code == 0 assert "contextWindowTokens" in result.stdout @@ -127,7 +127,7 @@ def test_onboard_refresh_backfills_missing_channel_fields(tmp_path, monkeypatch) from typer.testing import CliRunner from nanobot.cli.commands import app runner = CliRunner() - result = runner.invoke(app, ["onboard"], input="n\n") + result = runner.invoke(app, ["onboard", "--no-interactive"], input="n\n") assert result.exit_code == 0 saved = json.loads(config_path.read_text(encoding="utf-8"))