tree 400a1f17eacf3517214c2bb4f21ab48d748bd9a0
parent fffc8cfead64a5d3e404da06ace390a62f82921d
author Jonathan Amsterdam <jba@google.com> 1777631329 -0400
committer Jonathan Amsterdam <jba@google.com> 1777653795 -0700

internal/tests/api: inline insertModules

Put the code of insertModules back into testServePackage.

The motivation behind having a separate function was
to ultimately insert modules for all tests just once.
But after some thought, I think it's a bad idea.
It would couple tests together, and put the module
definitions far from the point where the desired results
are.

Change-Id: I687df3ff9ecb53d20610211c7a65dba80161a2f5
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/772980
LUCI-TryBot-Result: golang-scoped@luci-project-accounts.iam.gserviceaccount.com <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
kokoro-CI: kokoro <noreply+kokoro@google.com>
Reviewed-by: Ethan Lee <ethanalee@google.com>