Merge pull request #98 from Fiery-Fenix/master

Fix parameters interpolation with empty params

conn.go

@@ -286,7 +286,7 @@ func (c *conn) buildRequest(ctx context.Context, query string, params []driver.V
 		method string
 		err    error
 	)
-	if params != nil {
+	if params != nil && len(params) > 0 {
 		if query, err = interpolateParams(query, params); err != nil {
 			return nil, err
 		}

conn_test.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"database/sql"
 	"database/sql/driver"
+	"io/ioutil"
 	"net/http"
 	"testing"
 	"time"
@@ -367,6 +368,18 @@ func (s *connSuite) TestBuildRequestWithQueryIdAndQuotaKey() {
 		}
 	}
 }
+func (s *connSuite) TestBuildRequestParamsInterpolation() {
+	query := `INSERT INTO test (str) VALUES ("Question?")`
+	cn := newConn(NewConfig())
+	req, err := cn.buildRequest(context.Background(), query, make([]driver.Value, 0), false)
+	if s.NoError(err) {
+		body, e := ioutil.ReadAll(req.Body)
+		if s.NoError(e) {
+			s.Equal(query, string(body))
+		}
+	}
+}
+
 func TestConn(t *testing.T) {
 	suite.Run(t, new(connSuite))
 }