search

moricho / tparallel

tparallel finds inappropriate usage of `t.Parallel()` method in your Go test codes
MIT License
24 stars 4 forks source link

Review by tenntenn #4

Open tenntenn opened 4 years ago

tenntenn commented 4 years ago

https://github.com/moricho/tparallel/blob/master/tparallel.go#L15 いい感じのコメントを書いたほうが良さそう

https://github.com/moricho/tparallel/blob/master/tparallel.go#L34 非公開な型に依存すると将来壊れる可能性あるのでtypes.LookupFieldOrMethodを使った方がよさそう。 参考：https://docs.google.com/presentation/d/1I4pHnzV2dFOMbRcpA-XD0TaLcX6PBKpls6WxGHoMjOg/edit#slide=id.g80c1410104_5_250

https://github.com/moricho/tparallel/blob/master/tparallel.go#L43 よいコメント 👍

https://github.com/moricho/tparallel/blob/master/tparallel.go#L89 先頭の命令だけで良さそう

よくかけてて良いですね！！

tenntenn commented 4 years ago

【コメント】 メルカリのテックブログでも話題になったt.Parallelがされているかの検出を行うツールを作ってくれました。コーナーケースを頑張って探して潰してくれているので、実用的なツールになってるんでは無いでしょうか。