-
Notifications
You must be signed in to change notification settings - Fork 342
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Potencial improvements to default tests template #98
Comments
@matrixik for changing from
to
I think it's better to create different issue for changing struct slice to map struct |
I think if we have a complex scenario like if we have in order dependency, the map randomly maybe cause result will be unknown. and if need that map style, generated then manually modify also is easy. |
@matrixik I feel the same. @davecheney wrote a great article about PROs to stick with maps in table-driven tests. |
I'm not sure if this will be interesting for this project but Dave Cheney gave nice presentation about writing tests for Go code with some improvements over default template from
gotests
:https://dave.cheney.net/paste/gopherchina-2019-testing-talk.pdf
Like having
name
inmap[string]struct
instead of[]struct
or using https://github.com/google/go-cmpDiff
for better displaying what differ between outputs (instead of%v
).The text was updated successfully, but these errors were encountered: