- Fixed a bug where setting pool or debug in the connection string p…

…assed to ORM.connect would always be set to true.

 - Wrote a test to demonstrate the passing.

lib/ORM.js

@@ -112,8 +112,8 @@ exports.connect = function (opts, cb) {
 		var debug    = extractOption(opts, "debug");
 		var pool     = extractOption(opts, "pool");
 		var driver   = new Driver(opts, null, {
-			debug    : (debug !== null ? Boolean(debug) : settings.get("connection.debug")),
-			pool     : (pool !== null ? Boolean(pool) : settings.get("connection.pool")),
+			debug    : (debug !== null ? ((debug === "false" || debug === "0") ? false : true) : settings.get("connection.debug")),
+			pool     : (pool !== null ? ((pool === "false" || pool === "0") ? false : true) : settings.get("connection.pool")),
 			settings : settings
 		});
 

test/integration/orm-exports.js

@@ -190,6 +190,15 @@ describe("ORM.connect()", function () {
 				return done();
 			});
 		});
+
+		it("should allow pool and debug settings to be false", function(done) {
+			var connString = common.getConnectionString() + "debug=false&pool=false";
+			ORM.connect(connString, function(err, db) {
+				db.driver.opts.pool.should.equal(false);
+				db.driver.opts.debug.should.equal(false);
+				done();
+			});
+		});
 	});
 });