From bb180145c654c4bc1ab9eca15a7acaa494d9fa50 Mon Sep 17 00:00:00 2001
From: Solly Ross <sross@redhat.com>
Date: Mon, 24 Aug 2015 19:30:44 -0400
Subject: [PATCH] Tests: Fixed bug in displayed assertion

This `displayed` assertion had a bug that was causing it to not
actually check anything (it was using obj.length instead of
data_cl.length).  This fixes that.
---
 tests/assertions.js | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/tests/assertions.js b/tests/assertions.js
index 930e1460..4bd0cf40 100644
--- a/tests/assertions.js
+++ b/tests/assertions.js
@@ -6,10 +6,15 @@ chai.use(function (_chai, utils) {
         // NB(directxman12): PhantomJS 1.x doesn't implement Uint8ClampedArray, so work around that
         var data = new Uint8Array(data_cl);
         var same = true;
-        for (var i = 0; i < obj.length; i++) {
-            if (data[i] != target_data[i]) {
-                same = false;
-                break;
+        var len = data_cl.length;
+        if (len != target_data.length) {
+            same = false;
+        } else {
+            for (var i = 0; i < len; i++) {
+                if (data[i] != target_data[i]) {
+                    same = false;
+                    break;
+                }
             }
         }
         if (!same) {