tree bf65ae5cb9a923962ea63187bd840bc44fc0c715
parent f25e14ca273c89d3b4f19db82be674666c1df13f
author Damien Neil <dneil@google.com> 1546820533 -0800
committer Damien Neil <dneil@google.com> 1546903037 +0000

cmd/protoc-gen-go: add additional import public test case

New test case demonstrating an import public bug:

  a.proto publicly imports b.proto and c.proto
  b.proto publicly imports c.proto
  a.pb.go includes two copies of symbols in c.pb.go

The problem is that the new implementation of public imports, which
parses the generated code for the import to determine what symbols
are present, doesn't distinguish between symbols defined in a file
and symbols imported from a public import of some other file. This
can then lead to duplicate definitions in cases like the above.

Change-Id: Ia86e9f188d7bae8d9d4afbd2b5db9b64071425c3
Reviewed-on: https://go-review.googlesource.com/c/156347
Reviewed-by: Joe Tsai <thebrokentoaster@gmail.com>