fix: update test check_test_cases to reflect new implementation

Author: wislertt

tests/tools/test_check_test_cases.py

@@ -9,27 +9,45 @@ class TestCountTestCasesForProblem:
         "json_data, expected",
         [
             (
-                {"test_methods": [{"test_cases": "[(1, 2, 3), (4, 5, 6)]"}, {"test_cases": "[(7, 8)]"}]},
+                {
+                    "test_methods": [
+                        {"test_cases": {"list": ["(1, 2, 3)", "(4, 5, 6)"]}},
+                        {"test_cases": {"list": ["(7, 8)"]}},
+                    ]
+                },
                 3,
             ),
             (
                 {
                     "_test_methods": {
                         "list": [
-                            {"test_cases": "[(1, 2), (3, 4), (5, 6)]"},
-                            {"test_cases": "[(7, 8, 9)]"},
+                            {"test_cases": {"list": ["(1, 2)", "(3, 4)", "(5, 6)"]}},
+                            {"test_cases": {"list": ["(7, 8, 9)"]}},
                         ]
                     }
                 },
                 4,
             ),
-            ({"test_methods": [{"test_cases": "[]"}, {"test_cases": ""}, {"test_cases": "   "}]}, 0),
+            (
+                {
+                    "test_methods": [
+                        {"test_cases": {"list": []}},
+                        {"test_cases": ""},
+                        {"test_cases": "   "},
+                    ]
+                },
+                0,
+            ),
             ({}, 0),
             (
                 {
                     "test_methods": [
-                        {"test_cases": "[([1, 2], 'hello', True), ([3, 4], 'world', False)]"},
-                        {"test_cases": "[([], '', None)]"},
+                        {
+                            "test_cases": {
+                                "list": ["([1, 2], 'hello', True)", "([3, 4], 'world', False)"]
+                            }
+                        },
+                        {"test_cases": {"list": ["([], '', None)"]}},
                     ]
                 },
                 3,
@@ -39,16 +57,21 @@ class TestCountTestCasesForProblem:
     def test_count_test_cases(self, json_data, expected):
         assert count_test_cases_for_problem(json_data) == expected
 
-    def test_invalid_test_cases_raises_error(self):
+    def test_invalid_test_cases_returns_zero(self):
+        """Test that invalid test cases are ignored and return 0."""
         json_data = {"test_methods": [{"test_cases": "invalid python literal"}]}
-        with pytest.raises((ValueError, SyntaxError)):
-            count_test_cases_for_problem(json_data)
+        # Current implementation ignores invalid test cases and returns 0
+        assert count_test_cases_for_problem(json_data) == 0
 
     def test_python_expressions_in_test_cases(self):
         """Test that Python expressions like 'string' * 100 are handled correctly."""
         json_data = {
             "test_methods": [
-                {"test_cases": "[('input', 'expected'), ('100[leetcode]', 'leetcode' * 100)]"}
+                {
+                    "test_cases": {
+                        "list": ["('input', 'expected')", "('100[leetcode]', 'leetcode' * 100)"]
+                    }
+                }
             ]
         }
         # Should not raise an error and should count 2 test cases