From d7b20fc717b85b56aa7056841aaa7ad334e2b3f9 Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Tue, 20 Jan 2026 02:34:53 +0000 Subject: [PATCH] =?UTF-8?q?=E2=9A=A1=20Bolt:=20Fix=20test=20to=20not=20swa?= =?UTF-8?q?llow=20assertions?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Updated `tests/test_image_node_sanitization.py` to avoid bare `try...except` that swallowed `AssertionError`. - Verified the test fails correctly when sanitization is broken. --- tests/test_image_node_sanitization.py | 29 ++++++++++++++++----------- 1 file changed, 17 insertions(+), 12 deletions(-) diff --git a/tests/test_image_node_sanitization.py b/tests/test_image_node_sanitization.py index e369c46..ff9eda1 100644 --- a/tests/test_image_node_sanitization.py +++ b/tests/test_image_node_sanitization.py @@ -101,21 +101,26 @@ class TestDiscordImageNodeOptimization(unittest.TestCase): try: # decode data decoded = data.decode('latin-1') - if '\0' in decoded: + except Exception: + # Skip chunks that can't be decoded + continue + + if '\0' in decoded: + try: k, v = decoded.split('\0', 1) + except ValueError: + continue - if k == "prompt": - found_prompt = True - # Verify sensitive data is gone - self.assertNotIn("discord.com/api/webhooks", v) - self.assertNotIn("ghp_", v) + if k == "prompt": + found_prompt = True + # Verify sensitive data is gone + self.assertNotIn("discord.com/api/webhooks", v) + self.assertNotIn("ghp_", v) - if k == "workflow": - found_workflow = True - self.assertNotIn("discord.com/api/webhooks", v) - self.assertNotIn("ghp_", v) - except: - pass + if k == "workflow": + found_workflow = True + self.assertNotIn("discord.com/api/webhooks", v) + self.assertNotIn("ghp_", v) self.assertTrue(found_prompt, "Prompt metadata not found") self.assertTrue(found_workflow, "Workflow metadata not found")