Changeset 176

Timestamp:

05/23/06 15:11:21 (2 years ago)

Author:

xi

Message:

Fixed #16 (duplicate keys in dictionary cause unpredictable behavior).
Thanks to nickesk(at)cs.bu.edu for the report.

If a mapping node contains duplicate keys, it is transformed to a list of pairs.
This behaviour may change in the future since it contradicts the YAML specification.

Files:

• pysyck/trunk/lib/syck/loaders.py (modified) (1 diff)
• pysyck/trunk/tests/test_loader.py (modified) (2 diffs)

pysyck/trunk/lib/syck/loaders.py

@@ -80 +80 @@
-                    value_object = self._convert(node.value[key_node],
-                            node_to_object)
-
+    
-        node.value = value
-        object = self.construct(node)

pysyck/trunk/tests/test_loader.py

@@ -143 +143 @@
-"""
-
+DUPLICATE_KEY = """
+0: 0
+0: 1
+"""
+
-class TestDocuments(test_parser.TestDocuments):
-
@@ -289 +294 @@
-        self.assertEqual(len(document[0]), 2)
-        self.assertEqual(document[0][0], document[0][1])
+
+    def testDuplicateKey(self):
+        document = syck.load(DUPLICATE_KEY)
+        self.assertEqual(type(document), list)
+        self.assertEqual(len(document), 2)
+        self.assertEqual(len(document[0]), 2)
+        self.assertEqual(len(document[1]), 2)
+        self.assertEqual(document[0][0], document[1][0])
+