tree c6134dc0380d24cc8fe71d45398a83916676d099
parent 1321339c8979f1c2b5efb0ddf5d98e0dade18622
author Jonathan Amsterdam <jba@google.com> 1591730089 -0400
committer Jonathan Amsterdam <jba@google.com> 1591812430 +0000

internal/postgres: upsert module data

When we insert module information, make sure we overwrite existing
rows. Previously, we specified that new data should be ignored if the
row already existed, which meant that we were adding rows but never
changing them. That didn't matter at one point, when we deleted a
module before re-inserting it, but we no longer delete, so we must
upsert.

We also fix a number of places where our test modules had duplicate
directories. The upserts failed for these.

Change-Id: I97465b11e4ea6cbb7835e883f36f098445207eba
Reviewed-on: https://team-review.git.corp.google.com/c/golang/discovery/+/766365
Reviewed-by: Julie Qiu <julieqiu@google.com>