diff --git a/EvoScientist/onboard.py b/EvoScientist/onboard.py index 06809a0..8a1715b 100644 --- a/EvoScientist/onboard.py +++ b/EvoScientist/onboard.py @@ -655,7 +655,7 @@ def _detect_node_install_method() -> tuple[str, str]: Returns: Tuple of (method_name, install_command). """ - # Check if inside a conda environment + # Conda environment (any platform) if os.environ.get("CONDA_PREFIX"): return "conda", "conda install -y nodejs" diff --git a/tests/test_onboard.py b/tests/test_onboard.py index 078cc76..72539b2 100644 --- a/tests/test_onboard.py +++ b/tests/test_onboard.py @@ -32,10 +32,6 @@ class TestConstants: from prompt_toolkit.styles import Style assert isinstance(WIZARD_STYLE, Style) - def test_wizard_style_has_required_keys(self): - """Test that WIZARD_STYLE defines expected style classes.""" - assert WIZARD_STYLE is not None - def test_confirm_style_is_style_instance(self): """Test that CONFIRM_STYLE is a prompt_toolkit Style.""" from prompt_toolkit.styles import Style @@ -69,12 +65,6 @@ class TestRenderProgress: for step in STEPS: assert step in content_str - def test_renders_all_steps_completed(self): - """Test rendering when all steps are completed.""" - panel = render_progress(current_step=5, completed={0, 1, 2, 3, 4, 5}) - content_str = str(panel.renderable) - assert "Parameters" in content_str - def test_panel_has_title(self): """Test that the panel has the expected title.""" panel = render_progress(current_step=0, completed=set()) diff --git a/tests/test_stream_state.py b/tests/test_stream_state.py index 1a054c5..6128621 100644 --- a/tests/test_stream_state.py +++ b/tests/test_stream_state.py @@ -522,16 +522,6 @@ class TestParseTodoItemsAdvanced: assert len(result) == 1 assert result[0]["content"] == "Do X" - def test_multiple_items(self): - import json - items = [ - {"status": "done", "content": "A"}, - {"status": "active", "content": "B"}, - {"status": "todo", "content": "C"}, - ] - result = _parse_todo_items(json.dumps(items)) - assert len(result) == 3 - def test_non_list_json(self): """JSON object (not list) should return None.""" assert _parse_todo_items('{"status": "todo"}') is None diff --git a/tests/test_stream_utils.py b/tests/test_stream_utils.py index f67cb5b..209e6e5 100644 --- a/tests/test_stream_utils.py +++ b/tests/test_stream_utils.py @@ -98,10 +98,9 @@ class TestFormatToolCompact: def test_write_file_memory(self): result = format_tool_compact("write_file", {"path": "/MEMORY.md"}) assert result == "Updating memory" - - def test_write_file_memory_with_prefix(self): - result = format_tool_compact("write_file", {"path": "/memory/MEMORY.md"}) - assert result == "Updating memory" + # Also covers paths with /memory/ prefix + result2 = format_tool_compact("write_file", {"path": "/memory/MEMORY.md"}) + assert result2 == "Updating memory" def test_edit_file_memory(self): result = format_tool_compact("edit_file", {"path": "/memory/MEMORY.md"})