Variadic function issues. about mockery HOT 10 CLOSED

@shortsyoungster @gurudesu I found the issue and submitted a PR to fix it. If the tests pass I will merge and tag the fix.

Sorry for the delay in getting to this. @gurudesu after you told me you were telling people to use gomock I knew I wouldn't be able to sleep until my competition is crushed, so I hope this fixes your issue 😅

This bug actually had nothing to do with the variadic-ness of the function (although I did find a separate bug that I don't think was part of the original complaint here). It was failing on the return value of the function argument. Details in the linked PR.

PS I guess a way to get me to do stuff is to just say gomock does something better. That should never be the case in my opinion!

from mockery.

Hello, please provide a reproducer. Without this, I cannot provide any help.

from mockery.

type MyNewInterface interface {
	FetchData(
		ctx context.Context,
		dataType reflect.Type,
		fetchFunc func(xxx ...yyy.ZZ) ([]aa.BB, error),
	) ([]aa.BB, error)
}

This worked on Mockery v2.20.2

from mockery.

I would presume that it has to do with this commit d119c66

from mockery.

I have tested on v2.40.0, and everything works, this is the version right before d119c66 so I am fairly confident that it is the cause.

from mockery.

I think the thing to do here is add your example interface as a fixture and have mockery generate it within this project (mainly for testing/backwards compat reasons), then figure out precisely why this fails. Variadic argument have been the bane of my existence in this project, so I'm guessing we'll need yet another special exception for whatever is going wrong here.

from mockery.

I'm experiencing the same issues here - especially with trying to mock the AWS SDK Go V2 interfaces. As an example, see this struct.

Not a bash on this project (thank you for supporting this) but for the time being we've been pushing folks to use gomock instead since they seem to handle variadic arguments well.

from mockery.

I'll do what I can to fix this, if anyone would be able to submit a PR that would be great, I probably can't get to it for the next few weeks due to being busy at work.

I hope to not drive people away to the main competition for this project, that irks me :)

from mockery.

Thanks @LandonTClipp! Will try this out and recommend folks in my org to just upgrade 👍

from mockery.

This worked for me @LandonTClipp. Thank you so much!

from mockery.