cockroachdb: set default port

Author: xiam

Diff for: adapter/cockroachdb/connection_pgx.go

@@ -49,14 +49,14 @@ func (c ConnectionURL) String() (s string) {
 			if host == "" {
 				host = "127.0.0.1"
 			}
-			if port == "" {
-				port = "26257"
-			}
 			u = append(u, "host="+escaper.Replace(host))
-			u = append(u, "port="+escaper.Replace(port))
 		} else {
 			u = append(u, "host="+escaper.Replace(c.Host))
 		}
+		if port == "" {
+			port = "26257"
+		}
+		u = append(u, "port="+escaper.Replace(port))
 	}
 
 	if c.Socket != "" {

Diff for: adapter/cockroachdb/connection_pgx_test.go

@@ -42,15 +42,15 @@ func TestConnectionURL(t *testing.T) {
 
 	// Adding a host.
 	c.Host = "localhost"
-	assert.Equal(t, "host=localhost sslmode=prefer statement_cache_capacity=0", c.String())
+	assert.Equal(t, "host=localhost port=26257 sslmode=prefer statement_cache_capacity=0", c.String())
 
 	// Adding a username.
 	c.User = "Anakin"
-	assert.Equal(t, `host=localhost sslmode=prefer statement_cache_capacity=0 user=Anakin`, c.String())
+	assert.Equal(t, `host=localhost port=26257 sslmode=prefer statement_cache_capacity=0 user=Anakin`, c.String())
 
 	// Adding a password with special characters.
 	c.Password = "Some Sort of ' Password"
-	assert.Equal(t, `host=localhost password=Some\ Sort\ of\ \'\ Password sslmode=prefer statement_cache_capacity=0 user=Anakin`, c.String())
+	assert.Equal(t, `host=localhost password=Some\ Sort\ of\ \'\ Password port=26257 sslmode=prefer statement_cache_capacity=0 user=Anakin`, c.String())
 
 	// Adding a port.
 	c.Host = "localhost:1234"

Diff for: adapter/cockroachdb/connection_pq.go

@@ -49,14 +49,14 @@ func (c ConnectionURL) String() (s string) {
 			if host == "" {
 				host = "127.0.0.1"
 			}
-			if port == "" {
-				port = "26257"
-			}
 			u = append(u, "host="+escaper.Replace(host))
-			u = append(u, "port="+escaper.Replace(port))
 		} else {
 			u = append(u, "host="+escaper.Replace(c.Host))
 		}
+		if port == "" {
+			port = "26257"
+		}
+		u = append(u, "port="+escaper.Replace(port))
 	}
 
 	if c.Socket != "" {

Diff for: adapter/cockroachdb/connection_pq_test.go

@@ -42,15 +42,15 @@ func TestConnectionURL(t *testing.T) {
 
 	// Adding a host.
 	c.Host = "localhost"
-	assert.Equal(t, "host=localhost sslmode=prefer", c.String())
+	assert.Equal(t, "host=localhost port=26257 sslmode=prefer", c.String())
 
 	// Adding a username.
 	c.User = "Anakin"
-	assert.Equal(t, `host=localhost sslmode=prefer user=Anakin`, c.String())
+	assert.Equal(t, `host=localhost port=26257 sslmode=prefer user=Anakin`, c.String())
 
 	// Adding a password with special characters.
 	c.Password = "Some Sort of ' Password"
-	assert.Equal(t, `host=localhost password=Some\ Sort\ of\ \'\ Password sslmode=prefer user=Anakin`, c.String())
+	assert.Equal(t, `host=localhost password=Some\ Sort\ of\ \'\ Password port=26257 sslmode=prefer user=Anakin`, c.String())
 
 	// Adding a port.
 	c.Host = "localhost:1234"