review fix

Author: foteinigk

openformats/tests/formats/structuredkeyvaluejson/test_keyvaluejson.py

@@ -839,21 +839,46 @@ def test_non_empty_values_with_non_existing_template_value(self):
         compiled = self.handler.compile(template, expected)
         self.assertEqual(compiled, expected_compilation)
 
+    def test_sync_template_all_structured_deleted_dict_root(self):
+        # Dict root with only structured entries
+        s1 = generate_random_string()
+        s2 = generate_random_string()
+        source = '{"a": {"string": "%s"}, "b": {"string": "%s"}}' % (s1, s2)
+
+        template, _ = self.handler.parse(source)
+
+        # Empty stringset -> all structured entries should be removed
+        updated = self.handler.sync_template(template, [])
+
+        data = json.loads(updated)
+        # Dict root should become an empty object
+        self.assertEqual(data, {})
+
+    def test_sync_template_all_structured_deleted_list_root(self):
+        s1 = generate_random_string()
+        s2 = generate_random_string()
+        source = '[{"a": {"string": "%s"}, "b": {"string": "%s"}}]' % (s1, s2)
+
+        template, _ = self.handler.parse(source)
+
+        updated = self.handler.sync_template(template, [])
+
+        data = json.loads(updated)
+        self.assertEqual(data, [])
+
+
     def test_sync_template_keeps_all_when_strings_match(self):
-        # Template with two strings
         s1 = generate_random_string()
         s2 = generate_random_string()
         source = '{"a": {"string": "%s"}, "b": {"string": "%s"}}' % (s1, s2)
 
         template, stringset = self.handler.parse(source)
 
-        # Sanity: template now has hashes
         hash1 = stringset[0].template_replacement
         hash2 = stringset[1].template_replacement
         self.assertIn(hash1, template)
         self.assertIn(hash2, template)
 
-        # If we pass back the same stringset, sync_template should be a no-op
         updated = self.handler.sync_template(template, stringset)
 
         self.assertEqual(updated, template)