diff options
| author | Jonathan Amsterdam <jba@google.com> | 2021-11-15 09:18:34 -0500 |
|---|---|---|
| committer | Jonathan Amsterdam <jba@google.com> | 2021-11-15 20:17:11 +0000 |
| commit | 664ef416968ca8bc699cf240615bbce4bd366f49 (patch) | |
| tree | fc4e0292c7e56b6ad9a67a0ee36056e978c2b7a9 /internal/postgres/unit_test.go | |
| parent | e4128003cbb95f3ff3b5795f5db9524f6a6f77e7 (diff) | |
| download | go-x-pkgsite-664ef416968ca8bc699cf240615bbce4bd366f49.tar.xz | |
internal/licenses: remove Metadata.OldCoverage
We don't need the OldCoverage field anymore. All DBs have
only the new Coverage information.
Change-Id: Ib2d8e0cdacdc2f7b6514a8a5310d6905be15136c
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/363954
Trust: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Jonathan Amsterdam <jba@google.com>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jamal Carvalho <jamal@golang.org>
Diffstat (limited to 'internal/postgres/unit_test.go')
| -rw-r--r-- | internal/postgres/unit_test.go | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/internal/postgres/unit_test.go b/internal/postgres/unit_test.go index 973e9f9a..9c7813c0 100644 --- a/internal/postgres/unit_test.go +++ b/internal/postgres/unit_test.go @@ -78,7 +78,7 @@ func testGetUnitMeta(t *testing.T, ctx context.Context) { } opts := []cmp.Option{ cmpopts.EquateEmpty(), - cmpopts.IgnoreFields(licenses.Metadata{}, "Coverage", "OldCoverage"), + cmpopts.IgnoreFields(licenses.Metadata{}, "Coverage"), cmpopts.IgnoreFields(internal.UnitMeta{}, "HasGoMod"), cmp.AllowUnexported(source.Info{}, safehtml.HTML{}), } @@ -355,7 +355,7 @@ func TestGetUnitMetaBypass(t *testing.T) { } opts := []cmp.Option{ cmpopts.EquateEmpty(), - cmpopts.IgnoreFields(licenses.Metadata{}, "Coverage", "OldCoverage"), + cmpopts.IgnoreFields(licenses.Metadata{}, "Coverage"), cmpopts.IgnoreFields(internal.UnitMeta{}, "HasGoMod"), cmp.AllowUnexported(source.Info{}, safehtml.HTML{}), } @@ -756,7 +756,7 @@ func checkUnit(ctx context.Context, t *testing.T, db *DB, um *internal.UnitMeta, opts := []cmp.Option{ cmp.AllowUnexported(source.Info{}, safehtml.HTML{}), // The packages table only includes partial license information; it omits the Coverage field. - cmpopts.IgnoreFields(licenses.Metadata{}, "Coverage", "OldCoverage"), + cmpopts.IgnoreFields(licenses.Metadata{}, "Coverage"), } want.SourceInfo = um.SourceInfo want.NumImports = len(want.Imports) @@ -862,7 +862,7 @@ func TestGetUnitFieldSet(t *testing.T) { cmp.AllowUnexported(source.Info{}, safehtml.HTML{}), cmpopts.EquateEmpty(), // The packages table only includes partial license information; it omits the Coverage field. - cmpopts.IgnoreFields(licenses.Metadata{}, "Coverage", "OldCoverage"), + cmpopts.IgnoreFields(licenses.Metadata{}, "Coverage"), } test.want.CommitTime = um.CommitTime test.want.SourceInfo = um.SourceInfo |