Replaced deprecated fs.existsSync() call by fs.accessSync();

Updated dependencies;
Updated unit test to match latest `shouldjs` API;

History.md

@@ -1,3 +1,9 @@
+# 1.0.0 / 2016-06-17
+
+* Replaced deprecated `fs.existsSync()` call by `fs.accessSync()`. `fs.existsSync()` was deprecated since Node.js v1.0.0.
+* Updated `mocha` and `should` dependencies
+* Proven stable enough to be finally called version 1.0.0
+
 # 0.1.5 / 2016-04-09
 
 * Updated to latest package versions

index.js

@@ -25,7 +25,7 @@ var LOG = process.env.NODE_CONFIG_LOG !== undefined;
  * SYNCHRONOUSLY fetches all environments available. Generally, returns the list of sub-directories under the CONFIG_DIR.
  * @returns {Array} List of environments available or an empty array in case there is no environment sub-directory
  */
-function getEnvs() {
+function    getEnvs() {
     if (LOG)
         console.info('Reading environments available in', path.resolve(CONFIG_DIR));
 
@@ -84,9 +84,14 @@ function getConfigs(env) {
 
 
 // check for non-existence of configuration directory
-if (!fs.existsSync(CONFIG_DIR)) {
-    if (LOG || HALT) // always print the error if halting the app
-        console.error('Configuration directory for the app is missing. Expected location is', CONFIG_DIR);
+try {
+    fs.accessSync(CONFIG_DIR, fs.R_OK);
+}
+catch (e) {
+    if (LOG || HALT) { // always print the error if halting the app
+        console.error('Configuration directory for the app is either missing or non-readable. Expected location is', CONFIG_DIR);
+        console.error(e);
+    }
     if (HALT) {
         console.error('Halting the app.');
         process.exit(-1);

package.json

@@ -1,6 +1,6 @@
 {
     "name": "app-config",
-    "version": "0.1.5",
+    "version": "1.0.0",
     "description": "Simple utility for Node.js to load configuration files depending on your environment",
     "main": "./index.js",
     "scripts": {
@@ -22,10 +22,13 @@
     "bugs": {
         "url": "https://github.com/lobodpav/node-app-config/issues?state=open"
     },
+    "engines" : {
+        "node" : ">=6.1.0"
+    },
     "readmeFilename": "README.md",
     "license": "MIT",
     "devDependencies": {
-        "mocha": "~2.4.5",
-        "should": "~8.3.0"
+        "mocha": "~2.5.3",
+        "should": "~9.0.2"
     }
 }

test/test.js

@@ -29,14 +29,14 @@ describe(modName, function() {
             process.env.NODE_ENV = 'blah';
 
             var config = require(modFile);
-            should(config).equal(null);
+            should(config).be.equal(null);
         });
 
         it('should fail to load configs in non-existing directory', function() {
             process.env.NODE_CONFIG_DIR = 'blah';
 
             var config = require(modFile);
-            should(config).equal(null);
+            should(config).be.equal(null);
         });
 
         it('should fail to load non-existing config for DEV environment in default directory', function() {
@@ -52,7 +52,7 @@ describe(modName, function() {
             process.env.NODE_CONFIG_DIR = processDir + '/config1';
 
             var config = require(modFile);
-            should(config).equal(null);
+            should(config).be.equal(null);
         });
 
         it('should halt the app on failure', function() {
@@ -70,12 +70,12 @@ describe(modName, function() {
             var warn = console.warn;
 
             // mock the process.exit() function to catch its call
-            process.exit = function(code) {
+            process.exit = function() {
                 halted = true;
-            }
+            };
 
             // mocks to suppress logging attempts when HATL is being enabled
-            console.error = console.info = console.warn = function() {}
+            console.error = console.info = console.warn = function() {};
 
 
             require(modFile);
@@ -87,15 +87,15 @@ describe(modName, function() {
             console.info = info;
             console.warn = warn;
 
-            halted.should.be.true;
+            halted.should.be.true();
         });
 
         it('should not halt the app on failure', function() {
             // set the environment to non-existing one to cause an error
             process.env.NODE_ENV = 'blah';
 
             var config = require(modFile);
-            should(config).equal(null);
+            should(config).be.equal(null);
         });
     });
 
@@ -107,8 +107,8 @@ describe(modName, function() {
 
             config = config.db;
             should.exist(config);
-            config.should.be.an.Object;
-            config.should.have.a.property('dbURI', 'mongodb://localhost:27017/dev-db');
+            config.should.be.an.Object();
+            config.should.have.property('dbURI', 'mongodb://localhost:27017/dev-db');
         });
 
         it('should load LOG config for PROD environment in default directory', function() {
@@ -118,7 +118,7 @@ describe(modName, function() {
 
             config = config.log;
             should.exist(config);
-            config.should.be.an.Object;
+            config.should.be.an.Object();
             config.should.have.properties('fileLogConfig', 'consoleLogConfig');
             config.fileLogConfig.should.have.property('filename', '/var/log/server-prod.log');
             config.consoleLogConfig.should.have.property('level', 'error');
@@ -132,8 +132,8 @@ describe(modName, function() {
 
             config = config.db;
             should.exist(config);
-            config.should.be.an.Object;
-            config.should.have.a.property('dbURI', 'mongodb://localhost:27017/dev-db-config1');
+            config.should.be.an.Object();
+            config.should.have.property('dbURI', 'mongodb://localhost:27017/dev-db-config1');
         });
 
         it('should print out log information', function() {
@@ -155,7 +155,7 @@ describe(modName, function() {
             console.info = info;
             console.warn = warn;
 
-            log.should.be.true;
+            log.should.be.true();
         });
 
         it('should not print out log information', function() {
@@ -173,7 +173,7 @@ describe(modName, function() {
             console.error = err;
             console.info = info;
 
-            log.should.be.false;
+            log.should.be.false();
         });
 
         it('should reload config files when `app-config` is deleted from `require.cache`', function() {