more eslint fixes

Author: amejiarosario

lib/data-structures/graphs/graph.js

@@ -144,11 +144,11 @@ class Graph {
   /**
    * Generic graph search where we can pass a Stack or Queue
    * @param {Node} first node to start the search
-   * @param {Stack|Queue} dsType Stack for DFS or Queue for BFS
+   * @param {Stack|Queue} Type Stack for DFS or Queue for BFS
    */
-  static* graphSearch(first, dsType = Stack) {
+  static* graphSearch(first, Type = Stack) {
     const visited = new Map();
-    const visitList = new dsType();
+    const visitList = new Type();
 
     visitList.add(first);
 
@@ -198,22 +198,23 @@ class Graph {
 
     path.set(sourceNode);
 
-    // eslint-disable-next-line no-iterator
-    for (const node of sourceNode.getAdjacents()) {
+    const wasFound = sourceNode.getAdjacents().find((node) => {
       if (node === destinationNode) {
         path.set(destinationNode);
-        return Array.from(path.keys());
+        return true;
       }
 
       if (!path.has(node)) {
         const newPath = this.findPath(node.value, destination, path);
         if (newPath.length) {
-          return newPath;
+          return true;
         }
       }
-    }
 
-    return [];
+      return false;
+    });
+
+    return wasFound ? Array.from(path.keys()) : [];
   }
 }
 

lib/data-structures/graphs/graph.spec.js

@@ -2,6 +2,7 @@ const Graph = require('./graph');
 
 describe('Graph', () => {
   let graph;
+  const getValue = node => node.value;
 
   beforeEach(() => {
     graph = new Graph();
@@ -268,9 +269,4 @@ describe('Graph', () => {
       });
     });
   });
-
-
-  function getValue(node) {
-    return node.value;
-  }
-});
+});

package-lock.json

@@ -10,6 +10,7 @@
     "eslint": "4.19.1",
     "eslint-config-airbnb-base": "12.1.0",
     "eslint-plugin-import": "2.12.0",
+    "eslint-plugin-jest": "21.17.0",
     "jest": "23.0.1"
   },
   "scripts": {