From ccd36309fc414fa6d64130ab861503cf3171bca8 Mon Sep 17 00:00:00 2001 From: Jonathan Amsterdam Date: Mon, 22 Jun 2020 08:55:53 -0400 Subject: all.bash,internal: force DB testing in CI If the environment variable GO_DISCOVERY_TESTDB is "true" wherever we would skip DB tests, fail instead. Set it to "true" in all.bash. Fixes golang/go#39713. Change-Id: I80fe27578a05610e29cc3ad9f6d477c9768fd51a Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/239257 Reviewed-by: Julie Qiu --- internal/database/database.go | 1 - 1 file changed, 1 deletion(-) (limited to 'internal/database/database.go') diff --git a/internal/database/database.go b/internal/database/database.go index d13973dd..4c8aff18 100644 --- a/internal/database/database.go +++ b/internal/database/database.go @@ -138,7 +138,6 @@ func processRows(rows *sql.Rows, f func(*sql.Rows) error) error { // // If the isolation level requires it, Transact will retry the transaction upon // serialization failure, so txFunc may be called more than once. - func (db *DB) Transact(ctx context.Context, iso sql.IsolationLevel, txFunc func(*DB) error) (err error) { defer derrors.Wrap(&err, "Transact(%s)", iso) // For the levels which require retry, see -- cgit v1.3