Bug 1204174 - Reduce test iterations to reduce intermittent timeouts. r=vp
authorJordan Santell <jsantell@mozilla.com>
Wed, 27 Jan 2016 08:54:00 +0100
changeset 320383 2e91c47d5ced479b43d9d46ee8f2a5f43adec60d
parent 320382 7581aa0f335f6787457f1fd3115ae413060cdd33
child 320384 72f6605f080992f8558a70ba5239b68f5b56bcaf
push id1128
push userjlund@mozilla.com
push dateWed, 01 Jun 2016 01:31:59 +0000
treeherdermozilla-release@fe0d30de989d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersvp
bugs1204174
milestone47.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1204174 - Reduce test iterations to reduce intermittent timeouts. r=vp
devtools/client/performance/test/browser_perf-overview-render-02.js
devtools/client/performance/test/browser_perf-overview-render-03.js
--- a/devtools/client/performance/test/browser_perf-overview-render-02.js
+++ b/devtools/client/performance/test/browser_perf-overview-render-02.js
@@ -1,15 +1,16 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
 
 /**
  * Tests that the overview graphs cannot be selected during recording
  * and that they're cleared upon rerecording.
  */
+const TIMES_TO_UPDATE = 2;
 function* spawnTest() {
   // This test seems to take a long time to cleanup on Ubuntu VMs.
   requestLongerTimeout(2);
 
   let { panel } = yield initPerformance(SIMPLE_URL);
   let { EVENTS, OverviewView } = panel.panelWin;
 
   // Enable memory to test all the overview graphs.
@@ -46,17 +47,17 @@ function* spawnTest() {
   is(memory.selectionEnabled, false,
     "The selection should not be enabled for the memory overview (2).");
   is(memory.hasSelection(), false,
     "The memory overview shouldn't have a selection before recording.");
 
   let updated = 0;
   OverviewView.on(EVENTS.OVERVIEW_RENDERED, () => updated++);
 
-  ok((yield waitUntil(() => updated > 10)),
+  ok((yield waitUntil(() => updated > TIMES_TO_UPDATE)),
     "The overviews were updated several times.");
 
   ok("selectionEnabled" in framerate,
     "The selection should still not be enabled for the framerate overview (1).");
   is(framerate.selectionEnabled, false,
     "The selection should still not be enabled for the framerate overview (2).");
   is(framerate.hasSelection(), false,
     "The framerate overview still shouldn't have a selection before recording.");
--- a/devtools/client/performance/test/browser_perf-overview-render-03.js
+++ b/devtools/client/performance/test/browser_perf-overview-render-03.js
@@ -1,14 +1,15 @@
 /* Any copyright is dedicated to the Public Domain.
    http://creativecommons.org/publicdomain/zero/1.0/ */
 
 /**
  * Tests that the overview graphs share the exact same width and scaling.
  */
+const TIMES_TO_UPDATE = 2;
 function* spawnTest() {
   // This test seems to take a long time to cleanup on Ubuntu VMs.
   requestLongerTimeout(2);
 
   let { panel } = yield initPerformance(SIMPLE_URL);
   let { EVENTS, PerformanceController, OverviewView } = panel.panelWin;
 
   // Enable memory to test all the overview graphs.
@@ -18,17 +19,17 @@ function* spawnTest() {
 
   let updated = 0;
   OverviewView.on(EVENTS.OVERVIEW_RENDERED, () => updated++);
 
   yield busyWait(100);
   yield waitUntil(() => PerformanceController.getCurrentRecording().getMarkers().length);
   yield waitUntil(() => PerformanceController.getCurrentRecording().getMemory().length);
   yield waitUntil(() => PerformanceController.getCurrentRecording().getTicks().length);
-  yield waitUntil(() => updated > 10);
+  yield waitUntil(() => updated > TIMES_TO_UPDATE);
 
   yield stopRecording(panel);
 
   let markers = OverviewView.graphs.get("timeline");
   let framerate = OverviewView.graphs.get("framerate");
   let memory = OverviewView.graphs.get("memory");
 
   ok(markers.width > 0,