Bug 1390006 - fix up expectations of BrowserUITelemetry tests to match what devedition is doing, r?johannh draft
authorGijs Kruitbosch <gijskruitbosch@gmail.com>
Mon, 14 Aug 2017 12:17:34 +0100
changeset 645859 43f12b7f670fdf502ee7a44a7ac24109535a754a
parent 645833 df9beb781895fcd0493c21e95ad313e0044515ec
child 726053 cdc253b7cd9aec3c15b1bea8f5d95127e1c9d43c
push id73914
push userbmo:gijskruitbosch+bugs@gmail.com
push dateMon, 14 Aug 2017 11:20:08 +0000
reviewersjohannh
bugs1390006
milestone57.0a1
Bug 1390006 - fix up expectations of BrowserUITelemetry tests to match what devedition is doing, r?johannh MozReview-Commit-ID: I03npiUHECK
browser/modules/test/browser/browser_BrowserUITelemetry_defaults.js
--- a/browser/modules/test/browser/browser_BrowserUITelemetry_defaults.js
+++ b/browser/modules/test/browser/browser_BrowserUITelemetry_defaults.js
@@ -13,20 +13,21 @@ function test() {
   // Bug 1278176 - DevEdition never has the UI in a default state by default.
   if (!AppConstants.MOZ_DEV_EDITION) {
     Assert.ok(CustomizableUI.inDefaultState,
               "No other test should have left CUI in a dirty state.");
   }
 
   let result = BrowserUITelemetry._getWindowMeasurements(window, 0);
 
-  // Bug 1278176 - DevEdition always reports the developer-button is moved.
+  Assert.deepEqual(result.defaultMoved, []);
+  // Bug 1278176 - DevEdition always reports the developer-button is added to
+  // the toolbar.
   if (!AppConstants.MOZ_DEV_EDITION) {
-    Assert.deepEqual(result.defaultMoved, []);
+    Assert.deepEqual(result.nondefaultAdded, []);
   }
-  Assert.deepEqual(result.nondefaultAdded, []);
 
   Assert.deepEqual(result.defaultRemoved, []);
 
   // And mochi insists there's only a single window with a single tab when
   // starting a test, so check that for good measure.
   Assert.deepEqual(result.visibleTabs, [1]);
 }