fix: unnecessary trailing , on mock.Call

The `m.mock.Call()` invocation was generating a trailing comma with no arguments after it for methods with no parameters.
adamconnelly · Nov 30, 2023 · 49a2d52 · 49a2d52
1 parent e241b33
commit 49a2d52
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/cmd/kelpie/mock.go.tmpl b/cmd/kelpie/mock.go.tmpl
@@ -59,7 +59,7 @@ type Instance struct {
 {{- range $method := .Methods }}
 
 func (m *Instance) {{ $method.Name }}({{ template "parameterWithTypeList" $method.Parameters }}){{ if $method.Results }} ({{ template "resultWithTypeList" $method.Results }}){{ end }} {
-	expectation := m.mock.Call("{{ $method.Name }}", {{ template "parameterList" $method.Parameters }})
+	expectation := m.mock.Call("{{ $method.Name }}"{{ if $method.Parameters }}, {{ template "parameterList" $method.Parameters }}{{ end }})
 	if expectation != nil {
 		if expectation.ObserveFn != nil {
 			observe := expectation.ObserveFn.({{ template "observationCallback" $method }})

diff --git a/examples/mocks/accountservice/accountservice.go b/examples/mocks/accountservice/accountservice.go