Merge pull request serverless#4145 from medikoo/graceful-fs-packaging

Rely on graceful-fs for concurrent file operations (lambda packaging)

lib/plugins/package/lib/zipService.js

@@ -7,14 +7,11 @@ const archiver = require('archiver');
 const os = require('os');
 const path = require('path');
 const crypto = require('crypto');
-const fs = BbPromise.promisifyAll(require('fs'));
+const fs = BbPromise.promisifyAll(require('graceful-fs'));
 const childProcess = BbPromise.promisifyAll(require('child_process'));
 const globby = require('globby');
 const _ = require('lodash');
 
-const fsStat = BbPromise.promisify(fs.stat);
-const fsReadFile = BbPromise.promisify(fs.readFile);
-
 module.exports = {
   zipService(exclude, include, zipFileName) {
     const params = {
@@ -89,7 +86,7 @@ module.exports = {
             filePath
           );
 
-          return fsStat(fullPath).then(stats =>
+          return fs.statAsync(fullPath).then(stats =>
             this.getFileContent(fullPath).then(fileContent =>
               zip.append(fileContent, {
                 name: filePath,
@@ -104,7 +101,7 @@ module.exports = {
   },
 
   getFileContent(fullPath) {
-    return fsReadFile(fullPath, 'utf8');
+    return fs.readFileAsync(fullPath, 'utf8');
   },
 };
 

lib/plugins/package/lib/zipService.test.js

@@ -9,7 +9,7 @@ const JsZip = require('jszip');
 const globby = require('globby');
 const _ = require('lodash');
 const BbPromise = require('bluebird');
-const fs = BbPromise.promisifyAll(require('fs'));
+const fs = BbPromise.promisifyAll(require('graceful-fs'));
 const childProcess = BbPromise.promisifyAll(require('child_process'));
 const sinon = require('sinon');
 const Package = require('../package');

package.json

@@ -100,6 +100,7 @@
     "fs-extra": "^0.26.7",
     "get-stdin": "^5.0.1",
     "globby": "^6.1.0",
+    "graceful-fs": "^4.1.11",
     "graphql": "^0.10.1",
     "graphql-tag": "^2.4.0",
     "https-proxy-agent": "^1.0.0",