Fix #47, a stupid bug was triggered because of missing tests during t…

…he last refactor. Fixed the bug and updated tests

pyproject.toml

@@ -3,7 +3,7 @@ requires = ["setuptools>=61.0"]
 build-backend = "setuptools.build_meta"
 [project]
 name = "json_repair"
-version = "0.20.0"
+version = "0.20.1"
 license = {file = "LICENSE"}
 authors = [
   { name="Stefano Baccianella", email="4247706+mangiucugna@users.noreply.github.com" },

src/json_repair/json_repair.py

@@ -131,10 +131,14 @@ def parse_object(self) -> Dict[str, Any]:
                     # Sometimes the string search might not move the index at all, that might lead us to an infinite loop
                     self.index += 1
 
+            self.skip_whitespaces_at()
+
             # We reached the end here
             if (self.get_char_at() or "}") == "}":
                 continue
 
+            self.skip_whitespaces_at()
+
             # An extreme case of missing ":" after a key
             if (self.get_char_at() or "") != ":":
                 self.log(
@@ -178,7 +182,7 @@ def parse_array(self) -> List[Any]:
             value = self.parse_json()
 
             # It is possible that parse_json() returns nothing valid, so we stop
-            if not value:
+            if value == "":
                 break
 
             if value == "..." and self.get_char_at(-1) == ".":

tests/test_json_repair.py

@@ -32,7 +32,7 @@ def test_valid_json():
     assert repair_json('{"key": "value\\nvalue"}') == '{"key": "value\\nvalue"}'
 
 def test_brackets_edge_cases():
-    assert repair_json("[{]") == "[]"
+    assert repair_json("[{]") == "[{}]"
     assert repair_json("   {  }   ") == "{}"
     assert repair_json("[") == "[]"
     assert repair_json("]") == '""'
@@ -83,6 +83,7 @@ def test_array_edge_cases():
     assert repair_json("[1, 2, 3, ...]") == "[1, 2, 3]"
     assert repair_json("[1, 2, ... , 3]") == "[1, 2, 3]"
     assert repair_json("[1, 2, '...', 3]") == '[1, 2, "...", 3]'
+    assert repair_json("[true, false, null, ...]") == '[true, false, null]'
     assert (
         repair_json('{"employees":["John", "Anna",')
         == '{"employees": ["John", "Anna"]}'
@@ -119,7 +120,7 @@ def test_object_edge_cases():
     assert repair_json('{""answer"":[{""traits"":''Female aged 60+'',""answer1"":""5""}]}') == '{"answer": [{"traits": "Female aged 60+", "answer1": "5"}]}'
     assert repair_json('{ "words": abcdef", "numbers": 12345", "words2": ghijkl" }') == '{"words": "abcdef", "numbers": 12345, "words2": "ghijkl"}'
     assert repair_json('''{"number": 1,"reason": "According...""ans": "YES"}''') == '{"number": 1, "reason": "According...", "ans": "YES"}'
-    assert repair_json('''{ "a": "{ b": {} }" }''') == '{"a": "{ b"}'
+    assert repair_json('''{ "a" : "{ b": {} }" }''') == '{"a": "{ b"}'
     assert repair_json("""{"b": "xxxxx" true}""") == '{"b": "xxxxx"}'
     assert repair_json('{"key": "Lorem "ipsum" s,"}') == '{"key": "Lorem \\"ipsum\\" s,"}'