Merge pull request #32 from JuanjoArreola/master

Add connect with params test, simpler connect string creation

Sources/PostgreSQL/Connection.swift

@@ -16,17 +16,11 @@ public final class Connection: ConnInfoInitializable {
         let string: String
 
         switch conninfo {
-        case .raw(let ci):
-            string = ci
+        case .raw(let info):
+            string = info
         case .params(let params):
-            var ci = ""
-
-            params.forEach { (key, value) in
-                ci += "\(key)='\(value)'"
-            }
-
-            string = ci
-        case .basic(host: let host, port: let port, database: let database, user: let user, password: let password):
+            string = params.map({ "\($0)='\($1)'" }).joined()
+        case .basic(let host, let port, let database, let user, let password):
             string = "host='\(host)' port='\(port)' dbname='\(database)' user='\(user)' password='\(password)' client_encoding='UTF8'"
         }
 

Tests/PostgreSQLTests/ConnectionTests.swift

@@ -0,0 +1,18 @@
+import XCTest
+import PostgreSQL
+
+class ConnectionTests: XCTestCase {
+
+    func testConnInfoParams() {
+        do {
+            let postgreSQL = try PostgreSQL.Database(params: ["host": "127.0.0.1",
+                                                              "port": "5432",
+                                                              "dbname": "test",
+                                                              "user": "postgres",
+                                                              "password": ""])
+            try postgreSQL.execute("SELECT version()")
+        } catch {
+            XCTFail("Could not connect to database")
+        }
+    }
+}