Remove some useless test-level properties

Remove useless test-level properties, including: * Incorrect type of the properties parameter for `promise_test()` * `flags` property passed to `test()`, which is unused in testharness.js * Incorrect type of the properties `generate_tests`: moved to comments * `help` property: moved the head of the test files * Default empty properties dictionary Related: #14394
web-platform-tests · Aug 28, 2020 · a04e29b · a04e29b
1 parent bc2b0f3
commit a04e29b
Show file tree

Hide file tree

Showing 15 changed files with 168 additions and 283 deletions.
diff --git a/css/css-shapes/shape-outside/values/shape-margin-000.html b/css/css-shapes/shape-outside/values/shape-margin-000.html
@@ -27,9 +27,9 @@
             }
         ];
         generate_tests( ParsingUtils.testShapeMarginInlineStyle,
-                        ParsingUtils.buildTestCases(shape_margin_valid_tests, "inline"), false);
+                        ParsingUtils.buildTestCases(shape_margin_valid_tests, "inline"));
         generate_tests( ParsingUtils.testShapeMarginComputedStyle,
-                        ParsingUtils.buildTestCases(shape_margin_valid_tests, "computed"), false);
+                        ParsingUtils.buildTestCases(shape_margin_valid_tests, "computed"));
         </script>
     </body>
 </html>
diff --git a/css/css-shapes/shape-outside/values/shape-margin-001.html b/css/css-shapes/shape-outside/values/shape-margin-001.html
@@ -29,11 +29,11 @@
         });
 
         generate_tests( ParsingUtils.testShapeMarginInlineStyle,
-                        ParsingUtils.buildTestCases(shape_margin_valid_unit_tests, "inline"), true);
+                        ParsingUtils.buildTestCases(shape_margin_valid_unit_tests, "inline"));
         ParsingUtils.setupFonts();
         document.fonts.ready.then(()=> {
             generate_tests( ParsingUtils.testShapeMarginComputedStyle,
-                            ParsingUtils.buildTestCases(shape_margin_valid_unit_tests, "computed"), true);
+                            ParsingUtils.buildTestCases(shape_margin_valid_unit_tests, "computed"));
             ParsingUtils.restoreFonts();
             done();
         });

diff --git a/css/css-shapes/shape-outside/values/shape-margin-002.html b/css/css-shapes/shape-outside/values/shape-margin-002.html
@@ -37,9 +37,9 @@
             },
         ];
         generate_tests( ParsingUtils.testShapeMarginInlineStyle,
-                        ParsingUtils.buildTestCases(shape_margin_valid_tests, "inline"), true);
+                        ParsingUtils.buildTestCases(shape_margin_valid_tests, "inline"));
         generate_tests( ParsingUtils.testShapeMarginComputedStyle,
-                        ParsingUtils.buildTestCases(shape_margin_valid_tests, "computed"), true);
+                        ParsingUtils.buildTestCases(shape_margin_valid_tests, "computed"));
         </script>
     </body>
 </html>
diff --git a/css/css-transitions/transition-delay-001.html b/css/css-transitions/transition-delay-001.html
@@ -69,11 +69,7 @@
                         });
                         var result = computedStyle(transition, 'transition-delay');
                         assert_equals(result, values[key], "Expected computed value");
-                    }, "parse '" + key + "'",
-                    {
-                        // mark tests that fail as such
-                        flags: invalidTests[key] ? "invalid" : ""
-                    });
+                    }, "parse '" + key + "'");
                 }
             }
         </script>

diff --git a/css/css-transitions/transition-duration-001.html b/css/css-transitions/transition-duration-001.html
@@ -70,11 +70,7 @@
                         });
                         var result = computedStyle(transition, 'transition-duration');
                         assert_equals(result, values[key], "Expected computed value");
-                    }, "parse '" + key + "'",
-                    {
-                        // mark tests that fail as such
-                        flags: invalidTests[key] ? "invalid" : ""
-                    });
+                    }, "parse '" + key + "'");
                 }
             }
         </script>