Noissue - Allow identifying the element in case of error in test

test/browser/elemHideEmulation.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 39 matching lines @@
 }
 
 function unexpectedError(error)
 {
   console.error(error);
   this.ok(false, "Unexpected error: " + error);
 }
 
 function expectHidden(test, element, id)
 {
-  let elementId = id ? `: ${id}` : "";
+  let withId = "";
+  if (typeof id != "undefined")
+    withId = ` with ID '${id}'`;
+
   test.equal(
     window.getComputedStyle(element).display, "none",
-    `The element's display property should be set to 'none' ${elementId}`);

[kzar, 2018/03/19 16:55:20]

Please can you improve the message, like "The element with ID 'foo'...".

[hub, 2018/03/20 14:40:26]

Done.

+    `The element${withId}'s display property should be set to 'none'`);
 }
 
 function expectVisible(test, element, id)
 {
-  let elementId = id ? `: ${id}` : "";
+  let withId = "";
+  if (typeof id != "undefined")
+    withId = ` with ID '${id}'`;
+
   test.notEqual(
     window.getComputedStyle(element).display, "none",
-    `The element's display property should not be set to 'none' ${elementId}`);
+    `The element${withId}'s display property should not be set to 'none'`);
 }
 
 function findUniqueId()
 {
   let id = "elemHideEmulationTest-" + Math.floor(Math.random() * 10000);
   if (!testDocument.getElementById(id))
     return id;
   return findUniqueId();
 }
 
@@ skipping 604 matching lines @@
     expectVisible(test, child2);
     return timeout(REFRESH_INTERVAL);
   }).then(() =>
   {
     expectHidden(test, parent);
     expectVisible(test, child);
     expectHidden(test, sibling);
     expectVisible(test, child2);
   }).catch(unexpectedError.bind(test)).then(() => test.done());
 };