Merge pull request #16 from servicetitan/master

fix: remove tarballs on package removal

Author: favoyang

src/PackageStorage.ts

@@ -42,11 +42,17 @@ export default class StoragePluginManager implements ILocalPackageManager {
   }
 
   public deletePackage(fileName: string, callback: CallbackAction): void {
+    if (fileName.toLowerCase().endsWith('.tgz')) {
+      return this.tarballPackageStorage.deletePackage(fileName, callback);
+    }
+
     return this.packumentPackageStorage.deletePackage(fileName, callback);
   }
 
   public removePackage(callback: CallbackAction): void {
-    return this.packumentPackageStorage.removePackage(callback);
+    return this.packumentPackageStorage.removePackage(() => {
+      this.tarballPackageStorage.removePackage(callback);
+    });
   }
 
   public createPackage(name: string, data: Package, callback: CallbackAction): void {

tests/plugin.test.ts

@@ -149,7 +149,7 @@ describe('verdaccio-storage-proxy', () => {
     test('removePackage', () => {
       const cb = jest.fn();
       packageStorage.removePackage(cb);
-      expect(packageStorage.packumentPackageStorage.removePackage).toHaveBeenCalledWith(cb);
+      expect(packageStorage.packumentPackageStorage.removePackage).toHaveBeenCalled();
     });
     test('deletePackage', () => {
       const cb = jest.fn();