Bug 1337674 followup 3 - Fix assertion check.
authorXidorn Quan <me@upsuper.org>
Sat, 18 Feb 2017 12:17:24 +1100
changeset 372707 53cc100473f9bf2e284b489ab83bacdac71dcd77
parent 372706 48cf7b975f179a52bf86fa4bc6bea16d771528bb
child 372708 ec071aa8a8e899a72cefe64a1fe432c839a1d392
push id10863
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 23:02:23 +0000
treeherdermozilla-aurora@0931190cd725 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1337674
milestone54.0a1
Bug 1337674 followup 3 - Fix assertion check. MozReview-Commit-ID: EckSAiEeou5
layout/style/test/stylo-failures.md
testing/mochitest/tests/SimpleTest/TestRunner.js
--- a/layout/style/test/stylo-failures.md
+++ b/layout/style/test/stylo-failures.md
@@ -569,24 +569,16 @@ Any line which doesn't follow the format
 * getComputedStyle style doesn't contain custom properties bug 1336891
   * test_variable_serialization_computed.html [36]
   * test_variables.html `custom property name` [2]
 * test_css_supports.html: issues around @supports syntax servo/servo#15482 [8]
 * test_load_events_on_stylesheets.html: stylesheets from data uri are not sync loaded [3]
 
 ## Assertions
 
-* Table wrapper with borders and paddings bug 1324661
-  * test_animations_async_tests.html asserts [3]
-  * test_animations_effect_timing_enddelay.html asserts [2]
-  * test_animations_event_order.html asserts [1]
-  * test_animations_omta_start.html asserts [1]
-  * test_animations_playbackrate.html asserts [1]
-  * test_at_rule_parse_serialize.html asserts [1]
-  * test_unclosed_parentheses.html asserts [1]
 * Content glue not implemented
   * assertion in frame constructor bug 1324704
     * test_rule_insertion.html asserts [1]
   * assertion in computed style bug 1337635
     * test_value_cloning.html asserts [18]
     * test_value_computation.html asserts [18]
     * test_value_storage.html asserts [32]
 * test_value_cloning.html asserts: negative radius bug 1337618 [6]
--- a/testing/mochitest/tests/SimpleTest/TestRunner.js
+++ b/testing/mochitest/tests/SimpleTest/TestRunner.js
@@ -615,17 +615,17 @@ TestRunner.testUnloaded = function() {
 
         var url = TestRunner.getNextUrl();
         var max = TestRunner._expectedMaxAsserts;
         var min = TestRunner._expectedMinAsserts;
         if (Array.isArray(TestRunner.expected)) {
             // Accumulate all assertion counts recorded in the failure pattern file.
             let additionalAsserts = TestRunner.expected.reduce((acc, [pat, count]) => {
                 return pat == "ASSERTION" ? acc + count : acc;
-            });
+            }, 0);
             min += additionalAsserts;
             max += additionalAsserts;
         }
         if (numAsserts > max) {
             TestRunner.structuredLogger.testEnd(url,
                                                 "ERROR",
                                                 "OK",
                                                 "Assertion count " + numAsserts + " is greater than expected range " +