Use jsxhint and fix all warnings/errors

Author: mjackson

.jshintignore

@@ -0,0 +1,2 @@
+dist
+node_modules

.jshintrc

@@ -0,0 +1,6 @@
+{
+  "eqnull": true,
+  "esnext": true,
+  "unused": "vars",
+  "predef": [ "-Promise" ]
+}

eslint.json

@@ -1,12 +1,10 @@
-var assert = require('assert');
 var expect = require('expect');
 var React = require('react');
+var Router = require('../index');
 var Route = require('../components/Route');
-var RouteHandler = require('../components/RouteHandler');
 var TestLocation = require('../locations/TestLocation');
 var ScrollToTopBehavior = require('../behaviors/ScrollToTopBehavior');
 var getWindowScrollPosition = require('../utils/getWindowScrollPosition');
-var Router = require('../index');
 
 var {
   Foo,
@@ -681,7 +679,7 @@ describe('Router.run', function () {
 
   describe('locations', function () {
     it('defaults to HashLocation', function (done) {
-      var routes = <Route path="/" handler={Foo}/>
+      var routes = <Route path="/" handler={Foo}/>;
       var div = document.createElement('div');
       Router.run(routes, function (Handler) {
         React.render(<Handler/>, div, function () {

modules/__tests__/Router-test.js

@@ -16,19 +16,19 @@ exports.Nested = React.createClass({
 
 exports.Foo = React.createClass({
   render: function () {
-    return <div className="Foo">Foo</div>
+    return <div className="Foo">Foo</div>;
   }
 });
 
 exports.Bar = React.createClass({
   render: function () {
-    return <div className="Bar">Bar</div>
+    return <div className="Bar">Bar</div>;
   }
 });
 
 exports.Baz = React.createClass({
   render: function () {
-    return <div className="Baz">Baz</div>
+    return <div className="Baz">Baz</div>;
   }
 });
 
@@ -42,7 +42,7 @@ exports.Async = React.createClass({
   },
 
   render: function () {
-    return <div className="Async">Async</div>
+    return <div className="Async">Async</div>;
   }
 });
 
@@ -112,6 +112,6 @@ exports.EchoFooProp = React.createClass({
 exports.EchoBarParam = React.createClass({
   mixins: [ State ],
   render: function () {
-    return <div className="EchoBarParam">{this.getParams().bar}</div>
+    return <div className="EchoBarParam">{this.getParams().bar}</div>;
   }
 });

modules/__tests__/TestHandlers.js

@@ -1,11 +1,8 @@
-/** @jsx React.DOM */
-var assert = require('assert');
 var expect = require('expect');
-var React = require('react/addons');
+var React = require('react');
 var Router = require('../../index');
 var DefaultRoute = require('../DefaultRoute');
 var Route = require('../Route');
-var RouteHandler = require('../RouteHandler');
 var { Foo, Bar, Nested } = require('../../__tests__/TestHandlers');
 
 describe('DefaultRoute', function () {

modules/components/__tests__/DefaultRoute-test.js

@@ -2,7 +2,6 @@ var assert = require('assert');
 var expect = require('expect');
 var React = require('react/addons');
 var Router = require('../../index');
-var ReactTestUtils = React.addons.TestUtils;
 var Route = require('../Route');
 var Link = require('../Link');
 var RouteHandler = require('../RouteHandler');
@@ -15,7 +14,7 @@ describe('A Link', function () {
     it('knows how to make its href', function () {
       var LinkHandler = React.createClass({
         render: function () {
-          return <Link to="foo" params={{bar: 'baz'}} query={{qux: 'quux'}}>Link</Link>
+          return <Link to="foo" params={{bar: 'baz'}} query={{qux: 'quux'}}>Link</Link>;
         }
       });
 

modules/components/__tests__/Link-test.js

@@ -1,11 +1,8 @@
-/** @jsx React.DOM */
-var assert = require('assert');
 var expect = require('expect');
-var React = require('react/addons');
+var React = require('react');
 var Router = require('../../index');
 var NotFoundRoute = require('../NotFoundRoute');
 var Route = require('../Route');
-var RouteHandler = require('../RouteHandler');
 var { Nested, Foo, Bar } = require('../../__tests__/TestHandlers');
 
 describe('NotFoundRoute', function () {

modules/components/__tests__/NotFoundRoute-test.js

@@ -1,7 +1,5 @@
-/** @jsx React.DOM */
-var assert = require('assert');
 var expect = require('expect');
-var React = require('react/addons');
+var React = require('react');
 var Router = require('../../index');
 var TestLocation = require('../../locations/TestLocation');
 var { Nested, Bar } = require('../../__tests__/TestHandlers');
@@ -20,7 +18,7 @@ describe('Redirect', function () {
     ];
 
     Router.run(routes, TestLocation, function (Handler) {
-      var html = React.render(<Handler />, div);
+      React.render(<Handler />, div);
       expect(div.innerHTML).toMatch(/Bar/);
     });
   });
@@ -36,7 +34,7 @@ describe('Redirect', function () {
       ];
 
       Router.run(routes, TestLocation, function (Handler) {
-        var html = React.render(<Handler />, div);
+        React.render(<Handler />, div);
         expect(div.innerHTML).toMatch(/Bar/);
       });
     });
@@ -57,7 +55,7 @@ describe('Redirect', function () {
       );
 
       Router.run(routes, TestLocation, function (Handler) {
-        var html = React.render(<Handler />, div);
+        React.render(<Handler />, div);
         expect(div.innerHTML).toMatch(/Bar/);
       });
     });
@@ -76,7 +74,7 @@ describe('Redirect', function () {
       );
 
       Router.run(routes, TestLocation, function (Handler) {
-        var html = React.render(<Handler />, div);
+        React.render(<Handler />, div);
         expect(div.innerHTML).toMatch(/Bar/);
       });
     });

modules/components/__tests__/Redirect-test.js

@@ -1,16 +1,10 @@
-/** @jsx React.DOM */
-var assert = require('assert');
 var expect = require('expect');
 var React = require('react');
 var Router = require('../../index');
 var Route = require('../Route');
 var RouteHandler = require('../RouteHandler');
 var TestLocation = require('../../locations/TestLocation');
-var {
-  Bar,
-  Foo
-} = require('../../__tests__/TestHandlers');
-
+var { Bar, Foo } = require('../../__tests__/TestHandlers');
 
 describe('RouteHandler', function () {