Issue 6822 - Report browser test errors

test/runners/chromium_remote_process.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-present eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 168 matching lines @@
       });
       if (promiseResult.exceptionDetails)
         throwException(promiseResult.exceptionDetails, scriptName);
 
       return Runtime.evaluate({
         expression: "window._consoleLogs",
         returnByValue: true
       }).then(result =>
       {
         if (result.result.value.failures != 0)
-          return Promise.reject("Chromium (Remote Interface)");

[Sebastian Noack, 2018/09/21 14:22:20]

Nit: You can just use throw here.

[hub, 2018/09/21 15:39:35]

Done.

+          throw "Chromium (Remote Interface)";
       });
     }
     finally
     {
       client.close();
     }
   });
 }
 
 module.exports = function(script, scriptName, ...scriptArgs)
 {
   return ensureChromium(CHROMIUM_REVISION).then(chromiumPath =>
   {
     let child = startChromium(chromiumPath);
     return Promise.race([
       child.done,
       runScript(script, scriptName, scriptArgs)
     ]).then(result =>
     {
       child.kill();
       return result;
     }).catch(error =>
     {
       child.kill();
-      return Promise.reject(error);
+      throw error;
     });
   });
 };