Skip to content

Commit

Permalink
fix citext resolution issues (#1629)
Browse files Browse the repository at this point in the history
* fix citext resolution issues
* add citext end to end test
  • Loading branch information
ryan-berger committed Jun 7, 2023
1 parent 6dceb87 commit edc1e5a
Show file tree
Hide file tree
Showing 11 changed files with 209 additions and 1 deletion.
2 changes: 1 addition & 1 deletion internal/codegen/golang/postgresql_type.go
Original file line number Diff line number Diff line change
Expand Up @@ -256,7 +256,7 @@ func postgresType(req *plugin.CodeGenRequest, col *plugin.Column) string {
}
return "sql.NullTime"

case "text", "pg_catalog.varchar", "pg_catalog.bpchar", "string":
case "text", "pg_catalog.varchar", "pg_catalog.bpchar", "string", "citext":
if notNull {
return "string"
}
Expand Down
32 changes: 32 additions & 0 deletions internal/endtoend/testdata/citext/pgx/go/db.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

14 changes: 14 additions & 0 deletions internal/endtoend/testdata/citext/pgx/go/models.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

35 changes: 35 additions & 0 deletions internal/endtoend/testdata/citext/pgx/go/query.sql.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 9 additions & 0 deletions internal/endtoend/testdata/citext/pgx/query.sql
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
CREATE TABLE foo (
bar citext,
bat citext not null
);

-- name: GetCitexts :many
SELECT bar, bat
FROM foo;

13 changes: 13 additions & 0 deletions internal/endtoend/testdata/citext/pgx/sqlc.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
{
"version": "1",
"packages": [
{
"path": "go",
"engine": "postgresql",
"sql_package": "pgx/v4",
"name": "querytest",
"schema": "query.sql",
"queries": "query.sql"
}
]
}
31 changes: 31 additions & 0 deletions internal/endtoend/testdata/citext/stdlib/go/db.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

14 changes: 14 additions & 0 deletions internal/endtoend/testdata/citext/stdlib/go/models.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

38 changes: 38 additions & 0 deletions internal/endtoend/testdata/citext/stdlib/go/query.sql.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

10 changes: 10 additions & 0 deletions internal/endtoend/testdata/citext/stdlib/query.sql
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
CREATE TABLE foo (
bar citext,
bat citext not null
);

-- name: GetCitexts :many
SELECT bar, bat
FROM foo;


12 changes: 12 additions & 0 deletions internal/endtoend/testdata/citext/stdlib/sqlc.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
{
"version": "1",
"packages": [
{
"path": "go",
"engine": "postgresql",
"name": "querytest",
"schema": "query.sql",
"queries": "query.sql"
}
]
}

0 comments on commit edc1e5a

Please sign in to comment.