Codacity review

Author: blairjordan

js/chess.js

@@ -48,7 +48,7 @@ const SAN = {
   CAPTURE: "x",
   CHECK: "+",
   CHECKMATE: "#"
-}
+};
 const Action = {
   MOVE: 1,
   PLAYER_CAPTURE: 2,
@@ -611,7 +611,7 @@ class Chess {
     this.moves--;
     this.turn({ change: true });
 
-    const [from, to] = [this.get({square:`${move.from.file}${move.from.rank}`}),this.get({square:`${move.to.file}${move.to.rank}`})]
+    const [from, to] = [this.get({square:`${move.from.file}${move.from.rank}`}),this.get({square:`${move.to.file}${move.to.rank}`})];
 
     this.get({square:`${move.from.file}${move.from.rank}`}).piece = to.piece;
 
@@ -656,7 +656,7 @@ class Chess {
 
   // return a copy of the board
   copy() {
-    return this._board.map(r => r.slice().map(f=>Object.assign(Object.create(Object.getPrototypeOf(f)),f)));
+    return this._board.map(r => r.slice().map(f => Object.assign(Object.create(Object.getPrototypeOf(f)),f)));
   }
 
   // returns true if a square exists, else false
@@ -732,7 +732,7 @@ class Chess {
       else if (ambiguous.filter(a => a.rank === from.rank).length === 0)
         fromCoord = from.rank;
       else
-        fromCoord = `${from.file}${from.rank}`
+        fromCoord = `${from.file}${from.rank}`;
     }
 
     if (this._move(from, to) === Action.INVALID_ACTION)
@@ -741,10 +741,10 @@ class Chess {
     if (this.checkmate().white || this.checkmate().black)
       check = "#";
     else if (this.check().length > 0)
-      check = "+"
+      check = "+";
     this.undo();
 
-    return `${pieceName}${fromCoord}${capture}${to.file}${to.rank}${check}`
+    return `${pieceName}${fromCoord}${capture}${to.file}${to.rank}${check}`;
   }
 }
 

test/game.js

@@ -127,7 +127,7 @@ describe("Game tests", () => {
       chess.undo();
       let enPassant = chess.fen() === "rnbqkbnr/pp1ppppp/8/2pP4/8/8/PPP1PPPP/RNBQKBNR";
       chess.undo();
-      let two_moves = chess.fen() == "rnbqkbnr/pppppppp/8/3P4/8/8/PPP1PPPP/RNBQKBNR";
+      let twoMoves = chess.fen() === "rnbqkbnr/pppppppp/8/3P4/8/8/PPP1PPPP/RNBQKBNR";
 
       chess = new Chess({fen:"r3k2r/pppppppp/8/8/8/8/PPPPPPPP/R3K2R"});
       chess.move({from:"e1", to: "a1"});
@@ -147,9 +147,9 @@ describe("Game tests", () => {
       chess = new Chess({fen:"r3k2r/pppppppp/8/8/8/8/PPPPPPPP/R3K2R"});
       chess.move({from:"e8", to: "h8"});
       chess.undo();
-      let blackCastleKing = chess.fen() === "r3k2r/pppppppp/8/8/8/8/PPPPPPPP/R3K2R"
+      let blackCastleKing = chess.fen() === "r3k2r/pppppppp/8/8/8/8/PPPPPPPP/R3K2R";
 
-      assert (start && enPassant && two_moves && whiteCastleQueen && whiteCastleKing && blackCastleQueen && blackCastleKing);
+      assert (start && enPassant && twoMoves && whiteCastleQueen && whiteCastleKing && blackCastleQueen && blackCastleKing);
 
       done();
     } catch (e) {