Force Environment Variables to be Strings

This forces any environment variables to be sent to GCF as strings,
even if the value was supplied as a non-string (e.g., number).

Author: pickypg

package/lib/compileFunctions.js

@@ -46,10 +46,13 @@ module.exports = {
         'nodejs8';
       funcTemplate.properties.timeout =
         _.get(funcObject, 'timeout') || _.get(this, 'serverless.service.provider.timeout') || '60s';
-      funcTemplate.properties.environmentVariables = _.merge(
-        {},
-        _.get(this, 'serverless.service.provider.environment'),
-        funcObject.environment // eslint-disable-line comma-dangle
+      funcTemplate.properties.environmentVariables = _.mapValues(
+        _.merge(
+          {},
+          _.get(this, 'serverless.service.provider.environment'),
+          funcObject.environment // eslint-disable-line comma-dangle
+        ),
+        (value) => value.toString() // eslint-disable-line comma-dangle
       );
 
       if (!funcTemplate.properties.serviceAccountEmail) {

package/lib/compileFunctions.test.js

@@ -376,6 +376,8 @@ describe('CompileFunctions', () => {
           handler: 'func1',
           environment: {
             TEST_VAR: 'test',
+            TEST_VAR_NUMBER: 123,
+            TEST_VAR_BOOL: false,
           },
           events: [{ http: 'foo' }],
         },
@@ -393,6 +395,8 @@ describe('CompileFunctions', () => {
             availableMemoryMb: 256,
             environmentVariables: {
               TEST_VAR: 'test',
+              TEST_VAR_NUMBER: '123',
+              TEST_VAR_BOOL: 'false',
             },
             timeout: '60s',
             sourceArchiveUrl: 'gs://sls-my-service-dev-12345678/some-path/artifact.zip',