Issue 6884 - Migrate from nodeunit to mocha

test/firefox.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 51 matching lines @@
       return driver.wait(() =>
         driver.getAllWindowHandles().then(handles => handles[1])
       ).then(handle =>
         driver.switchTo().window(handle)
       ).then(() =>
         driver.executeScript("return location.origin;")
       ).then(result => { origin = result; });
     })
   );
 
-  after(() => driver.quit());

[tlucas, 2018/08/26 15:10:51]

Nit: IMHO this should go after it(), to reflect the actual execution order.

[Sebastian Noack, 2018/08/26 19:02:00]

Done. Though one could argue the other way around to keep the hooks together.

-
-  function reportElements(success)
-  {
-    return driver.findElements(
-      By.css(`#qunit-tests ${success ? ".pass" : ".fail"} .test-name`)
-    ).then(elements => Promise.all(elements.map(elem =>
-      elem.getAttribute("innerHTML").then(data => assert.ok(success, data))
-    )));
-  }
-
   it("qunit", () =>
     driver.navigate().to(origin + "/qunit/index.html").then(() =>

[tlucas, 2018/08/26 15:10:51]

Just FMI, why do you deem this more appropriate than
"driver.executeScript("location.href = \"qunit/index.html\";")",
especially since you run executeScript() anyway in line 67 to read to retrieve
origin?

[Sebastian Noack, 2018/08/26 15:42:18]

I need the extension URL prefix for the tests I'm going to add anyway. Also once
we add more tests (reusing the same browser instance), it will be a hassle to
guarantee that the first run page will be open and focused at the beginning of
each test. By getting the origin upfront, we have larger code reuse, and less
assumptions in the individual tests.

[tlucas, 2018/08/26 21:13:50]

Thanks for clarifying.

       // Wait for qunit-results to be present
       driver.wait(until.elementLocated(By.id("qunit-testresult")))
     ).then(() =>
       // Wait for tests to finish
       driver.wait(() =>
         driver.findElement(By.id("qunit-testresult"))
           .getAttribute("innerHTML").then(data =>
              data.includes("Tests completed")))
-    ).then(() => Promise.all([
-      reportElements(true),
-      reportElements(false)
-    ]))
+    ).then(() => Promise.all([[true, ".pass"], [false, ".fail"]].map(
+      ([success, sel]) => driver.findElements(
+        By.css(`#qunit-tests ${sel} .test-name`)
+      ).then(elements => Promise.all(elements.map(elem =>
+        elem.getAttribute("textContent").then(data => assert.ok(success, data))
+      )))
+    )))
   );
+
+  after(() => driver.quit());
 });