From 5b39134501732d2d4c116fd0bccb0ce55c9f920f Mon Sep 17 00:00:00 2001 From: rjz Date: Sat, 3 Oct 2015 14:09:53 -0700 Subject: [PATCH] Deprecates config.envs (closes #12) `process.env` mixins were broken as implemented, and they're an easy addition outside the test helper. We'll discourage their use (in case anyone was somehow using them). --- index.js | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/index.js b/index.js index 36631ce..cca871a 100644 --- a/index.js +++ b/index.js @@ -23,6 +23,15 @@ function serializeCookie (c) { }, []); } +function assignEnvs (envs) { + Object.keys(envs).forEach(function(e) { + process.env[e] = envs[e]; + }); +} + +var deprecatedAssignEnvs = util.deprecate(assignEnvs, + 'supertest-session env configuration is deprecated and will be removed in next version.'); + module.exports = function (config) { if (!config) config = {}; @@ -31,9 +40,7 @@ module.exports = function (config) { this.app = config.app; if (config.envs && (config.envs instanceof Object)) { - Object.keys(config.envs).forEach(function(e) { - process.env[e] = config.envs[e]; - }); + deprecatedAssignEnvs(config.envs); } }