codegen: Fix errant database/sql imports (#789)

* codegen: Fix errant database/sql imports

Map return type of void to interface{}, not sql.NullBool

* remove debug

* actually test the import fix

Author: kyleconroy

internal/codegen/golang/postgresql_type.go

@@ -129,9 +129,8 @@ func postgresType(r *compiler.Result, col *compiler.Column, settings config.Comb
 		return "sql.NullInt64"
 
 	case "void":
-		// A void value always returns NULL. Since there is no built-in NULL
-		// value into the SQL package, we'll use sql.NullBool
-		return "sql.NullBool"
+		// A void value can only be scanned into an empty interface.
+		return "interface{}"
 
 	case "any":
 		return "interface{}"

internal/codegen/golang/query.go

@@ -1,6 +1,10 @@
 package golang
 
-import "strings"
+import (
+	"strings"
+
+	"github.com/kyleconroy/sqlc/internal/metadata"
+)
 
 type QueryValue struct {
 	Emit   bool
@@ -103,5 +107,6 @@ type Query struct {
 }
 
 func (q Query) hasRetType() bool {
-	return q.Cmd != ":exec" && !q.Ret.isEmpty()
+	scanned := q.Cmd == metadata.CmdOne || q.Cmd == metadata.CmdMany
+	return scanned && !q.Ret.isEmpty()
 }

internal/endtoend/testdata/pg_advisory_xact_lock/exec.sql

@@ -0,0 +1,5 @@
+-- name: AdvisoryLockExec :exec
+SELECT pg_advisory_lock($1);
+
+-- name: AdvisoryLockExecRows :execrows
+SELECT pg_advisory_lock($1);

internal/endtoend/testdata/pg_advisory_xact_lock/go/exec.sql.go

@@ -1,2 +1,8 @@
--- name: AdvisoryLock :many
+-- name: AdvisoryLockOne :one
+SELECT pg_advisory_lock($1);
+
+-- name: AdvisoryUnlock :many
 SELECT pg_advisory_unlock($1);
+
+-- name: AdvisoryLockExecResult :execresult
+SELECT pg_advisory_lock($1);

internal/endtoend/testdata/pg_advisory_xact_lock/go/query.sql.go

@@ -5,7 +5,7 @@
       "path": "go",
       "name": "querytest",
       "schema": "query.sql",
-      "queries": "query.sql"
+      "queries": ["query.sql", "exec.sql"]
     }
   ]
 }