update test harness and jshint checking on top of #27

Author: bgrins

Gruntfile.js

@@ -13,10 +13,10 @@ module.exports = function(grunt) {
       options: {
         browser: true,
         sub: true,
-        globals: {
-        }
+        undef: true,
+        unused: true
       },
-      all: ['astar.js']
+      all: ['astar.js', 'test/tests.js', 'demo/demo.js']
     }
   });
 

astar.js

@@ -6,6 +6,7 @@
 // http://eloquentjavascript.net/appendix2.html
 
 (function(definition) {
+    /* global module, define */
     if(typeof module === 'object' && typeof module.exports === 'object') {
         module.exports = definition();
     } else if(typeof define === 'function' && define.amd) {

demo/demo.js

@@ -3,36 +3,10 @@
 
     Set up the demo page for the A* Search
 */
-
-window.log = function(){
-    if(this.console){
-        console.log( Array.prototype.slice.call(arguments) );
-    }
-};
+/* global Graph, astar, $ */
 
 var WALL = 0,
-    OPEN = 1;
-
-var generateRandom = function (width, height, wallFrequency) {
-    var nodes = [];
-    for (var x = 0; x < width; x++) {
-        var nodeRow = [],
-            gridRow = [];
-
-        for (var y = 0; y < height; y++) {
-            var isWall = Math.floor(Math.random()*(1/wallFrequency));
-            if (isWall === 0) {
-                nodeRow.push(WALL);
-            }
-            else {
-                nodeRow.push(OPEN);
-            }
-        }
-        nodes.push(nodeRow);
-    }
-
-    return new Graph(nodes);
-};
+    performance = window.performance;
 
 $(function() {
 
@@ -165,7 +139,6 @@ GraphSearch.prototype.cellClicked = function($end) {
     var end = this.nodeFromElement($end);
 
     if($end.hasClass(css.wall) || $end.hasClass(css.start)) {
-        log("clicked on wall or start...", $end);
         return;
     }
 
@@ -196,7 +169,7 @@ GraphSearch.prototype.drawDebugInfo = function() {
     this.$cells.html(" ");
     var that = this;
     if(this.opts.debug) {
-        that.$cells.each(function(i) {
+        that.$cells.each(function() {
             var node = that.nodeFromElement($(this)),
                 debug = false;
             if (node.visited) {

package.json

@@ -16,8 +16,8 @@
   },
   "homepage": "http://bgrins.github.io/javascript-astar",
   "devDependencies": {
-    "grunt": "~0.4.2",
-    "grunt-contrib-qunit": "~0.4.0",
-    "grunt-contrib-jshint": "~0.8.0"
+    "grunt": "~0.4.5",
+    "grunt-contrib-qunit": "~0.5.0",
+    "grunt-contrib-jshint": "~0.10.0"
   }
 }

test/tests.js

@@ -1,4 +1,7 @@
 
+/* global Graph, astar, ok, test, equal */
+
+
 test( "Sanity Checks", function() {
   ok (typeof Graph !== "undefined", "Graph exists");
   ok (typeof astar !== "undefined", "Astar exists");