summaryrefslogtreecommitdiff
path: root/tests/test_error_codes.py
diff options
context:
space:
mode:
authortanaka takashi <isazaka@gmail.com>2026-07-11 19:57:02 +0900
committertanaka takashi <isazaka@gmail.com>2026-07-11 19:57:02 +0900
commite16782c208320138d77b88f2ff359e3e9e059e76 (patch)
tree5e19fde4fa841b9b3f2ddef9f5efdb342ba6e911 /tests/test_error_codes.py
parentc2e5098d030fe902eff258e10d2fefc01226e4bf (diff)
fix: レビュー指摘事項の修正(エラー戻り値統一、タイ延長、ネストリピート、専用エラーコード)
High severity: - compose action エラー戻り値の validation キー統一(server.py) - ppmck/pyxel タイ & の duration 延長修正(_tie_pending フラグ導入) - pyxel ネストリピート対応(RepeatFrame + イベントスナップショット方式) Medium severity: - 専用エラーコード追加: VALIDATION_MISSING_PARAMETER, VALIDATION_INVALID_MODE, VALIDATION_INVALID_ACTION - SYSTEM_INTERNAL_ERROR キャッチ範囲拡大(compose/validate全体をtry/except) Low severity: - 設計書 line/column を1始まりに明記、APIレベルエラーコード追記 テスト: 129 passed, ruff: All checks passed
Diffstat (limited to 'tests/test_error_codes.py')
-rw-r--r--tests/test_error_codes.py41
1 files changed, 39 insertions, 2 deletions
diff --git a/tests/test_error_codes.py b/tests/test_error_codes.py
index 19b4575..496ce84 100644
--- a/tests/test_error_codes.py
+++ b/tests/test_error_codes.py
@@ -17,14 +17,16 @@ def _find_code(result, code, key="errors"):
def _has_error(result):
- return any(e["severity"] == "error" for e in result["validation"]["errors"])
+ if "validation" in result:
+ return any(e["severity"] == "error" for e in result["validation"]["errors"])
+ return any(e["severity"] == "error" for e in result["errors"])
# --- SYNTAX_INVALID_TOKEN ---
def test_syntax_invalid_token_mode(compose_mml):
result = compose_mml(action="compose", mml="A c", mode="xyz")
assert result["success"] is False
- assert _find_code(result, ErrorCode.SYNTAX_INVALID_TOKEN.value)
+ assert _find_code(result, ErrorCode.VALIDATION_INVALID_MODE.value)
# --- SYNTAX_INVALID_NUMBER ---
@@ -201,6 +203,27 @@ def test_syntax_unterminated_header(compose_mml):
assert _find_code(result, ErrorCode.SYNTAX_UNTERMINATED_HEADER.value)
+# --- VALIDATION_MISSING_PARAMETER ---
+def test_validation_missing_parameter(compose_mml):
+ result = compose_mml(action="compose", mml="", mode="ppmck")
+ assert result["success"] is False
+ assert _find_code(result, ErrorCode.VALIDATION_MISSING_PARAMETER.value)
+
+
+# --- VALIDATION_INVALID_MODE ---
+def test_validation_invalid_mode(compose_mml):
+ result = compose_mml(action="compose", mml="A c", mode="xyz")
+ assert result["success"] is False
+ assert _find_code(result, ErrorCode.VALIDATION_INVALID_MODE.value)
+
+
+# --- VALIDATION_INVALID_ACTION ---
+def test_validation_invalid_action(compose_mml):
+ result = compose_mml(action="unknown")
+ assert result["success"] is False
+ assert _find_code(result, ErrorCode.VALIDATION_INVALID_ACTION.value)
+
+
# --- SYSTEM_SYNTHESIS_FAILED ---
def test_system_synthesis_failed(compose_mml):
with patch(
@@ -234,6 +257,20 @@ def test_system_wav_write_failed(tmp_path):
# --- SYSTEM_INTERNAL_ERROR ---
+def test_system_internal_error(compose_mml):
+ with patch(
+ "mml_composemusic_mcp.server._parse_mml",
+ side_effect=RuntimeError("unexpected internal error"),
+ ):
+ result = compose_mml(
+ action="compose",
+ mml="A t120 l4 o4\n c",
+ mode="ppmck",
+ )
+ assert result["success"] is False
+ assert _find_code(result, ErrorCode.SYSTEM_INTERNAL_ERROR.value)
+
+
def test_system_internal_error_not_exposed(compose_mml):
result = compose_mml(action="compose", mml="A t120 l4 o4\n c", mode="ppmck")
assert isinstance(result, dict)