Skip to content
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

テスト戦略 #584

Open
3 of 6 tasks
ras0q opened this issue Oct 24, 2023 · 0 comments
Open
3 of 6 tasks

テスト戦略 #584

ras0q opened this issue Oct 24, 2023 · 0 comments
Labels

Comments

@ras0q
Copy link
Member

ras0q commented Oct 24, 2023

テストのデータが密になってきてかなりわかりにくくなってきた
もう1度整理したい
#556 がマージされた後の世界

  • infrastructure/repository
    • DBを使った各メソッドのテスト
    • 現状のsqlmockを使った単体テストは消す
    • 現integration_tests/repositoryを移動すれば大体十分だがrepository単体で複雑なシナリオテストを書く必要はないはず
  • integration_tests/api
    • 現integration_tests/handler
    • 各APIのTable Driven Testで十分
    • ステータスメッセージ周りが雑なのでどうにかしたい
  • integration_tests/scenario
    • 実際の利用を想定して複数APIを組み合わせたシナリオテスト
    • 余裕があったらでいいと思う
  • interfaces/handler
    • handler層の単体テスト
    • service層&repository層のモックを使う
  • usecases/service
    • service層の単体テスト
    • repository層のモックを使う
    • service自体が消えるかもしれない(service層を消す #431)のでテストが成功する範囲で放置していてもまあOK
  • その他汎用関数のテスト
@ras0q ras0q added the test label Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant