Merge pull request #24 from quaertym/revert-23-check-dependencies-when-included

Revert "Move dependency checking to included hook"

Author: quaertym

lib/dependency-checker.js

@@ -16,11 +16,8 @@ function isUnsatisfied(pkg) {
 function EmberCLIDependencyChecker(project) {
   this.name    = 'ember-cli-dependency-checker';
   this.project = project;
-}
-
-EmberCLIDependencyChecker.prototype.included = function(/*app*/) {
   this.checkDependencies();
-};
+}
 
 EmberCLIDependencyChecker.prototype.checkDependencies = function() {
 

tests/helpers/assert-error.js

@@ -5,12 +5,10 @@ var DependencyChecker = require('../../lib/dependency-checker');
 var DependencyError   = require('../../lib/dependency-error');
 
 module.exports = function(project, type) {
-
-  var includeDependencyChecker = function() {
-    var dependencyChecker = new DependencyChecker(project);
-    dependencyChecker.included();
+  var dependencyChecker = function() {
+    return new DependencyChecker(project);
   };
 
-  assert.throws(includeDependencyChecker, DependencyError, 'Missing ' + type + ' packages');
+  assert.throws(dependencyChecker, DependencyError, 'Missing ' + type + ' packages');
   DependencyChecker.setAlreadyChecked(false);
 };

tests/helpers/assert-no-error.js

@@ -5,12 +5,10 @@ var DependencyChecker = require('../../lib/dependency-checker');
 var DependencyError   = require('../../lib/dependency-error');
 
 module.exports = function(project, type) {
-
-  var includeDependencyChecker = function() {
-    var dependencyChecker = new DependencyChecker(project);
-    dependencyChecker.included();
+  var dependencyChecker = function() {
+    return new DependencyChecker(project);
   };
 
-  assert.doesNotThrow(includeDependencyChecker, DependencyError, 'Missing ' + type + 'packages');
+  assert.doesNotThrow(dependencyChecker, DependencyError, 'Missing ' + type + 'packages');
   DependencyChecker.setAlreadyChecked(false);
 };