Fix eslint config (#34)

Upgrade devDependencies, add linting scripts, update eslintrc, run test coverage in test.yml

.eslintrc

@@ -1,3 +1,7 @@
 {
-  "extends": "airbnb"
+    "extends": ["plugin:import/recommended", "plugin:jsx-a11y/recommended", "prettier"],
+    "plugins": ["import", "jsx-a11y", "prettier"],
+    "rules": {
+      "prettier/prettier": ["error"]
+    }
 }

.github/workflows/test.yml

@@ -11,6 +11,7 @@ jobs:
     runs-on: ubuntu-latest
 
     strategy:
+      fail-fast: false
       matrix:
         node-version: [16.x, 18.x]
 

.gitignore

@@ -1,3 +1,4 @@
 node_modules
 *.log
 coverage
+.nyc_output

Gulpfile.js

@@ -1,11 +1,11 @@
 /* eslint import/no-extraneous-dependencies: ["error", {"devDependencies": true}] */
 
-const babel = require('gulp-babel');
-const browserify = require('browserify');
-const gulp = require('gulp');
-const rename = require('gulp-rename');
-const source = require('vinyl-source-stream');
-const uglify = require('gulp-uglify');
+const babel = require("gulp-babel");
+const browserify = require("browserify");
+const gulp = require("gulp");
+const rename = require("gulp-rename");
+const source = require("vinyl-source-stream");
+const uglify = require("gulp-uglify");
 
 /**
  * Prepares the files for browser usage
@@ -14,19 +14,18 @@ const uglify = require('gulp-uglify');
  *  - Transpile with Babel
  *  - Minify with uglify
  */
-gulp.task('build', ['bundle'], () => {
-  gulp.src('./dist/dijkstra.js')
+gulp.task("build", ["bundle"], () => {
+  gulp
+    .src("./dist/dijkstra.js")
     .pipe(babel())
-    .pipe(gulp.dest('./dist'))
+    .pipe(gulp.dest("./dist"))
     .pipe(uglify())
-    .pipe(rename({ suffix: '.min' }))
-    .pipe(gulp.dest('./dist'));
+    .pipe(rename({ suffix: ".min" }))
+    .pipe(gulp.dest("./dist"));
 });
 
-gulp.task('bundle', () => {
-  const b = browserify({ entries: './libs/Graph.js' });
+gulp.task("bundle", () => {
+  const b = browserify({ entries: "./libs/Graph.js" });
 
-  return b.bundle()
-    .pipe(source('dijkstra.js'))
-    .pipe(gulp.dest('./dist'));
+  return b.bundle().pipe(source("dijkstra.js")).pipe(gulp.dest("./dist"));
 });

libs/Graph.js

@@ -1,11 +1,10 @@
-const Queue = require('./PriorityQueue');
-const removeDeepFromMap = require('./removeDeepFromMap');
-const toDeepMap = require('./toDeepMap');
-const validateDeep = require('./validateDeep');
+const Queue = require("./PriorityQueue");
+const removeDeepFromMap = require("./removeDeepFromMap");
+const toDeepMap = require("./toDeepMap");
+const validateDeep = require("./validateDeep");
 
 /** Creates and manages a graph */
 class Graph {
-
   /**
    * Creates a new Graph, optionally initializing it a nodes graph representation.
    *
@@ -287,7 +286,6 @@ class Graph {
   shortestPath(...args) {
     return this.path(...args);
   }
-
 }
 
 module.exports = Graph;

libs/PriorityQueue.js

@@ -9,7 +9,6 @@
  * provided by the class.
  */
 class PriorityQueue {
-
   /**
    * Creates a new empty priority queue
    */
@@ -99,9 +98,8 @@ class PriorityQueue {
    * @return {object}
    */
   get(key) {
-    return this.queue.find(element => element.key === key);
+    return this.queue.find((element) => element.key === key);
   }
-
 }
 
 module.exports = PriorityQueue;

libs/toDeepMap.js

@@ -28,12 +28,15 @@ function toDeepMap(source) {
   keys.forEach((key) => {
     const val = source[key];
 
-    if (val !== null && typeof val === 'object' && !Array.isArray(val)) {
+    if (val !== null && typeof val === "object" && !Array.isArray(val)) {
       return map.set(key, toDeepMap(val));
     }
 
     if (!isValidNode(val)) {
-      throw new Error(`Could not add node at key "${key}", make sure it's a valid node`, val);
+      throw new Error(
+        `Could not add node at key "${key}", make sure it's a valid node`,
+        val
+      );
     }
 
     return map.set(key, Number(val));

libs/validateDeep.js

@@ -9,13 +9,15 @@ function validateDeep(map) {
   }
 
   map.forEach((value, key) => {
-    if (typeof value === 'object' && value instanceof Map) {
+    if (typeof value === "object" && value instanceof Map) {
       validateDeep(value);
       return;
     }
 
-    if (typeof value !== 'number' || value <= 0) {
-      throw new Error(`Values must be numbers greater than 0. Found value ${value} at ${key}`);
+    if (typeof value !== "number" || value <= 0) {
+      throw new Error(
+        `Values must be numbers greater than 0. Found value ${value} at ${key}`
+      );
     }
   });
 }