test: fix clearing the config require cache

snyk · Aug 26, 2016 · bd197af · bd197af
1 parent df0c869
commit bd197af
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 7 deletions.
diff --git a/lib/index.js b/lib/index.js
@@ -10,7 +10,7 @@ const app = module.exports = {
 function main({ port, client } = {}) {
   // note: the config is loaded in the main function to allow us to mock in tests
   if (process.env.TAP) {
-    delete require.cache[require.resolve('./config.js')];
+    delete require.cache[require.resolve('./config')];
   }
   const config = require('./config');
   if (client === undefined) {

diff --git a/test/functional/no-filter.test.js b/test/functional/no-filter.test.js
@@ -27,8 +27,6 @@ test('no filters broker', t => {
   process.env.PORT = localPort;
   process.env.BROKER_URL = `http://localhost:${serverPort}`;
   process.env.BROKER_ID = '12345';
-  // invalidate the config require
-  delete require.cache[require.resolve(__dirname + '/../../lib/config.js')];
   const client = app.main({ port: port() });
 
   // wait for the client to successfully connect to the server and identify itself

diff --git a/test/utils.js b/test/utils.js
@@ -34,9 +34,6 @@ module.exports = (tap) => {
   return {
     localPort,
     port,
-    server,
-    resetConfig: () => {
-      delete require.cache[require.resolve(__dirname + '/../lib/config.js')];
-    }
+    server
   };
 };