Merge pull request #52 from SecJS/fix/len-reload-config-files

fix(config): set version on config file import to not cache

Author: jlenon7

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@secjs/utils",
-  "version": "1.9.3",
+  "version": "1.9.4",
   "description": "Utils functions and classes for Node.js",
   "license": "MIT",
   "author": "João Lenon <lenon@athenna.io>",

src/Helpers/Config.js

@@ -99,6 +99,9 @@ export class Config {
     }
 
     Debug.log(`Loading ${path} configuration file`, 'api:configurations')
-    Config.configs.set(name, (await import(file.href)).default)
+    Config.configs.set(
+      name,
+      (await import(`${file.href}?version=${Math.random()}${ext}`)).default,
+    )
   }
 }

tests/Stubs/config/app.js

@@ -9,4 +9,5 @@
 
 export default {
   name: 'SecJS',
+  env: process.env.NODE_ENV,
 }

tests/Unit/ConfigTest.js

@@ -8,12 +8,14 @@
  */
 
 import { test } from '@japa/runner'
-import { Path, Config, Folder } from '#src/index'
+import { Config, Folder, Path } from '#src/index'
 import { RecursiveConfigException } from '#src/Exceptions/RecursiveConfigException'
 import { ConfigNotNormalizedException } from '#src/Exceptions/ConfigNotNormalizedException'
 
 test.group('ConfigTest', group => {
   group.each.setup(async () => {
+    process.env.NODE_ENV = 'test'
+
     Config.configs.clear()
 
     const config = new Config()
@@ -59,4 +61,19 @@ test.group('ConfigTest', group => {
 
     assert.equal(Config.get('app.name'), 'SecJS')
   })
+
+  test('should be able to reload configuration values', async ({ assert }) => {
+    assert.equal(Config.get('app.env'), 'test')
+
+    process.env.NODE_ENV = 'example'
+
+    Config.configs.clear()
+
+    const config = new Config()
+
+    await config.safeLoad(Path.config('app.js'))
+    await config.safeLoad(Path.config('database.js'))
+
+    assert.equal(Config.get('app.env'), 'example')
+  })
 })