Fix for loopbackio#145

Author: Raymond Feng

lib/postgresql.js

@@ -68,7 +68,14 @@ function PostgreSQL(postgresql, settings) {
   // this._models = {};
   // this.settings = settings;
   this.constructor.super_.call(this, 'postgresql', settings);
-  this.clientConfig = settings.url || settings;
+  if (settings.url) {
+    // pg-pool doesn't handle string config correctly
+    this.clientConfig = {
+      connectionString: settings.url
+    }
+  } else {
+    this.clientConfig = settings;
+  }
   this.pg = new postgresql.Pool(this.clientConfig);
   this.settings = settings;
   if (settings.debug) {

test/init.js

@@ -17,9 +17,17 @@ if (process.env.CI) {
   };
 }
 
-global.getDataSource = global.getSchema = function () {
+var url = 'postgres://' + config.username || config.user + ':' +
+  config.password + '@' + config.host || config.hostname + ':' +
+  config.port + '/' + config.database;
+
+global.getDataSource = global.getSchema = function(useUrl) {
+  var settings = config;
+  if (useUrl) {
+    settings = {url: url};
+  }
   var db = new DataSource(require('../'), config);
-  db.log = function (a) {
+  db.log = function(a) {
     // console.log(a);
   };
   return db;

test/postgresql.transaction.test.js

@@ -13,7 +13,7 @@ var db, Post;
 describe('transactions', function() {
 
   before(function(done) {
-    db = getDataSource();
+    db = getDataSource(true);
     Post = db.define('PostTX', {
       title: {type: String, length: 255, index: true},
       content: {type: String}