Added unit tests for packageModules

lib/packageModules.js

@@ -75,7 +75,6 @@ function zip(directory, name) {
 }
 
 module.exports = {
-
   packageModules(stats) {
     return BbPromise.mapSeries(stats.stats, (compileStats, index) => {
       const entryFunction = _.get(this.entryFunctions, index, {});
@@ -94,7 +93,7 @@ module.exports = {
       });
     })
     .then(artifacts => {
-      if (!_.get(this.serverless, 'service.package.individually')) {
+      if (!_.get(this.serverless, 'service.package.individually') && !_.isEmpty(artifacts)) {
         // Set the service artifact to all functions
         const allFunctionNames = this.serverless.service.getAllFunctions();
         _.forEach(allFunctionNames, funcName => {
@@ -106,5 +105,4 @@ module.exports = {
       return null;
     });
   }
-
 };

tests/all.js

@@ -3,6 +3,7 @@
 describe('serverless-webpack', () => {
   require('./validate.test');
   require('./compile.test');
+  require('./packageModules.test');
   require('./run.test');
   require('./cleanup.test');
 });

tests/mocks/archiver.mock.js

@@ -0,0 +1,23 @@
+'use strict';
+
+/**
+ * Mock object for glob
+ */
+
+const sinon = require('sinon');
+
+const ZipMock = sandbox => ({
+  pipe: sandbox.stub(),
+  append: sandbox.stub(),
+  finalize: sandbox.stub(),
+  on: sandbox.stub()
+});
+
+module.exports.create = sandbox => {
+  const zipMock = ZipMock(sandbox);
+  const mock = {
+    create: sinon.stub().returns(zipMock),
+    _zipMock: zipMock
+  };
+  return mock;
+};

tests/mocks/fs.mock.js

@@ -0,0 +1,32 @@
+'use strict';
+
+/**
+ * Mock object for fs
+ */
+
+// We need sinon to create persistent stubs that are not cleared with the sandbox
+const sinon = require('sinon');
+
+const StreamMock = sandbox => ({
+  on: sandbox.stub()
+});
+
+const StatMock = sandbox => ({
+  isDirectory: sandbox.stub()
+});
+
+module.exports.create = sandbox => {
+  const streamMock = StreamMock(sandbox);
+  const statMock = StatMock(sandbox);
+  const fsMock = {
+    createWriteStream: sinon.stub().returns(streamMock),  // Persistent stub
+    readFileSync: sandbox.stub(),
+    statSync: sinon.stub().returns(statMock),  // Persistent stub
+    writeFileSync: sandbox.stub(),
+
+    _streamMock: streamMock,
+    _statMock: statMock
+  };
+
+  return fsMock;
+};

tests/mocks/glob.mock.js

@@ -0,0 +1,13 @@
+'use strict';
+
+/**
+ * Mock object for glob
+ */
+
+const GlobMock = sandbox => ({
+  sync: sandbox.stub()
+});
+
+module.exports.create = sandbox => {
+  return GlobMock(sandbox);
+};