Issue 6882 - always shutdown the browser gracefully

test/firefox.js

Index: test/firefox.js
diff --git a/test/firefox.js b/test/firefox.js
index 75d744358703ee3106fadcc91b3e0a8bdfb67209..9cd0a025122764d5d129dcca60d6f7553ee1929e 100644
--- a/test/firefox.js
+++ b/test/firefox.js
@@ -94,15 +94,9 @@ exports.runFirefox = function(test)
     ).then(() => Promise.all([
       reportElements(test, driver, true),
       reportElements(test, driver, false)
-    ])).then(() =>
-    {
-      driver.quit();
-      test.done();
-    }, err =>
-      driver.quit().then(() =>
-      {
-        throw err;
-      })
+    ])).then(
+      () => driver.quit().then(() => test.done()),
+      err => driver.quit().then(() => { throw err; })

[tlucas, 2018/08/25 09:36:01]

Our coding-style (https://adblockplus.org/en/coding-style @ "General") tells us
to always put opening braces on their own line.

I looked for exceptions, but found none. Do we practice this even in this case,
where newlines would unnecessarily (imho) mess up readability?

[Sebastian Noack, 2018/08/25 09:51:38]

One liners are fine too.

     );
   });
 };