cjolowicz / muckr-web / 26

100%

push

travis-ci-com

Add test for constants

This test module serves mainly to provide coverage for the ternary
operator in the constants module, which depends on an environment
variable.

Some implementation notes:

 - We use `require`, because we need to import the module after
   setting the process environment. Appease eslint by disabling the
   `global-require` rule.

 - `process.env` needs to be set in the `it` block, not in the
   `describe` block, to avoid a race with `require`.

 - We need to reset the module cache before each `require`. This can
   be done using `jest.resetModules` instead of a manual `delete
   require.cache[require.resolve(module)]`.

 - We reset `process.env` after each test, to avoid leakage into other
   tests.

Closes: #9

23 of 23 branches covered (100.0%)

Branch coverage included in aggregate %.

70 of 70 relevant lines covered (100.0%)

7.67 hits per line