From 65478e0b17ddfbaaca45403c32d58e52215f7833 Mon Sep 17 00:00:00 2001 From: Niels Geens Date: Wed, 13 Nov 2024 14:33:56 +0100 Subject: [PATCH] Fix issues in response_parser_test --- test/response_parser_test.py | 19 +++++-------------- 1 file changed, 5 insertions(+), 14 deletions(-) diff --git a/test/response_parser_test.py b/test/response_parser_test.py index d5125d7..75fe9a4 100644 --- a/test/response_parser_test.py +++ b/test/response_parser_test.py @@ -61,7 +61,7 @@ class ResponseParserTest(unittest.TestCase): self.assertIsInstance(result, ParseErrorEntry) self.assertEqual(result.content, xml) - self.assertIn("missing script content", result.error) + self.assertIn("Background entry requires (only) script content", result.error) def test_repeat_entry(self): """Test parsing repeat entry""" @@ -78,7 +78,7 @@ class ResponseParserTest(unittest.TestCase): self.assertIsInstance(result, ParseErrorEntry) self.assertEqual(result.content, xml) - self.assertIn("missing script content", result.error) + self.assertIn("Repeat entry requires (only) script content", result.error) def test_single_entry(self): """Test parsing single shot entry""" @@ -95,7 +95,7 @@ class ResponseParserTest(unittest.TestCase): self.assertIsInstance(result, ParseErrorEntry) self.assertEqual(result.content, xml) - self.assertIn("missing script content", result.error) + self.assertIn("Single entry requires (only) script content", result.error) def test_reasoning_entry(self): """Test parsing reasoning entry""" @@ -112,7 +112,7 @@ class ResponseParserTest(unittest.TestCase): self.assertIsInstance(result, ParseErrorEntry) self.assertEqual(result.content, xml) - self.assertIn("missing content", result.error) + self.assertIn("Reasoning entry requires (only) text content", result.error) def test_read_stdin_entry(self): """Test parsing read stdin entry""" @@ -137,7 +137,7 @@ class ResponseParserTest(unittest.TestCase): self.assertIsInstance(result, ParseErrorEntry) self.assertEqual(result.content, xml) - self.assertIn("missing content", result.error) + self.assertIn("Write stdout entry requires (only) text content", result.error) def test_unknown_element(self): """Test parsing unknown element returns error entry""" @@ -180,15 +180,6 @@ class ResponseParserTest(unittest.TestCase): self.assertIsInstance(result, ReasoningEntry) self.assertTrue(result.content.strip()) - def test_script_with_special_characters(self): - """Test parsing script with special characters""" - script = 'echo "Special chars: \n\t\r\'\"\\"' - xml = f'{script}' - - result = self.parser.parse(xml) - self.assertIsInstance(result, SingleEntry) - self.assertEqual(result.script, script) - def test_cdata_content(self): """Test parsing content with CDATA sections""" xml = """