Bug 1383980 - Enable animation mochitests in mochitest-style task. r?birtles draft
authorXidorn Quan <me@upsuper.org>
Tue, 25 Jul 2017 14:17:04 +1000
changeset 614910 9630ad723f3aa1a7d85f2e83983fd249f3b9940e
parent 614766 cd4897b32c1e5723c347dec5aa44133ca36c4dbb
child 639005 c4d3afe64f8cfeee2d4dd99c8dd9cf5129066b5c
push id70167
push userxquan@mozilla.com
push dateTue, 25 Jul 2017 05:57:22 +0000
reviewersbirtles
bugs1383980
milestone56.0a1
Bug 1383980 - Enable animation mochitests in mochitest-style task. r?birtles MozReview-Commit-ID: 38Ir2MG7IMO
dom/animation/test/chrome.ini
dom/animation/test/mochitest.ini
dom/smil/test/mochitest.ini
testing/mochitest/runtests.py
testing/mozharness/configs/unittests/linux_unittest.py
--- a/dom/animation/test/chrome.ini
+++ b/dom/animation/test/chrome.ini
@@ -4,18 +4,19 @@ support-files =
   ../../imptests/testharness.js
   ../../imptests/testharnessreport.js
   !/dom/animation/test/chrome/file_animate_xrays.html
 
 [chrome/test_animate_xrays.html]
 # file_animate_xrays.html needs to go in mochitest.ini since it is served
 # over HTTP
 [chrome/test_animation_observers_async.html]
+skip-if = stylo
 [chrome/test_animation_observers_sync.html]
 [chrome/test_animation_performance_warning.html]
 [chrome/test_animation_properties.html]
 [chrome/test_cssanimation_missing_keyframes.html]
 [chrome/test_generated_content_getAnimations.html]
 [chrome/test_restyles.html]
-skip-if = os == 'android' # bug 1335986
+skip-if = os == 'android' || stylo # bug 1335986
 [chrome/test_running_on_compositor.html]
 [chrome/test_simulate_compute_values_failure.html]
 skip-if = !debug
--- a/dom/animation/test/mochitest.ini
+++ b/dom/animation/test/mochitest.ini
@@ -76,48 +76,53 @@ support-files =
 [css-animations/test_animation-starttime.html]
 [css-animations/test_cssanimation-animationname.html]
 [css-animations/test_document-get-animations.html]
 [css-animations/test_effect-target.html]
 [css-animations/test_element-get-animations.html]
 [css-animations/test_event-dispatch.html]
 [css-animations/test_event-order.html]
 [css-animations/test_keyframeeffect-getkeyframes.html]
+skip-if = stylo
 [css-animations/test_pseudoElement-get-animations.html]
 [css-animations/test_setting-effect.html]
 [css-transitions/test_animation-cancel.html]
 [css-transitions/test_animation-computed-timing.html]
 [css-transitions/test_animation-currenttime.html]
 [css-transitions/test_animation-finished.html]
 [css-transitions/test_animation-pausing.html]
 [css-transitions/test_animation-ready.html]
 [css-transitions/test_animation-starttime.html]
 [css-transitions/test_csstransition-transitionproperty.html]
 [css-transitions/test_document-get-animations.html]
 [css-transitions/test_effect-target.html]
 [css-transitions/test_element-get-animations.html]
+skip-if = stylo
 [css-transitions/test_event-dispatch.html]
 [css-transitions/test_keyframeeffect-getkeyframes.html]
 [css-transitions/test_pseudoElement-get-animations.html]
 [css-transitions/test_setting-effect.html]
 [document-timeline/test_document-timeline.html]
 [document-timeline/test_request_animation_frame.html]
 [mozilla/test_cubic_bezier_limits.html]
 [mozilla/test_deferred_start.html]
 [mozilla/test_disable_animations_api_core.html]
 [mozilla/test_disabled_properties.html]
 [mozilla/test_discrete-animations.html]
+skip-if = stylo
 [mozilla/test_document-timeline-origin-time-range.html]
 [mozilla/test_hide_and_show.html]
 [mozilla/test_moz-prefixed-properties.html]
 [mozilla/test_restyling_xhr_doc.html]
 [mozilla/test_set-easing.html]
 [mozilla/test_transform_limits.html]
+skip-if = stylo
 [mozilla/test_transition_finish_on_compositor.html]
 skip-if = toolkit == 'android'
 [mozilla/test_underlying-discrete-value.html]
+skip-if = stylo
 [style/test_animation-seeking-with-current-time.html]
 [style/test_animation-seeking-with-start-time.html]
 [style/test_animation-setting-effect.html]
 [style/test_composite.html]
 [style/test_interpolation-from-interpolatematrix-to-none.html]
 [style/test_missing-keyframe.html]
 [style/test_missing-keyframe-on-compositor.html]
--- a/dom/smil/test/mochitest.ini
+++ b/dom/smil/test/mochitest.ini
@@ -14,50 +14,59 @@ support-files =
 
 [test_smilAccessKey.xhtml]
 [test_smilAnimateMotion.xhtml]
 [test_smilAnimateMotionInvalidValues.xhtml]
 [test_smilAnimateMotionOverrideRules.xhtml]
 [test_smilBackwardsSeeking.xhtml]
 [test_smilCSSFontStretchRelative.xhtml]
 [test_smilCSSFromBy.xhtml]
