tree 801c86390426df2ce3a7ef3d2c540db44d00b354
parent 84987778548c8997c44fceeb47b489cd3f37f5ec
author Jean de Klerk <deklerk@google.com> 1611940224 -0700
committer Jean de Klerk <deklerk@google.com> 1612185300 +0000

x/exp/cmd/gorelease: add test for tidy submodule

Details:

- Adds a test for the missing req submodule case. However, it turns out
  we don't need any code in the filepath.Walk to handle submodules,
  since earlier in copyModuleToTempDir we only copy the module's
  contents (not any submodule contents) with zip.CreateFromDir. This
  means that prepareLoadDir is never going to consider any submodule
  content, and we don't have to build logic for that. The test will
  maintain that guarantee.

- Remove some extraneous go.mod dupes from other tests.

- Remove a TODO related to filepath.Walk submodule checking which is no
  longer needed.

Fixes golang/go#41456

Change-Id: Ib0bac2fb227d4175c31603880b629374991b8bde
Reviewed-on: https://go-review.googlesource.com/c/exp/+/287972
Run-TryBot: Jean de Klerk <deklerk@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Trust: Jean de Klerk <deklerk@google.com>
Reviewed-by: Jay Conrod <jayconrod@google.com>