deduplicate methods to allow overlapping methods on embedded interfaces (#498)

Author: marten-seemann

.github/workflows/test.yaml

@@ -47,4 +47,9 @@ jobs:
         ./ci/check_panic_handling.sh
 
     - name: Run Go tests
-      run: go test -v ./...
+      run: |
+        for i in $(find $PWD -name go.mod); do
+          pushd $(dirname $i)
+          go test ./...
+          popd
+        done 

ci/check_go_generate.sh

@@ -18,7 +18,13 @@ trap cleanup EXIT
 
 cp -r . "${TEMP_DIR}/"
 cd $TEMP_DIR
-go generate ./...
+
+for i in $(find $PWD -name go.mod); do
+    pushd $(dirname $i)
+    go generate ./...
+    popd
+done 
+
 if ! diff -r . "${BASE_DIR}"; then
     echo
     echo "The generated files aren't up to date."

ci/check_go_mod.sh

@@ -3,7 +3,11 @@
 
 set -euo pipefail
 
-go mod tidy
+for i in $(find $PWD -name go.mod); do
+    pushd $(dirname $i)
+    go mod tidy
+    popd
+done 
 
 if [ ! -z "$(git status --porcelain)" ]; then
     git status

mockgen/internal/tests/overlapping_methods/go.mod

@@ -0,0 +1,5 @@
+module github.com/golang/mock/mockgen/internal/tests/overlapping_methods
+
+go 1.14
+
+require github.com/golang/mock v1.4.4

mockgen/internal/tests/overlapping_methods/go.sum

@@ -0,0 +1,8 @@
+github.com/golang/mock v1.4.4 h1:l75CXGRSwbaYNpl/Z2X1XIIAMSCquvXgpVZDhwEIJsc=
+github.com/golang/mock v1.4.4/go.mod h1:l3mdAwkq5BuhzHwde/uurv3sEJeZMXNpwsxVWU71h+4=
+golang.org/x/crypto v0.0.0-20190308221718-c2843e01d9a2/go.mod h1:djNgcEr1/C05ACkg1iLfiJU5Ep61QUkGW8qpdssI0+w=
+golang.org/x/net v0.0.0-20190311183353-d8887717615a/go.mod h1:t9HGtf8HONx5eT2rtn7q6eTqICYqUVnKs3thJo3Qplg=
+golang.org/x/sync v0.0.0-20190423024810-112230192c58/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
+golang.org/x/sys v0.0.0-20190215142949-d0b11bdaac8a/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY=
+golang.org/x/text v0.3.0/go.mod h1:NqM8EUOU14njkJ3fqMW+pc6Ldnwhi/IjpwHt7yyuwOQ=
+golang.org/x/tools v0.0.0-20190425150028-36563e24a262/go.mod h1:RgjU9mgBXZiqYHBnxXauZ1Gv1EHHAz9KjViQ78xBX0Q=

mockgen/internal/tests/overlapping_methods/interfaces.go

@@ -0,0 +1,13 @@
+// +build go1.14
+
+package overlap
+
+type ReadCloser interface {
+	Read([]byte) (int, error)
+	Close() error
+}
+
+type WriteCloser interface {
+	Write([]byte) (int, error)
+	Close() error
+}

mockgen/internal/tests/overlapping_methods/mock.go

@@ -0,0 +1,9 @@
+// +build go1.14
+
+package overlap
+
+//go:generate mockgen -package overlap -destination mock.go -source overlap.go -aux_files github.com/golang/mock/mockgen/internal/tests/overlapping_methods=interfaces.go
+type ReadWriteCloser interface {
+	ReadCloser
+	WriteCloser
+}

mockgen/internal/tests/overlapping_methods/overlap.go

@@ -0,0 +1,21 @@
+// +build go1.14
+
+package overlap
+
+import (
+	"errors"
+	"testing"
+
+	gomock "github.com/golang/mock/gomock"
+)
+
+// TestValidInterface assesses whether or not the generated mock is valid
+func TestValidInterface(t *testing.T) {
+	ctrl := gomock.NewController(t)
+	defer ctrl.Finish()
+
+	s := NewMockReadWriteCloser(ctrl)
+	s.EXPECT().Close().Return(errors.New("test"))
+
+	s.Close()
+}

mockgen/internal/tests/overlapping_methods/overlap_test.go

@@ -237,10 +237,12 @@ func TestGenerateMockInterface_Helper(t *testing.T) {
 				}
 			}
 
-			if err := g.GenerateMockInterface(&model.Interface{
-				Name:    "Somename",
-				Methods: test.Methods,
-			}, "somepackage"); err != nil {
+			intf := &model.Interface{Name: "Somename"}
+			for _, m := range test.Methods {
+				intf.AddMethod(m)
+			}
+
+			if err := g.GenerateMockInterface(intf, "somepackage"); err != nil {
 				t.Fatal(err)
 			}