Prefer const/let over var

[mirror_novnc.git] / tests / assertions.js

diff --git a/tests/assertions.js b/tests/assertions.js
index 4f1cdc1787f2cae7559047716fbbd7ccb2cce08b..a336a7a1bb3b61d6967ca6df9cfcb039f2d96900 100644 (file)
--- a/tests/assertions.js
+++ b/tests/assertions.js
@@ -5,15 +5,15 @@ chai.use(sinonChai);
 // noVNC specific assertions
 chai.use(function (_chai, utils) {
     _chai.Assertion.addMethod('displayed', function (target_data) {
-        var obj = this._obj;
-        var ctx = obj._target.getContext('2d');
-        var data_cl = ctx.getImageData(0, 0, obj._target.width, obj._target.height).data;
+        const obj = this._obj;
+        const ctx = obj._target.getContext('2d');
+        const data_cl = ctx.getImageData(0, 0, obj._target.width, obj._target.height).data;
         // NB(directxman12): PhantomJS 1.x doesn't implement Uint8ClampedArray, so work around that
-        var data = new Uint8Array(data_cl);
-        var len = data_cl.length;
+        const data = new Uint8Array(data_cl);
+        const len = data_cl.length;
         new chai.Assertion(len).to.be.equal(target_data.length, "unexpected display size");
-        var same = true;
-        for (var i = 0; i < len; i++) {
+        let same = true;
+        for (let i = 0; i < len; i++) {
             if (data[i] != target_data[i]) {
                 same = false;
                 break;
@@ -31,19 +31,19 @@ chai.use(function (_chai, utils) {
     });
 
     _chai.Assertion.addMethod('sent', function (target_data) {
-        var obj = this._obj;
+        const obj = this._obj;
         obj.inspect = function () {
-            var res = { _websocket: obj._websocket, rQi: obj._rQi, _rQ: new Uint8Array(obj._rQ.buffer, 0, obj._rQlen),
+            const res = { _websocket: obj._websocket, rQi: obj._rQi, _rQ: new Uint8Array(obj._rQ.buffer, 0, obj._rQlen),
                         _sQ: new Uint8Array(obj._sQ.buffer, 0, obj._sQlen) };
             res.prototype = obj;
             return res;
         };
-        var data = obj._websocket._get_sent_data();
-        var same = true;
+        const data = obj._websocket._get_sent_data();
+        let same = true;
         if (data.length != target_data.length) {
             same = false;
         } else {
-            for (var i = 0; i < data.length; i++) {
+            for (let i = 0; i < data.length; i++) {
                 if (data[i] != target_data[i]) {
                     same = false;
                     break;
@@ -68,13 +68,12 @@ chai.use(function (_chai, utils) {
     _chai.Assertion.overwriteMethod('equal', function (_super) {
         return function assertArrayEqual(target) {
             if (utils.flag(this, 'array')) {
-                var obj = this._obj;
+                const obj = this._obj;
 
-                var i;
-                var same = true;
+                let same = true;
 
                 if (utils.flag(this, 'deep')) {
-                    for (i = 0; i < obj.length; i++) {
+                    for (let i = 0; i < obj.length; i++) {
                         if (!utils.eql(obj[i], target[i])) {
                             same = false;
                             break;
@@ -86,7 +85,7 @@ chai.use(function (_chai, utils) {
                         "expected #{this} not to have elements deeply equal to #{exp}",
                         Array.prototype.slice.call(target));
                 } else {
-                    for (i = 0; i < obj.length; i++) {
+                    for (let i = 0; i < obj.length; i++) {
                         if (obj[i] != target[i]) {
                             same = false;
                             break;