From 29bcc724c4b9deea904e1ced1740f7abaf694bec Mon Sep 17 00:00:00 2001
From: seonghobae <8172694+seonghobae@users.noreply.github.com>
Date: Tue, 16 Jun 2026 05:12:31 +0000
Subject: [PATCH] =?UTF-8?q?=F0=9F=A7=AA=20Add=20test=20for=20error=20path?=
 =?UTF-8?q?=20in=20opencode=5Freview=5Fnormalize=5Foutput.py?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

🎯 What: Added unit test to verify that iter_json_objects handles json.JSONDecodeError correctly during character-by-character JSON decoding.
📊 Coverage: Added missing coverage for lines 89-93 inside the except block.
✨ Result: Ensured that parsing loop gracefully skips invalid JSON boundaries.
---
 .../test_opencode_review_normalize_output.py  | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)
 create mode 100644 tests/test_opencode_review_normalize_output.py

diff --git a/tests/test_opencode_review_normalize_output.py b/tests/test_opencode_review_normalize_output.py
new file mode 100644
index 0000000..6c4cc83
--- /dev/null
+++ b/tests/test_opencode_review_normalize_output.py
@@ -0,0 +1,19 @@
+import json
+from unittest.mock import patch
+
+from scripts.ci.opencode_review_normalize_output import iter_json_objects
+
+
+def test_iter_json_objects_decode_error():
+    """Test that iter_json_objects handles JSONDecodeError when decoding."""
+    text = "prefix { valid looking json } suffix"
+
+    # We mock raw_decode to raise JSONDecodeError to hit the except block explicitly
+    # This fulfills the 'Requires mocking the operation that throws the exception' rationale.
+    with patch("json.JSONDecoder.raw_decode") as mock_raw_decode:
+        mock_raw_decode.side_effect = json.JSONDecodeError("Mocked error", text, 0)
+
+        result = iter_json_objects(text)
+
+        assert result == []
+        assert mock_raw_decode.called