Fix bug where column called updated_at was being skipped in readonly queries

Author: jarri-abidi

internal/codegen/golang/gen.go

@@ -424,12 +424,12 @@ func readOnly(queries []Query) []Query {
 		}
 
 		qsql := strings.ToUpper(q.SQL)
-		if !strings.Contains(qsql, "SELECT") {
+		if !strings.Contains(qsql, "SELECT ") {
 			continue
 		}
-		if strings.Contains(qsql, "INSERT") ||
-			strings.Contains(qsql, "UPDATE") ||
-			strings.Contains(qsql, "DELETE") {
+		if strings.Contains(qsql, "INSERT ") ||
+			strings.Contains(qsql, "UPDATE ") ||
+			strings.Contains(qsql, "DELETE ") {
 			continue
 		}
 

internal/endtoend/testdata/read_write_segregation/mysql/go/models.go

@@ -1,5 +1,5 @@
 /* name: GetUserByID :one */
-SELECT first_name, id, last_name FROM users WHERE id = sqlc.arg('target_id');
+SELECT first_name, id, last_name, updated_at FROM users WHERE id = sqlc.arg('target_id');
 
 /* name: ListUsers :many */
 SELECT first_name, last_name FROM users;

internal/endtoend/testdata/read_write_segregation/mysql/go/query.sql.go

@@ -1,6 +1,6 @@
 CREATE TABLE users (
     id SERIAL NOT NULL,
     first_name varchar(255) NOT NULL,
-    last_name varchar(255)
+    last_name varchar(255),
+    updated_at timestamp NULL DEFAULT CURRENT_TIMESTAMP
 );
-

internal/endtoend/testdata/read_write_segregation/mysql/go/read.go

@@ -1,5 +1,5 @@
 /* name: GetUserByID :one */
-SELECT first_name, id, last_name FROM users WHERE id = sqlc.arg('target_id');
+SELECT first_name, id, last_name, updated_at FROM users WHERE id = sqlc.arg('target_id');
 
 /* name: ListUsers :many */
 SELECT first_name, last_name FROM users;