bug 1533831: remote: remove test_FatalError; r=ochameau

FatalError uses Services.startup.quit and this is apparently fine
to run in an xpcshell test with an optimised build, but not with
debug builds where it causes MOZ_CRASH() to be hit.

This removes test_FatalError so that we can run test_Error.js on try.

Differential Revision: https://phabricator.services.mozilla.com/D28705

--HG--
extra : moz-landing-system : lando

remote/test/unit/test_Error.js

@@ -5,7 +5,6 @@
 /* eslint-disable no-tabs */
 
 const {
-  FatalError,
   RemoteAgentError,
   UnknownMethodError,
   UnsupportedError,
@@ -81,17 +80,6 @@ caused by: DogError: woof:
   run_next_test();
 });
 
-// FatalError calls nsIAppStartup.quit(), but it's OK because this is an xpcshell test:
-add_test(function test_FatalError() {
-  const e = new FatalError();
-  ok(e instanceof RemoteAgentError);
-  e.notify();
-  equal(e.toString(), RemoteAgentError.format(e));
-  e.quit();
-
-  run_next_test();
-});
-
 add_test(function test_UnsupportedError() {
   ok(new UnsupportedError() instanceof RemoteAgentError);
   run_next_test();