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;
+      });
   }
 
   /**
@@ -36,15 +41,17 @@ class StorageRef extends ReferenceBase {
     listeners.push(this.storage._addListener('download_resumed', callback));
 
     const path = this.pathToString();
+    this.log.debug('download(', path, ') -> ', downloadPath);
     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(err => {
-        console.log('Got an error ->', err);
-      })
+        this.log.error('Error downloading ', path, ' to ', downloadPath, '.  Error: ', err);
+        throw err;
+      });
   }
 }
 
@@ -79,12 +86,13 @@ export default class Storage extends Base {
    * @return {Promise}
    */
   uploadFile(name: string, filepath: string, metadata: Object={}, cb: (evt: Object) => Object): Promise<Object> {
+    this.log.debug('uploadFile(', filepath, ') -> ', name);
     let callback = cb;
     if (!callback || typeof callback !== 'function') {
       callback = (evt: Object) => ({});
     }
 
-    filepath = filepath.replace("file://", "");
+    filepath = filepath.replace('file://', '');
 
     const listeners = [];
     listeners.push(this._addListener('upload_progress', callback));
@@ -94,6 +102,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;
       });
   }