Merge pull request #94 from Turbo87/pnp

DependencyChecker: Reuse `resolveSync()` method on `Project` class

Author: rwjblue

lib/dependency-checker.js

@@ -1,7 +1,6 @@
 'use strict';
 
 const path       = require('path');
-const resolve = require('resolve');
 const fs         = require('fs');
 const findYarnWorkspaceRoot = require('find-yarn-workspace-root');
 const readFile   = fs.readFileSync;
@@ -89,7 +88,7 @@ class EmberCLIDependencyChecker {
 
   lookupNodeModule(name, versionSpecified) {
     try {
-      const nodePackage = resolve.sync(path.join(name, 'package.json'), { basedir: this.project.root });
+      const nodePackage = this.project.resolveSync(path.join(name, 'package.json'));
       const version = this.lookupPackageVersion(nodePackage, versionSpecified);
       return { version: version, path: path.dirname(nodePackage) };
     } catch(err) {

tests/helpers/project-builder.js

@@ -1,5 +1,7 @@
 'use strict';
 
+const resolve = require('resolve');
+
 let defaultProject, projectBuilder;
 
 function simpleMerge(target, object) {
@@ -46,7 +48,11 @@ defaultProject = {
   root: 'tests/fixtures/project-npm-check',
   bowerDirectory: 'bower_components',
   dependencies: projectBuilder.buildDependencies(),
-  bowerDependencies: projectBuilder.buildBowerDependencies()
+  bowerDependencies: projectBuilder.buildBowerDependencies(),
+
+  resolveSync(path) {
+    return resolve.sync(path, { basedir: this.root });
+  }
 };
 
 module.exports = projectBuilder;