Bug 1340631 - Remove the telemetry-related crash annotation tests r?Ted draft
authorGabriele Svelto <gsvelto@mozilla.com>
Tue, 21 Feb 2017 22:36:12 +0100
changeset 487616 92e4960c4fb1b62dcbc04995d01edaba10ba5ab2
parent 487583 c7b015c488cfb2afbcff295a9639acd85df332f8
child 487863 015ed0f2e17efb4246c894430746c5f2f5ad39dc
child 487867 d4682b4265a07b9666fbb3cea1f26e5d0c4e19e9
push id46272
push userbmo:gsvelto@mozilla.com
push dateTue, 21 Feb 2017 21:43:59 +0000
reviewersTed
bugs1340631
milestone54.0a1
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");
-  });
 }