Makes checkOnConnect to accept 'false', 'no' or '0'. Tests added.

Author: thiagodp

README.md

@@ -41,12 +41,12 @@ Options can be passed as arguments to the database connection string, in URL-for
 - `charset`: defines the charset (encoding) used to handle the JSON file
   - Defaults to `utf-8`
   - Example: `const connection = new Database( 'json:///test.json?charset=utf-16' );`
-  - Available in database-json-json version `1.0.0` or later
+  - Available in database-js-json version `1.0.0` or later
 
 - `checkOnConnect`: whether it should check if the file exists when connecting to it
   - Defaults to `true`
   - Example: `const connection = new Database( 'json:///test.json?checkOnConnect=0' );`
-  - Available in database-json-json version `1.1.0` or later  
+  - Available in database-js-json version `1.1.0` or later  
 
 
 ## Additional Options

index.js

@@ -173,6 +173,12 @@ module.exports = {
                     }
                     options[ key ][ subKey ] = val;
                 }
+
+                // Adjust 'checkOnConnect'
+                if ( options[ 'checkOnConnect' ] ) {
+                    var isFalse = function ( v ) { return 'false' === v || 'no' === v || '0' == v; };
+                    options[ 'checkOnConnect' ] = ! isFalse( val );
+                }
             });
         }
 

package-lock.json

@@ -4,7 +4,7 @@
   "description": "Database-js driver for JSON files",
   "main": "index.js",
   "scripts": {
-    "test": "echo \"Error: no test specified\" && exit 1"
+    "test": "mocha"
   },
   "keywords": [
     "database-js",
@@ -14,5 +14,10 @@
   "license": "MIT",
   "dependencies": {
     "jl-sql-api": "^2.8.1"
+  },
+  "devDependencies": {
+    "jl-sql-api": "^2.8.1",
+    "database-js": "^3.0.1",
+    "mocha": "^5.0.0"
   }
 }

package.json

@@ -0,0 +1,5 @@
+[
+    { "state": "Ohio" },
+    { "state": "Dakota" },
+    { "state": "New York" }
+]

test/states.json

@@ -0,0 +1,40 @@
+var dbjs = require( 'database-js' );
+var assert = require( 'assert' );
+
+describe( 'database-js-json', function () {
+
+    var obj = {
+        driverName: '../../../index', // hack to load the index as the json module of database-js
+        database: './test/states.json'
+    };
+
+    it( 'queries an existing file correctly', function( done ) {
+        var conn = new dbjs.Connection( obj );
+        var st = conn.prepareStatement( 'SELECT * WHERE state = ?' );
+        var search = 'Dakota';
+        st.query( search )
+            .then( function( data ) {
+                assert.equal( data[ 0 ].state, search );
+                done();
+            } )
+            .catch( function( err ) {
+                done( err );
+            } );
+    } );
+
+    it( 'throws when an inexisting file is required', function() {
+        var obj2 = Object.assign( obj, { database: 'not-found.json' } );
+        assert.throws(
+            function() {
+                new dbjs.Connection( obj2 );
+            },
+            /not exist/
+        );
+    } );
+
+    it( 'does not throw when an inexisting file is not required', function() {
+        var obj2 = Object.assign( obj, { database: 'not-found.json', parameters: 'checkOnConnect=false' } );
+        new dbjs.Connection( obj2 );
+    } );    
+
+} );