fix(core): update

test/test.ts

@@ -11,15 +11,15 @@ tap.test('should create valid instances', async () => {
   npmRegistry = new smartnpm.NpmRegistry();
   expect(npmRegistry).to.be.instanceof(smartnpm.NpmRegistry);
 
-  testPackage = new smartnpm.NpmPackage({});
+  testPackage = new smartnpm.NpmPackage();
   expect(testPackage).to.be.instanceof(smartnpm.NpmPackage);
 });
 
 tap.test('should produce a valid search string and this return npmts', async () => {
   const packages = await npmRegistry.search({
-    name: 'npmts'
+    name: '@pushrocks/smartupdate'
   });
-  expect(packages[0].name).to.equal('npmts');
+  expect(packages[0].name).to.equal('@pushrocks/smartupdate');
 });
 
 // lets test things with the verdaccio registry
@@ -31,8 +31,8 @@ tap.test('should create a verdaccio registry', async () => {
 });
 
 tap.test('should get package from verdaccui', async () => {
-  const packageInfo = await verdaccioRegistry.getPackageInfo('@pushrocks/smartupdate');
+  const npmPackage = await verdaccioRegistry.getPackageInfo('@pushrocks/smartupdate');
-  expect(packageInfo.license).to.equal('MIT');
+  expect(npmPackage.license).to.equal('MIT');
 });
 
 tap.start();

ts/smartnpm.classes.npmpackage.ts

@@ -1,27 +1,35 @@
 import * as plugins from './smartnpm.plugins';
 
 export class NpmPackage { 
-  name: string = null;
+  public static async createFromFullMetadata(fullMetadata: plugins.packageJson.FullMetadata) {
-  scope: string = null;
+    const npmPackage = new NpmPackage();
-  version: string = null;
+    Object.assign(npmPackage, fullMetadata);
-  description: string = null;
+    return npmPackage;
-  keywords: string[] = null;
+  }
-  date: '2017-08-02T11:22:49.144Z';
+
-  links: {
+  // INSTANCE
+  public name: string = null;
+  public scope: string = null;
+  public version: string = null;
+  public description: string = null;
+  public keywords: string[] = null;
+  public date: string;
+  public license: string;
+  public links: {
     npm: string;
     homepage: string;
     repository: string;
     bugs: string;
-  } = null;
+  };
-  author: {
+  public author: {
     name: 'Lossless GmbH';
-  } = null;
+  };
-  publisher: {
+  public publisher: {
     username: 'gitzone';
     email: 'npm@git.zone';
-  } = null;
+  };
-  maintainers: any = null;
+  public maintainers: any = null;
-  score: {
+  public score: {
     final: number;
     detail: {
       quality: number;
@@ -29,13 +37,5 @@ export class NpmPackage {
       maintenance: number;
     };
   } = null;
-  searchScore: number = null;
+  public searchScore: number = null;
-
-  constructor(descriptionArg) {
-    for (let key in descriptionArg) {
-      if (this[key] === null) {
-        this[key] = descriptionArg[key];
-      }
-    }
-  }
 }

ts/smartnpm.classes.npmregistry.ts

@@ -25,12 +25,13 @@ export class NpmRegistry {
     };
   }
 
-  public async getPackageInfo(packageName: string) {
+  public async getPackageInfo(packageName: string): Promise<NpmPackage> {
-    const result = await plugins.packageJson(packageName, {
+    const fullMetadata = await plugins.packageJson(packageName, {
       registryUrl: this.options.npmRegistryUrl,
       fullMetadata: true
     });
-    return result;
+    const npmPackage = await NpmPackage.createFromFullMetadata(fullMetadata);
+    return npmPackage;
   }
 
   public async search(searchObjectArg: ISearchObject) {
@@ -121,9 +122,9 @@ export class NpmRegistry {
       return packageArray;
     }
 
-    for (const packageArg of body.results) {
+    for (const packageSearchInfoArg of body.results) {
-      const localPackage = new NpmPackage(packageArg.package);
+      const npmPackage = await this.getPackageInfo(packageSearchInfoArg.package.name);
-      packageArray.push(localPackage);
+      packageArray.push(npmPackage);
     }
 
     return packageArray;