serverless-nodejs-starter icon indicating copy to clipboard operation
serverless-nodejs-starter copied to clipboard

Tests do not load .env or serverless.yml variables

Open Steffan-Carlos opened this issue 6 years ago • 8 comments

On newest release when running npm test which runs serverless-bundle test environment variables from serverless.yml

provider:
  name: aws
  runtime: nodejs10.x
  stage: dev
  region: us-east-1
  environment:
    TEST: "value"

or from .env file

TEST=value

This issue was noticed in the serverless-bundle package and noted by @Vadorequest https://github.com/AnomalyInnovations/serverless-bundle/issues/4

Using his solution I was able to load .env variables by adding the following to the package.json

"jest": {
    "setupFilesAfterEnv": [
      "./jest-preload-env.js"
    ]
  }

jest-preload-env.js

require('dotenv').config({
    path: './.env'
});

if (process.env.NODE_ENV !== 'test') {
    throw Error('Non-test environment');
}

severless-bundle test should load serverless.yml environment variables

Steffan-Carlos avatar Oct 26 '19 01:10 Steffan-Carlos

Oh good catch. Yeah I'll have a look at how to do this in the serverless-bundle plugin.

jayair avatar Oct 26 '19 02:10 jayair

@jayair Any update on this issue?

Steffan-Carlos avatar Nov 22 '19 15:11 Steffan-Carlos

Yeah I just pushed a beta version. Give it a try!

jayair avatar Jan 02 '20 21:01 jayair

Environment variables are working for me in tests, but the sample docs are a bit confusing. In the sample serverless.yml file, you have:

  #environment:
  #  sampleEnvVar: ${env:SAMPLE_ENV_VAR}

with the following in the env.example file:

SAMPLE_ENV_VAR=i-am-an-environment-variable

This leads to a mismatched environment between the code and the tests. In the code, the given environment variable is accessed under process.env.sampleEnvVar. However, the tests seem to be reading directly from the .env file. So, it seems the tests are expecting this variable under process.env.SAMPLE_ENV_VAR. To get it to work correctly, I had to change the format of my environment variables to make them consistent.

To avoid this confusion, I think the samples should be changed to be consistent also. The sample variable should either be sampleEnvVar or SAMPLE_ENV_VAR in both the .env file and the serverless.yml file.

Hope this makes sense... :)

jeff-kilbride avatar Feb 18 '20 06:02 jeff-kilbride

@jeff-kilbride Good catch, let me update the sample in the comment.

jayair avatar Feb 23 '20 03:02 jayair

looks good to me

Steffan-Carlos avatar Apr 17 '20 16:04 Steffan-Carlos

I'm coming across an issue using the same three serverless plugins as serverless-nodejs-starter. My project uses env vars defined in the serverless.yml based on custom serverless variables, like so:

# serverless.yml

provider:
  environment:
    USERS_TABLE_NAME: ${self:custom.usersTableName}

custom:
  stage: ${opt:stage, self:provider.stage}
  usersTableName: ${self:custom.stage}-myproject-users
// handler that is being tested

dynamoDb.put({
  TableName: process.env.USERS_TABLE_NAME,
  Item: {... }
})

Ideally, 'serverless-bundle test' would use env vars defined in serverless.yml instead of the .env file itself. Thanks for any ideas - could be missing something basic here.

kevinkoste avatar Jun 04 '20 23:06 kevinkoste

Hmm it doesn't right now. That's because the .env file is treated as the environment variables of the execution context, while the ones in the serverless.yml are the ones that are set in the Lambda function when it's deployed.

It's actually a little tricky to do this because serverelss-bundle test is running the tests agains the bundled code and not against what Serverless Framework creates.

jayair avatar Jun 13 '20 23:06 jayair