Fix linting errors

Also fix a bug in test helpers in which `util` was not required.
remarkablemark · Aug 23, 2016 · d62961a · d62961a
1 parent 9497215
commit d62961a
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 7 deletions.
diff --git a/lib/html-to-dom-client.js b/lib/html-to-dom-client.js
@@ -43,7 +43,8 @@ function formatDOM(nodes, parentNode) {
         };
 
         // set the next node for the previous node (if applicable)
-        if (prevNode = result[i - 1]) {
+        prevNode = result[i - 1];
+        if (prevNode) {
             prevNode.next = nodeObj;
         }
 

diff --git a/test/helpers.js b/test/helpers.js
@@ -30,9 +30,9 @@ describe('test helper', function() {
             var obj1;
             var obj2;
 
-            var obj1 = { foo: 'bar', baz: ['qux', 42, null], obj: {} };
+            obj1 = { foo: 'bar', baz: ['qux', 42, null], obj: {} };
             obj1.ref = obj1.obj;
-            var obj2 = { obj: {}, baz: ['qux', 42, null], foo: 'bar' };
+            obj2 = { obj: {}, baz: ['qux', 42, null], foo: 'bar' };
             obj2.ref = obj2.obj;
             deepEqualCircular(obj1, obj2);
         });

diff --git a/test/helpers/index.js b/test/helpers/index.js
@@ -4,6 +4,7 @@
  * Module dependencies.
  */
 var assert = require('assert');
+var util = require('util');
 
 /**
  * Test for deep equality between objects that have circular references.

diff --git a/test/utilities.js b/test/utilities.js
@@ -30,7 +30,7 @@ describe('utilties', function() {
             [undefined, null, 1337, {}, []].forEach(function(parameter) {
                 assert.throws(function() { camelCase(parameter); });
             });
-        })
+        });
     });
 
     describe('`invertObject` helper', function() {
@@ -40,7 +40,7 @@ describe('utilties', function() {
             [undefined, null, 'foo', 1337].forEach(function(parameter) {
                 assert.throws(function() { invertObject(parameter); });
             });
-        })
+        });
 
         it('swaps key with value for object', function() {
             assert.deepEqual(
@@ -67,7 +67,7 @@ describe('utilties', function() {
 
         it('uses override if valid', function() {
             assert.deepEqual(
-                invertObject({ foo: 'bar', baz: 'qux' }, function(key, value) {
+                invertObject({ foo: 'bar', baz: 'qux' }, function(key) {
                     if (key === 'foo') {
                         return ['key', 'value'];
                     }
@@ -78,7 +78,7 @@ describe('utilties', function() {
 
         it('does not use override if invalid', function() {
             assert.deepEqual(
-                invertObject({ foo: 'bar', baz: 'qux' }, function(key, value) {
+                invertObject({ foo: 'bar', baz: 'qux' }, function(key) {
                     if (key === 'foo') {
                         return ['key'];
                     } else if (key === 'baz') {