Repository: serverless-heaven/serverless-webpack (js)

Issue:
Set webpack target property to "node" in case it is missing in the config
<!--
1. Please check if an issue already exists so there are no duplicates
2. Fill out the whole template so we have a good overview on the issue
3. Do not remove any section of the template. If something is not applicable leave it empty but leave it in the Issue
4. Please follow the template, otherwise we'll have to ask you to update it
-->

# This is a Feature Proposal

## Description

It turned out in #274 , that, when the `target: "node"` property is missing in the webpack configuration, the compiles mostly work, but are missing the environment variables.
For the user this is not obvious, but can be checked by the plugin easily. It would be a great UX improvement, if the plugin could warn about the missing setting and set it to "node" automatically if absent.

Similar or dependent issue(s):
* #274 

## Additional Data

* ***Serverless-Webpack Version you're using***:
* ***Webpack version you're using***:
* ***Serverless Framework Version you're using***:
* ***Operating System***:
* ***Stack Trace (if available)***:


Pull request:
Set target to 'node' if not specified by the configuration.
<!--
1. Do not remove any section of the template. If something is not applicable leave it empty but leave it in the PR
2. Please follow the template, otherwise we'll have to ask you to update it and it will take longer until your PR is merged
-->

## What did you implement:

Closes #276 

<!--
Briefly describe the feature if no issue exists for this PR. If possible only
submit PRs for existing issues. If the PR is trivial (like doc changes or simple
code fixes) it can be submitted without a related issue, but as soon as it adds
or changes functionality, a related issue should be present.
-->

## How did you implement it:

<!--
If this is a nontrivial change please briefly describe your implementation so its easy for us to understand and review your code.
-->

Trivial change: Set target to 'node' in case it is not set in the configuration.
Added unit tests.

## How can we verify it:

<!--
Add any applicable config, commands, screenshots or other resources
to make it easy for us to verify this works. The easier you make it for us
to review a PR, the faster we can review and merge it.

Examples:
* Step by step description, how to verify
* Screenshots - Showing the difference between your output and the master
* Other - Anything else that comes to mind to help us evaluate
-->

Supply a webpack configuration without the target property. It should be set to 'node' automatically and a Serverless service should deploy and run correctly (especially the environment variables which break if the target is not set in the webpack compiler).

## Todos:

- [x] Write tests
- ~~[ ] Write documentation~~ not relevant
- [x] Fix linting errors
- [x] Make sure code coverage hasn't dropped
- [x] Provide verification config / commands / resources
- [x] Enable "Allow edits from maintainers" for this PR
- [x] Update the messages below

***Is this ready for review?:*** YES
***Is it a breaking change?:*** NO


Patch:
diff --git a/lib/validate.js b/lib/validate.js
index c2e501f..f3a10d0 100644
--- a/lib/validate.js
+++ b/lib/validate.js
@@ -59,7 +59,7 @@ module.exports = {
 
     const getEntryForFunction = (name, serverlessFunction) => {
       const handler = serverlessFunction.handler;
-      
+
       const handlerFile = getHandlerFile(handler);
       if (!handlerFile) {
         _.get(this.serverless, 'service.provider.name') !== 'google' &&
@@ -119,6 +119,11 @@ module.exports = {
       this.webpackConfig.context = this.serverless.config.servicePath;
     }
 
+    // Default target
+    if (!this.webpackConfig.target) {
+      this.webpackConfig.target = 'node';
+    }
+
     // Default output
     if (!this.webpackConfig.output || _.isEmpty(this.webpackConfig.output)) {
       const outputPath = path.join(this.serverless.config.servicePath, '.webpack');


Test patch:
diff --git a/tests/validate.test.js b/tests/validate.test.js
index de2ea28..f660317 100644
--- a/tests/validate.test.js
+++ b/tests/validate.test.js
@@ -122,6 +122,35 @@ describe('validate', () => {
       .then(() => expect(module.webpackConfig.context).to.equal(testServicePath));
   });
 
+  describe('default target', () => {
+    it('should set a default `webpackConfig.target` if not present', () => {
+      const testConfig = {
+        entry: 'test',
+        output: {},
+      };
+      const testServicePath = 'testpath';
+      module.serverless.config.servicePath = testServicePath;
+      module.serverless.service.custom.webpack = testConfig;
+      return module
+      .validate()
+      .then(() => expect(module.webpackConfig.target).to.equal('node'));
+    });
+
+    it('should not change `webpackConfig.target` if one is present', () => {
+      const testConfig = {
+        entry: 'test',
+        target: 'myTarget',
+        output: {},
+      };
+      const testServicePath = 'testpath';
+      module.serverless.config.servicePath = testServicePath;
+      module.serverless.service.custom.webpack = testConfig;
+      return module
+      .validate()
+      .then(() => expect(module.webpackConfig.target).to.equal('myTarget'));
+    });
+  });
+
   describe('default output', () => {
     it('should set a default `webpackConfig.output` if not present', () => {
       const testEntry = 'testentry';