when detecting the clipRect ignore elements that are not visible and trigger an error if no element is visible

1 files changed, 12 insertions, 0 deletions

diff --git a/tests/lib/screenshot-testing/support/page-renderer.js b/tests/lib/screenshot-testing/support/page-renderer.js
index ab9c0f170e..dd072a47e5 100644
--- a/tests/lib/screenshot-testing/support/page-renderer.js
+++ b/tests/lib/screenshot-testing/support/page-renderer.js
@@ -242,6 +242,11 @@ PageRenderer.prototype.capture = function (outputPath, callback, selector) {
                 element.each(function (index, node) {
                     var rect = node.getBoundingClientRect();
 
+                    if (!rect.width || !rect.height) {
+                        // element is not visible
+                        return;
+                    }
+
                     if (null === clipRect.left || rect.left < clipRect.left) {
                         clipRect.left = rect.left;
                     }
@@ -272,6 +277,13 @@ PageRenderer.prototype.capture = function (outputPath, callback, selector) {
             throw new Error("Error while detecting element clipRect " + selector + ": " + result.message);
         }
 
+        if (null === result.left
+            || null === result.top
+            || null === result.bottom
+            || null === result.right) {
+            throw new Error("Element(s) " + selector + " found but none is visible");
+        }
+
         page.clipRect = result;
     }