Fix varargs generation

[ryanbrainard]

This fixes vektra#44 in this fork (will push to main fork later). It still does not work nice for the MockOn convenience methods because things get tricky with converting to interface arrays and it is not clear what the expected behavior should be, so leaving it as is for now.

cc: @fabiokung

[fabiokung]

👍

[fabiokung]

the space in between the param name and the ellipsis (...) is optional in Go

[ryanbrainard]

Mockery normalizes the params up above directly from the AST, so this
should be ok, but I can test it to make sure.

On Friday, October 16, 2015, Fabio Kung notifications@github.com wrote:

In mockery/generator.go
#3 (comment):

@@ -277,6 +278,15 @@ func (g *Generator) genList(list *ast.FieldList, addNames bool) ([]string, []str
return names, types, params
}

+func (g *Generator) paramsAsArgNames(params []string) []string {

• args := []string{}
• r := regexp.MustCompile("(.+?) (.{3})?.+")

the space in between the param name and the ellipsis (...) is optional in
Go

—
Reply to this email directly or view it on GitHub
https://github.com/ryanbrainard/mockery/pull/3/files#r42300369.

[ryanbrainard]

I tried it out, and it does work, but I replaced the regex with direct use of the AST, which is how I really should have done it to begin with: #4