chore: avoid dbmock test errors in dbgen (#10923)

coderd/database/generate.sh

@@ -58,6 +58,7 @@ SCRIPT_DIR=$(dirname "${BASH_SOURCE[0]}")
 	go run golang.org/x/tools/cmd/goimports@latest -w queries.sql.go
 
 	go run ../../scripts/dbgen
-	# This will error if a view is broken.
+	# This will error if a view is broken. This is in it's own package to avoid
-	go test -run=TestViewSubset
+	# stuff like dbmock breaking builds if it's out of date from the interface.
+	go test ./gentest
 )

coderd/database/gentest/doc.go

@@ -0,0 +1,8 @@
+// Package gentest contains tests that are run at db generate time. These tests
+// need to exist in their own package to avoid importing stuff that gets
+// generated after the DB.
+//
+// E.g. if we put these tests in coderd/database, then we'd be importing dbmock
+// which is generated after the DB and can cause type problems when building
+// the tests.
+package gentest

coderd/database/gentest/models_test.go

@@ -1,4 +1,4 @@
-package database_test
+package gentest_test
 
 import (
 	"reflect"