feat: load environment variables in local process.env

Author: CorentinDoue

invokeLocal/googleInvokeLocal.js

@@ -22,7 +22,6 @@ class GoogleInvokeLocal {
         await this.validate();
         await this.setDefaults();
         await this.getDataAndContext();
-        // TODO load env var in process.env
       },
       'invoke:local:invoke': async () => this.invokeLocal(),
     };

invokeLocal/googleInvokeLocal.test.js

@@ -45,6 +45,7 @@ describe('GoogleInvokeLocal', () => {
       ${'invokeLocalNodeJs'}
       ${'loadFileInOption'}
       ${'validateEventsProperty'}
+      ${'addEnvironmentVariablesToProcessEnv'}
     `('should declare $method method', ({ method }) => {
       expect(googleInvokeLocal[method]).toBeDefined();
     });

invokeLocal/lib/nodeJs.js

@@ -2,6 +2,7 @@
 
 const chalk = require('chalk');
 const path = require('path');
+const _ = require('lodash');
 
 const tryToRequirePaths = (paths) => {
   let loaded;
@@ -38,6 +39,8 @@ module.exports = {
       throw new Error(`Failed to load function "${functionObj.handler}" from the loaded file`);
     }
 
+    this.addEnvironmentVariablesToProcessEnv(functionObj);
+
     function handleError(err) {
       let errorResult;
       if (err instanceof Error) {
@@ -91,4 +94,9 @@ module.exports = {
       return maybeThennable;
     });
   },
+
+  addEnvironmentVariablesToProcessEnv(functionObj) {
+    const environmentVariables = this.provider.getConfiguredEnvironment(functionObj);
+    _.merge(process.env, environmentVariables);
+  },
 };

invokeLocal/lib/nodeJs.test.js

@@ -15,12 +15,16 @@ describe('invokeLocalNodeJs', () => {
   const context = {
     name: contextName,
   };
+  const myVarValue = 'MY_VAR_VALUE';
   let serverless;
   let googleInvokeLocal;
 
   beforeEach(() => {
     serverless = new Serverless();
     serverless.setProvider('google', new GoogleProvider(serverless));
+    serverless.service.provider.environment = {
+      MY_VAR: myVarValue,
+    };
     serverless.serviceDir = path.join(process.cwd(), 'invokeLocal', 'lib', 'testMocks'); // To load the index.js of the mock folder
     serverless.cli.consoleLog = jest.fn();
     googleInvokeLocal = new GoogleInvokeLocal(serverless, {});
@@ -69,4 +73,13 @@ describe('invokeLocalNodeJs', () => {
       expect.stringContaining('"errorMessage": "ASYNC_ERROR"')
     );
   });
+
+  it('should give the environment variables to the handler', async () => {
+    const functionConfig = {
+      handler: 'envHandler',
+    };
+    await googleInvokeLocal.invokeLocalNodeJs(functionConfig, event, context);
+    // eslint-disable-next-line no-console
+    expect(console.log).toHaveBeenCalledWith(myVarValue);
+  });
 });

invokeLocal/lib/testMocks/index.js

@@ -25,4 +25,8 @@ module.exports = {
     console.log('ASYNC_HANDLER');
     throw new Error('ASYNC_ERROR');
   },
+  envHandler: async () => {
+    // eslint-disable-next-line no-console
+    console.log(process.env.MY_VAR);
+  },
 };

test/serverless.js

@@ -19,6 +19,7 @@ class Serverless {
       }
       return this.functions[functionName];
     };
+    this.service.provider = {};
     this.utils = {
       writeFileSync() {},
       readFileSync() {},