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 161 matching lines @@
         arguments: scriptArgs.map(arg => ({value: arg}))
       });
       if (callResult.exceptionDetails)
         throwException(callResult.exceptionDetails, scriptName);
 
       let promiseResult = await Runtime.awaitPromise({
         promiseObjectId: callResult.result.objectId
       });
       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.

+      });
     }
     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();
-      throw error;
+      return Promise.reject(error);
     });
   });
 };