style: Using a static get property for Dep class.

let dep = new Dep('foo', '0.0.1');

then call like this:

dep.name
dep.version

lib/dep.js

@@ -1,19 +1,23 @@
 'use strict';
 
+/**
+ * This class is just hold name and version of the dependency.
+ */
 class Dep {
 
-  constructor (name, version) {
-    this.name = name;
-    this.version = version;
+  constructor (n, v) {
+    this.name = n;
+    this.version = v;
   }
 
-  getName () {
+  static get name () {
     return this.name;
   }
 
-  getVersion () {
+  static get version () {
     return this.version;
   }
+
 }
 
 module.exports = Dep;

lib/reporter.js

@@ -82,8 +82,8 @@ function unused (declarations, dependencies) {
     dependencies.forEach(dependency => {
       declarations.forEach(declaration => {
         if (declaration) {
-          if (declaration.toUpperCase().includes(dependency.getName().toUpperCase())) {
-            unused.add(dependency.getName());
+          if (declaration.toUpperCase().includes(dependency.name.toUpperCase())) {
+            unused.add(dependency.name);
           }
         }
       });
@@ -94,7 +94,7 @@ function unused (declarations, dependencies) {
     });
   }
   const unusedArray = Array.from(unused);
-  const diff = dependencies.filter(d => unusedArray.indexOf(d.getName()) < 0);
+  const diff = dependencies.filter(d => unusedArray.indexOf(d.name) < 0);
   return diff.length ? diff : 'None.';
 }
 
@@ -119,7 +119,7 @@ function jsonReporter (result, dependencies, requires) {
 function licenseReporter (dependencies) {
   return Promise.resolve().then(() => {
     const licensePromises = dependencies.map(d => {
-      return roi.get({'endpoint': `http://registry.npmjs.org/${d.getName()}/${d.getVersion()}`});
+      return roi.get({'endpoint': `http://registry.npmjs.org/${d.name}/${d.version}`});
     });
 
     return Promise.all(licensePromises).then((results) => {
@@ -153,7 +153,7 @@ function consoleReport (jsonReport, options) {
   header('[ Unused dependencies ]');
   if (Array.isArray(jsonReport.unused)) {
     jsonReport.unused.forEach(d => {
-      log.red(d.getName());
+      log.red(d.name);
     });
   } else {
     log.red(jsonReport.unused);

lib/searcher.js

@@ -60,7 +60,7 @@ function searchDeclarations (lines, dependencies) {
   const declarations = [];
   dependencies.forEach(d => {
     lines.forEach(l => {
-      if (l.includes(d.getName())) {
+      if (l.includes(d.name)) {
         let extractedRequire = getRequireFromLine(l);
         if (extractedRequire) {
           let extractedVarName = getTextFromRequire(l);
@@ -120,10 +120,10 @@ function searchMissingDependencies (lines, dependencies) {
         missing = getTextFromRequire(missing).trim();
 
         let deps = dependencies[0].find(d => {
-          return d.getName() === missing;
+          return d.name === missing;
         });
         let devDeps = dependencies[1].find(d => {
-          return d.getName() === missing;
+          return d.name === missing;
         });
         let core = dependencies[2].find(d => {
           return d === missing;

test/szero-api-test.js

@@ -41,7 +41,7 @@ test('should return unused Array with 1 Dep Object', (t) => {
   szero.report(dir, options).then((jsonReport) => {
     t.true(jsonReport.unused, 'should have a unused object');
     t.equal(Array.isArray(jsonReport.unused), true, 'should be an array');
-    t.equal(jsonReport.unused[0].getName(), 'swapi-node', 'should be an array');
+    t.equal(jsonReport.unused[0].name, 'swapi-node', 'should be an array');
     t.end();
   });
 });

test/szero-cli-test.js

@@ -28,8 +28,8 @@ test('Should run a summary report', (t) => {
   cli(path.join(__dirname, '/fixtures'), {
     summary: true
   }).then((report) => {
-    t.deepEqual(report.dependencies.map(d => d.getName()), [ 'roi', 'fidelity', 'request', 'ramda' ]);
-    t.deepEqual(report.unused.map(d => d.getName()), [ 'request' ]);
+    t.deepEqual(report.dependencies.map(d => d.name), [ 'roi', 'fidelity', 'request', 'ramda' ]);
+    t.deepEqual(report.unused.map(d => d.name), [ 'request' ]);
     t.deepEqual(report.devDependencies, []);
     t.pass();
     t.end();

test/szero-test.js

@@ -40,7 +40,7 @@ test('Should find javascript files.', t => {
 test('Should search for dependencies.', t => {
   const lines = reader.read(path.join(__dirname, '/fixtures/package.json'));
   const dependencies = searcher.searchDependencies(lines, true);
-  t.equal(dependencies[0][0].getName() === 'roi', true);
+  t.equal(dependencies[0][0].name === 'roi', true);
   t.end();
 });
 
@@ -123,7 +123,7 @@ test('Should show unused dependencies from report.', t => {
   const javascriptLines = reader.read(path.join(__dirname, '/fixtures/foo/x.js'));
   const declarations = searcher.searchDeclarations(javascriptLines, dependencies[0]);
   const unused = reporter.unused(declarations, dependencies[0]);
-  let names = unused.map(u => u.getName());
+  let names = unused.map(u => u.name);
   t.equal(names.toString(), 'fidelity,request');
   t.end();
 });
@@ -144,7 +144,7 @@ test('Should show all unused dependencies.', t => {
   const javascriptLines = reader.read(path.join(__dirname, '/fixtures/bar/all-unused.js'));
   const declarations = searcher.searchDeclarations(javascriptLines, dependencies[0]);
   const unused = reporter.unused(declarations, dependencies[0]);
-  let names = unused.map(u => u.getName());
+  let names = unused.map(u => u.name);
   t.equal(names.toString(), 'roi');
   t.end();
 });