Bug 1340631 - Remove the telemetry-related crash annotation tests r=Ted
authorGabriele Svelto <gsvelto@mozilla.com>
Tue, 21 Feb 2017 22:36:12 +0100
changeset 394501 b73f06be7ce6c4d3d1aa4cac1c449a92d8f9c874
parent 394500 053c34ac7631c30954fd5cfabf5d6fcdc28571f0
child 394502 94047c88a870ad72ae864a16c695ed8a825ffb22
push id1468
push userasasaki@mozilla.com
push dateMon, 05 Jun 2017 19:31:07 +0000
treeherdermozilla-release@0641fc6ee9d1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersTed
bugs1340631
milestone54.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 1340631 - Remove the telemetry-related crash annotation tests r=Ted MozReview-Commit-ID: BS4jS9ph1fX
toolkit/crashreporter/test/unit/test_crashreporter_crash.js
--- a/toolkit/crashreporter/test/unit/test_crashreporter_crash.js
+++ b/toolkit/crashreporter/test/unit/test_crashreporter_crash.js
@@ -41,34 +41,10 @@ function run_test() {
              Components.utils.import("resource://gre/modules/TelemetryController.jsm", scope);
              scope.TelemetryController.testSetup();
            },
            function(mdump, extra) {
              do_check_eq(extra.TestKey, "TestValue");
              do_check_eq(extra["\u2665"], "\u{1F4A9}");
              do_check_eq(extra.Notes, "JunkMoreJunk");
              do_check_true("TelemetrySessionId" in extra);
-             Assert.ok(
-              "TelemetryServerURL" in extra,
-              "The TelemetryServerURL field is omitted when telemetry is off"
-             );
            });
-
-  do_crash(function() {
-    // Enable telemetry
-    let prefs = Components.classes["@mozilla.org/preferences-service;1"]
-                          .getService(Components.interfaces.nsIPrefBranch);
-
-    // Turn on telemetry and specify a telemetry server
-    prefs.setCharPref("toolkit.telemetry.server", "http://a.telemetry.server");
-    prefs.setBoolPref("toolkit.telemetry.enabled", true);
-
-    // TelemetrySession setup will trigger the session annotation
-    let scope = {};
-    Components.utils.import("resource://gre/modules/TelemetryController.jsm", scope);
-    scope.TelemetryController.testSetup();
-  }, function(mdump, extra) {
-    Assert.ok("TelemetryServerURL" in extra,
-              "The TelemetryServerURL field is present in the extra file");
-    Assert.equal(extra.TelemetryServerURL, "http://a.telemetry.server",
-                  "The TelemetryServerURL field is properly set");
-  });
 }