From d75cea856ae458acb3297362ca319d1c9c9a6420 Mon Sep 17 00:00:00 2001
From: El RIDO <elrido@gmx.net>
Date: Mon, 23 Oct 2017 05:34:55 +0200
Subject: [PATCH] concluding UiHelper tests that are possible in headless jsdom
 at this time

---
 js/test.js | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/js/test.js b/js/test.js
index 50fe2edd..6a67972b 100644
--- a/js/test.js
+++ b/js/test.js
@@ -913,6 +913,9 @@ describe('UiHelper', function () {
     });
 
     describe('isVisible', function () {
+        // TODO As per https://github.com/tmpvar/jsdom/issues/1048 there is no layout support in jsdom, yet.
+        // once it is supported or a workaround is found, uncomment the section below
+        /*
         before(function () {
             $.PrivateBin.Helper.reset();
         });
@@ -927,13 +930,16 @@ describe('UiHelper', function () {
                 var clean = jsdom(
                     '<' + element + ' id="' + id + '"></' + element + '>'
                 );
-                // TODO As per https://github.com/tmpvar/jsdom/issues/1048 there is no layout support in jsdom, yet.
-                // remove the "true || " below, once it is supported or a workaround is found
-                var result = true || $.PrivateBin.UiHelper.isVisible($('#' + id));
+                var result = $.PrivateBin.UiHelper.isVisible($('#' + id));
                 clean();
                 return result;
             }
         );
+        */
+    });
+
+    describe('scrollTo', function () {
+        // TODO Did not find a way to test that, see isVisible test above
     });
 });