+skip-if = stylo
 [test_smilCSSFromTo.xhtml]
+skip-if = stylo
 # [test_smilCSSInherit.xhtml]
 # disabled until bug 501183 is fixed
 [test_smilCSSInvalidValues.xhtml]
 [test_smilCSSPaced.xhtml]
+skip-if = stylo
 [test_smilChangeAfterFrozen.xhtml]
+skip-if = stylo
 [test_smilConditionalProcessing.html]
 [test_smilContainerBinding.xhtml]
 [test_smilCrossContainer.xhtml]
 [test_smilDynamicDelayedBeginElement.xhtml]
 [test_smilExtDoc.xhtml]
 skip-if = toolkit == 'android'
 [test_smilFillMode.xhtml]
 [test_smilGetSimpleDuration.xhtml]
 [test_smilGetStartTime.xhtml]
 [test_smilHyperlinking.xhtml]
 [test_smilInvalidValues.html]
 [test_smilKeySplines.xhtml]
 [test_smilKeyTimes.xhtml]
+skip-if = stylo
 [test_smilKeyTimesPacedMode.xhtml]
 [test_smilMappedAttrFromBy.xhtml]
+skip-if = stylo
 [test_smilMappedAttrFromTo.xhtml]
+skip-if = stylo
 [test_smilMappedAttrPaced.xhtml]
+skip-if = stylo
 [test_smilMinTiming.html]
 [test_smilRepeatDuration.html]
 [test_smilRepeatTiming.xhtml]
 skip-if = toolkit == 'android' #TIMED_OUT
 [test_smilReset.xhtml]
 [test_smilRestart.xhtml]
 [test_smilSetCurrentTime.xhtml]
 [test_smilSync.xhtml]
 [test_smilSyncTransform.xhtml]
 [test_smilSyncbaseTarget.xhtml]
 [test_smilTextZoom.xhtml]
+skip-if = stylo
 [test_smilTimeEvents.xhtml]
 [test_smilTiming.xhtml]
 [test_smilTimingZeroIntervals.xhtml]
 [test_smilUpdatedInterval.xhtml]
 [test_smilValues.xhtml]
 [test_smilWithTransition.html]
 [test_smilWithXlink.xhtml]
 [test_smilXHR.xhtml]
--- a/testing/mochitest/runtests.py
+++ b/testing/mochitest/runtests.py
@@ -1336,18 +1336,18 @@ toolbar#nav-bar {
                     test['path'],
                     'SKIP',
                     message=test['disabled'])
 
     def loadFailurePatternFile(self, pat_file):
         if pat_file in self.patternFiles:
             return self.patternFiles[pat_file]
         if not os.path.isfile(pat_file):
-            self.log.error("TEST-UNEXPECTED-ERROR | runtests.py | "
-                           "Cannot find failure pattern file " + pat_file)
+            self.log.warning("runtests.py | Cannot find failure pattern file " +
+                             pat_file)
             return None
 
         # Using ":error" to ensure it shows up in the failure summary.
         self.log.warning(
             "[runtests.py:error] Using {} to filter failures. If there "
             "is any number mismatch below, you could have fixed "
             "something documented in that file. Please reduce the "
             "failure count appropriately.".format(pat_file))
--- a/testing/mozharness/configs/unittests/linux_unittest.py
+++ b/testing/mozharness/configs/unittests/linux_unittest.py
@@ -200,18 +200,18 @@ config = {
         "mochitest-gl": ["--subsuite=webgl"],
         "mochitest-devtools-chrome": ["--flavor=browser", "--subsuite=devtools"],
         "mochitest-devtools-chrome-chunked": ["--flavor=browser", "--subsuite=devtools", "--chunk-by-runtime"],
         "mochitest-devtools-chrome-coverage": ["--flavor=browser", "--subsuite=devtools", "--chunk-by-runtime", "--timeout=1200"],
         "jetpack-package": ["--flavor=jetpack-package"],
         "jetpack-package-clipboard": ["--flavor=jetpack-package", "--subsuite=clipboard"],
         "jetpack-addon": ["--flavor=jetpack-addon"],
         "a11y": ["--flavor=a11y"],
-        "plain-style": ["--failure-pattern-file=stylo-failures.md", "layout/style/test"],
-        "chrome-style": ["--flavor=chrome", "--failure-pattern-file=../stylo-failures.md", "layout/style/test/chrome"],
+        "plain-style": ["--failure-pattern-file=stylo-failures.md", "layout/style/test", "dom/smil/test", "dom/animation/test"],
+        "chrome-style": ["--flavor=chrome", "--failure-pattern-file=../stylo-failures.md", "layout/style/test/chrome", "dom/animation/test"],
     },
     # local reftest suites
     "all_reftest_suites": {
         "crashtest": {
             "options": ["--suite=crashtest"],
             "tests": ["tests/reftest/tests/testing/crashtest/crashtests.list"]
         },
         "jsreftest": {