Bug 1403037 - Don't test attribution data in applications that don't use it. r?Dexter draft
authorMatt Howell <mhowell@mozilla.com>
Mon, 25 Sep 2017 19:39:58 -0700
changeset 670217 5e025dee212da64455c1183f2af2144aaa708510
parent 670182 e6b3498a39b94616ba36798fe0b71a3090b1b14c
child 733171 9ef04ed63ccd3775af78c1619604ec4b9a21edbc
push id81561
push userbmo:mhowell@mozilla.com
push dateTue, 26 Sep 2017 02:41:39 +0000
reviewersDexter
bugs1403037
milestone58.0a1
Bug 1403037 - Don't test attribution data in applications that don't use it. r?Dexter MozReview-Commit-ID: Giy2ZRlMVa2
toolkit/components/telemetry/tests/unit/test_TelemetryEnvironment.js
--- a/toolkit/components/telemetry/tests/unit/test_TelemetryEnvironment.js
+++ b/toolkit/components/telemetry/tests/unit/test_TelemetryEnvironment.js
@@ -445,17 +445,17 @@ function checkSettingsSection(data) {
   Assert.equal(typeof update.autoDownload, "boolean");
 
   // Check "defaultSearchEngine" separately, as it can either be undefined or string.
   if ("defaultSearchEngine" in data.settings) {
     checkString(data.settings.defaultSearchEngine);
     Assert.equal(typeof data.settings.defaultSearchEngineData, "object");
   }
 
-  if (gIsWindows) {
+  if (gIsWindows && AppConstants.MOZ_BUILD_APP == "browser") {
     Assert.equal(typeof data.settings.attribution, "object");
     Assert.equal(data.settings.attribution.source, "google.com");
   }
 }
 
 function checkProfileSection(data) {
   Assert.ok("profile" in data, "There must be a profile section in Environment.");
   Assert.equal(data.profile.creationDate, truncateToDays(PROFILE_CREATION_DATE_MS));