Merge pull request #49 from SecJS/feat/len-load-config-without-ext

feat: load file without extension

Author: jlenon7

README.md

@@ -331,6 +331,10 @@ config.load('example.ts')
 // You can also use safeLoad to not reload files that were already loaded
 config.safeLoad('app.ts') // Will just return without errors, but app.ts will not be reloaded.
 
+// And you can also load an configuration file without extension. 
+// Config class will verify the Env NODE_TS to get the .js or .ts extension file
+config.safeLoad('app')
+
 console.log(Config.get('app.name')) // 'secjs'
 ```
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@secjs/utils",
-  "version": "1.7.5",
+  "version": "1.7.6",
   "description": "",
   "license": "MIT",
   "author": "João Lenon",

src/Classes/Config.ts

@@ -25,7 +25,7 @@ export class Config {
       keys.forEach(key => (config = config[key]))
     }
 
-    if (!config) config = defaultValue
+    if (config === undefined) config = defaultValue
 
     return config
   }
@@ -53,10 +53,13 @@ export class Config {
       throw new InternalServerException(content)
     }
 
-    if (Config.configs.get(name)) return
+    if (Config.configs.has(name)) return
     if (base.includes('.map') || base.includes('.d.ts')) return
 
-    const file = new File(`${dir}/${base}`).loadSync()
+    const fileExtension = process.env.NODE_TS === 'true' ? 'ts' : 'js'
+    const fileBase = `${name}.${fileExtension}`
+
+    const file = new File(`${dir}/${fileBase}`).loadSync()
     const fileContent = file.getContentSync().toString()
 
     if (
@@ -87,6 +90,6 @@ export class Config {
     }
 
     Config.debug.log(`Loading ${name} configuration file`)
-    Config.configs.set(name, require(`${dir}/${name}`).default)
+    Config.configs.set(name, require(file.path).default)
   }
 }

tests/Classes/config.spec.ts

@@ -33,6 +33,14 @@ describe('\n Config Class', () => {
     expect(Config.get('test-ondemand.hello')).toBe(true)
   })
 
+  it('should be able to load configuration file without extension', async () => {
+    const config = new Config()
+
+    config.load(Path.tests('stubs/no-extension'))
+
+    expect(Config.get('no-extension.extension')).toBe(false)
+  })
+
   it('should throw an error when file is trying to use Config.get() to get information from other config file but this config file is trying to use Config.get() to this same file', async () => {
     try {
       new Config().load(Path.tests('stubs/infinite-callA.ts'))

tests/stubs/no-extension.ts

@@ -0,0 +1,12 @@
+/**
+ * @secjs/utils
+ *
+ * (c) João Lenon <lenon@secjs.com.br>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+export default {
+  extension: false,
+}