Duplicate property names can cause test to fail
authorbjonesbe@adobe.com <bjonesbe@adobe.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 1 Aug 2013 17:39:43 +0000 (17:39 +0000)
committerbjonesbe@adobe.com <bjonesbe@adobe.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 1 Aug 2013 17:39:43 +0000 (17:39 +0000)
https://bugs.webkit.org/show_bug.cgi?id=119399

Reviewed by Dirk Pranke.

This problem was noticed when porting the patch for
http://wkb.ug/119357 to Blink: a test would fail because the converted
and test property lists weren't the same length, but they contained
all of the same property names. It looks like this is actually caused
by a bug in converting the HTML, which is being currently rewritten.
In the meantime, as long as this compares the number of unique
properties, it makes the test more robust.

* Scripts/webkitpy/w3c/test_converter_unittest.py:
(verify_prefixed_properties):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@153590 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/webkitpy/w3c/test_converter_unittest.py

index 1ad9f22..4c8b49f 100644 (file)
@@ -1,5 +1,23 @@
 2013-08-01  Bem Jones-Bey  <bjonesbe@adobe.com>
 
+        Duplicate property names can cause test to fail
+        https://bugs.webkit.org/show_bug.cgi?id=119399
+
+        Reviewed by Dirk Pranke.
+
+        This problem was noticed when porting the patch for
+        http://wkb.ug/119357 to Blink: a test would fail because the converted
+        and test property lists weren't the same length, but they contained
+        all of the same property names. It looks like this is actually caused
+        by a bug in converting the HTML, which is being currently rewritten.
+        In the meantime, as long as this compares the number of unique
+        properties, it makes the test more robust.
+
+        * Scripts/webkitpy/w3c/test_converter_unittest.py:
+        (verify_prefixed_properties):
+
+2013-08-01  Bem Jones-Bey  <bjonesbe@adobe.com>
+
         Unreviewed. Adding my expertise.
 
         * Scripts/webkitpy/common/config/contributors.json:
index 0620ad2..5e74c2a 100644 (file)
@@ -292,7 +292,7 @@ CONTENT OF TEST
         self.assertEquals(len(converted.findAll(href=relpath_pattern)), num_href_paths, 'testharness href relative path not correct')
 
     def verify_prefixed_properties(self, converted, test_properties):
-        self.assertEqual(len(converted[0]), len(test_properties), 'Incorrect number of properties converted')
+        self.assertEqual(len(set(converted[0])), len(set(test_properties)), 'Incorrect number of properties converted')
         for test_prop in test_properties:
             self.assertTrue((test_prop in converted[1]), 'Property ' + test_prop + ' not found in converted doc')