Called methods improvements

derision-test / go-mockgen

MIT License

57 stars 7 forks source link

Called methods improvements #30

Closed kasvtv closed 7 months ago

kasvtv commented 2 years ago

Added CalledAtNWith method to conveniently check if a certain call (eg, the second or third) had the given arguments. I find that this is useful in quite a lot of testing scenarios
Made error messages from argument matchers more descriptive
Fixed some godoc statements

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

efritz commented 2 years ago

Would it be possible to fill out a symmetric match for the gomega testutil? Here is the implementation of BeCalledNWith. Having BeCalledAtNWith as well would be really great.

Otherwise I like the general approach of this PR and won't block 🎉

sonarcloud[bot] commented 7 months ago

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information