Better debugging logs for storage.js

Author: BlooJeans

lib/modules/storage.js

@@ -16,7 +16,12 @@ class StorageRef extends ReferenceBase {
 
   downloadUrl(): Promise<Object> {
     const path = this.pathToString();
-    return promisify('downloadUrl', FirestackStorage)(this.storage.storageUrl, path);
+    this.log.debug('downloadUrl(', path, ')');
+    return promisify('downloadUrl', FirestackStorage)(this.storage.storageUrl, path)
+      .catch(err => {
+        this.log.error('Error downloading URL for ', path, '.  Error: ', err);
+        throw err;
+      });
   }
 
   /**
@@ -27,6 +32,7 @@ class StorageRef extends ReferenceBase {
    */
   download(downloadPath: string, listener: Function = noop): Promise<Object> {
     const path = this.pathToString();
+    this.log.debug('download(', path, ') -> ', downloadPath);
     const listeners = [
       this.storage._addListener('download_progress', listener),
       this.storage._addListener('download_paused', listener),
@@ -35,13 +41,13 @@ class StorageRef extends ReferenceBase {
 
     return promisify('downloadFile', FirestackStorage)(this.storage.storageUrl, path, downloadPath)
       .then((res) => {
-        console.log('res --->', res);
+        this.log.debug('res --->', res);
         listeners.forEach(listener => listener.remove());
         return res;
       })
-      .catch((downloadError) => {
-        console.log('Got an error ->', downloadError);
-        return Promise.reject(downloadError);
+      .catch(err => {
+        this.log.error('Error downloading ', path, ' to ', downloadPath, '.  Error: ', err);
+        throw err;
       });
   }
 }
@@ -79,6 +85,7 @@ export default class Storage extends Base {
    */
   uploadFile(name: string, filePath: string, metadata: Object = {}, listener: Function = noop): Promise<Object> {
     const _filePath = filePath.replace('file://', '');
+    this.log.debug('uploadFile(', _filepath, ') -> ', name);
     const listeners = [
       this._addListener('upload_paused', listener),
       this._addListener('upload_resumed', listener),
@@ -89,6 +96,10 @@ export default class Storage extends Base {
       .then((res) => {
         listeners.forEach(listener => listener.remove());
         return res;
+      })
+      .catch(err => {
+        this.log.error('Error uploading file ', name, ' to ', filepath, '.  Error: ', err);
+        throw err;
       });
   }