test: Add redirection append test and coverage pragmas

Author: tumf

src/mcp_shell_server/shell_executor.py

@@ -15,7 +15,7 @@ def __init__(self):
         """
         Initialize the executor.
         """
-        pass
+        pass  # pragma: no cover
 
     def _get_allowed_commands(self) -> set[str]:
         """Get the set of allowed commands from environment variables"""

tests/test_shell_executor.py

@@ -617,3 +617,28 @@ async def execute_with_keyboard_interrupt():
     assert result["error"] == "Command timed out after 1 seconds"
     assert result["status"] == -1
     assert "execution_time" in result
+
+
+@pytest.mark.asyncio
+async def test_output_redirection_with_append(executor, temp_test_dir, monkeypatch):
+    """Test output redirection with append mode"""
+    clear_env(monkeypatch)
+    monkeypatch.setenv("ALLOW_COMMANDS", "echo,cat")
+    output_file = os.path.join(temp_test_dir, "test.txt")
+
+    # Write initial content
+    await executor.execute(["echo", "hello", ">", output_file], directory=temp_test_dir)
+
+    # Append content
+    result = await executor.execute(
+        ["echo", "world", ">>", output_file], directory=temp_test_dir
+    )
+    assert result["error"] is None
+    assert result["status"] == 0
+
+    # Verify contents
+    result = await executor.execute(["cat", output_file], directory=temp_test_dir)
+    lines = result["stdout"].strip().split("\n")
+    assert len(lines) == 2
+    assert lines[0] == "hello"
+    assert lines[1] == "world"

tests/test_shell_executor_pipeline.py

@@ -1,4 +1,5 @@
 """Test pipeline execution and cleanup scenarios."""
+
 import os
 import tempfile
 
@@ -61,7 +62,9 @@ async def test_pipeline_execution_success(executor, temp_test_dir, monkeypatch):
     monkeypatch.setenv("ALLOWED_COMMANDS", "echo,grep")
 
     result = await executor.execute(
-        ["echo", "hello world", "|", "grep", "world"], directory=temp_test_dir, timeout=5
+        ["echo", "hello world", "|", "grep", "world"],
+        directory=temp_test_dir,
+        timeout=5,
     )
 
     assert result["error"] is None
@@ -84,16 +87,12 @@ async def test_pipeline_cleanup_and_timeouts(executor, temp_test_dir, monkeypatc
     result = await executor.execute(
         ["cat", test_file, "|", "tr", "[:lower:]", "[:upper:]", "|", "head", "-n", "1"],
         temp_test_dir,
-        timeout=2
+        timeout=2,
     )
     assert result["status"] == 0
     assert result["stdout"].strip() == "TEST"
 
     # Test timeout handling in pipeline
-    result = await executor.execute(
-        ["sleep", "5"],
-        temp_test_dir,
-        timeout=1
-    )
+    result = await executor.execute(["sleep", "5"], temp_test_dir, timeout=1)
     assert result["status"] == -1
     assert "timed out" in result["error"].lower()  # タイムアウトエラーの確認