Applied corrections of code review

Author: cibernox

.jshintrc

@@ -4,8 +4,16 @@
   "undef":  true,
 
   "predef":    [
-    "Handlebars", "Ember",
+    "Handlebars",
+    "Ember",
     "EmberCPM",
-    "describe", "it", "expect", "beforeEach", "afterEach"
+    "describe",
+    "it",
+    "expect",
+    "beforeEach",
+    "afterEach",
+    "require",
+    "module",
+    "process"
   ]
 }

Brocfile.js

@@ -23,7 +23,7 @@ var minifiedFile = concat(emberCPMmin, {
 
 var outTrees = [emberCPM, minifiedFile];
 
-if (process.argv[2] != 'build') {
+if (process.argv[2] !== 'build') {
   var testDeps = pickFiles('node_modules/testem/public/testem/', {
     srcDir: '/',
     files: ['mocha.css', 'mocha.js', 'chai.js'],

Makefile

@@ -19,5 +19,7 @@ ensure_phantomjs:
 	@which phantomjs > /dev/null || (echo "Couldn't find phantomjs" && false)
 
 jshint: $(jshint_bin)
-	@jshint src/*.js spec/*Spec.js
+	@jshint Brocfile.js src/*.js spec/*Spec.js
 	@echo "JSHint OK"
+
+.PHONY: ensure_phantomjs install_dependencies test test-ci jshint

README.md

@@ -27,7 +27,7 @@ To run the rest suite in interactive mode execute `make test` and enter `http://
 
 To run the test suite just once and exit run `make test-ci`.
 
-You can also run the test suitein watch mode with `broccoli serve`.
+You can also run the test suite in watch mode with `broccoli serve`.
 
 ### Run tests
 

spec/index.html

@@ -7,13 +7,7 @@
     <script src="ember-cpm.js"></script>
     <script src="chai.js"></script>
     <script src="mocha.js"></script>
-    <script>
-      Ember.testing = true;
-      expect = chai.expect;
-      mocha.ui('bdd');
-      mocha.reporter('html');
-      mocha.globals([ 'jQuery*' ]);
-    </script>
+    <script>mocha.setup('bdd');</script>
     <script src="ember-cpm-specs.js"></script>
     <link rel="stylesheet" href="mocha.css" />
   </head>