refactor: Adapt inject to modern logs

Author: pgrzesik

lib/inject.js

@@ -92,10 +92,19 @@ function injectAllRequirements(funcArtifact) {
     return BbPromise.resolve();
   }
 
-  this.serverless.cli.log('Injecting required Python packages to package...');
+  let injectProgress;
+  if (this.progress) {
+    injectProgress = this.progress.get('python-inject-requirements');
+    injectProgress.update('Injecting required Python packages to package', {
+      isMainEvent: true,
+    });
+  } else {
+    this.serverless.cli.log('Injecting required Python packages to package...');
+  }
 
+  let returnPromise;
   if (this.serverless.service.package.individually) {
-    return BbPromise.resolve(this.targetFuncs)
+    returnPromise = BbPromise.resolve(this.targetFuncs)
       .filter((func) =>
         (func.runtime || this.serverless.service.provider.runtime).match(
           /^python.*/
@@ -132,12 +141,19 @@ function injectAllRequirements(funcArtifact) {
             );
       });
   } else if (!this.options.zip) {
-    return injectRequirements(
+    returnPromise = injectRequirements(
       path.join('.serverless', 'requirements'),
       this.serverless.service.package.artifact || funcArtifact,
       this.options
     );
   }
+
+  return returnPromise
+    .then(() => injectProgress && injectProgress.remove())
+    .catch((e) => {
+      injectProgress && injectProgress.remove();
+      throw e;
+    });
 }
 
 module.exports = { injectAllRequirements };