Fix empty error message when call is exhausted

issue-404

Author: cvgw

gomock/callset.go

@@ -84,7 +84,11 @@ func (cs callSet) FindMatch(receiver interface{}, method string, args []interfac
 	for _, call := range exhausted {
 		if err := call.matches(args); err != nil {
 			_, _ = fmt.Fprintf(&callsErrors, "\n%v", err)
+			continue
 		}
+		_, _ = fmt.Fprintf(
+			&callsErrors, "all expected calls of the method %q have been exhausted for that received", method,
+		)
 	}
 
 	if len(expected)+len(exhausted) == 0 {

gomock/callset_test.go

@@ -21,7 +21,8 @@ import (
 
 type receiverType struct{}
 
-func (receiverType) Func() {}
+func (receiverType) Func()          {}
+func (receiverType) Func2(a string) {}
 
 func TestCallSetAdd(t *testing.T) {
 	method := "TestMethod"
@@ -74,3 +75,26 @@ func TestCallSetRemove(t *testing.T) {
 		cs.Remove(c)
 	}
 }
+
+func TestCallSetFindMatch(t *testing.T) {
+	t.Run("call is exhausted", func(t *testing.T) {
+		cs := callSet{}
+		var receiver interface{} = "TestReceiver"
+		method := "TestMethod"
+		args := []interface{}{}
+
+		c1 := newCall(t, receiver, method, reflect.TypeOf(receiverType{}.Func))
+		cs.exhausted = map[callSetKey][]*Call{
+			callSetKey{receiver: receiver, fname: method}: []*Call{c1},
+		}
+
+		_, err := cs.FindMatch(receiver, method, args)
+		if err == nil {
+			t.Fatal("expected error, but was nil")
+		}
+
+		if err.Error() == "" {
+			t.Fatal("expected error to have message, but was empty")
+		}
+	})
+}

sample/user_test.go

@@ -5,7 +5,7 @@ import (
 	"testing"
 
 	"github.com/golang/mock/gomock"
-	"github.com/golang/mock/sample"
+	user "github.com/golang/mock/sample"
 	"github.com/golang/mock/sample/imp1"
 	mock_user "github.com/golang/mock/sample/mock_user"
 )