diff options
| author | Jonathan Amsterdam <jba@google.com> | 2021-06-14 08:34:13 -0400 |
|---|---|---|
| committer | Jonathan Amsterdam <jba@google.com> | 2021-06-14 15:29:59 +0000 |
| commit | b23d3c6e62bb91fdbeb0bfaac12e7cd359b0b553 (patch) | |
| tree | b164e6198e8e9bd652ed2defeb44c0c12c650a67 /internal/postgres/insert_module_test.go | |
| parent | 06a364cdacf24055dbd0a5c4d24897ea007f550a (diff) | |
| download | go-x-pkgsite-b23d3c6e62bb91fdbeb0bfaac12e7cd359b0b553.tar.xz | |
internal/postgres: don't touch package_imports
If the read-imports experiment is on, replace all references to the
package_imports table with the imports table: don't write into or read
from package_imports.
Change-Id: I02631aba574b9938a44b06bce855997606984939
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/327749
Trust: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Jonathan Amsterdam <jba@google.com>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Julie Qiu <julie@golang.org>
Diffstat (limited to 'internal/postgres/insert_module_test.go')
| -rw-r--r-- | internal/postgres/insert_module_test.go | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/internal/postgres/insert_module_test.go b/internal/postgres/insert_module_test.go index 5b26656c..981e7780 100644 --- a/internal/postgres/insert_module_test.go +++ b/internal/postgres/insert_module_test.go @@ -24,6 +24,7 @@ import ( "golang.org/x/pkgsite/internal" "golang.org/x/pkgsite/internal/database" "golang.org/x/pkgsite/internal/derrors" + "golang.org/x/pkgsite/internal/experiment" "golang.org/x/pkgsite/internal/licenses" "golang.org/x/pkgsite/internal/source" "golang.org/x/pkgsite/internal/stdlib" @@ -659,7 +660,15 @@ func TestIsAlternativeModulePath(t *testing.T) { func TestReInsertLatestVersion(t *testing.T) { t.Parallel() - ctx := context.Background() + t.Run("package_imports", func(t *testing.T) { + testReInsertLatestVersion(t, context.Background()) + }) + t.Run("imports", func(t *testing.T) { + testReInsertLatestVersion(t, experiment.NewContext(context.Background(), internal.ExperimentReadImports)) + }) +} + +func testReInsertLatestVersion(t *testing.T, ctx context.Context) { testDB, release := acquire(t) defer release() |