podhmo-sandbox / sample-api

0 stars 0 forks source link

Modelのテストを書く #10

Closed podhmo closed 2 years ago

podhmo commented 2 years ago

この辺からはPRを作ったほうが良いか。。

podhmo commented 2 years ago

https://github.com/podhmo-sandbox/sample-api/blob/3215279a2d42e842052d367bb71596b02dccc23f/model/repository/todo_repository.go#L47-L55

こんな感じで素直にrepositoryという名前のところに直書きされているらしい。

この辺は付帯事項

podhmo commented 2 years ago

まぁsqlxくらいは使って良い気がする?

podhmo commented 2 years ago

repositoryにinterfaceを作るのは意味がない。定義する側がinterfaceを作るのではなく利用する側がinterfaceを作るべき。

https://github.com/podhmo-sandbox/sample-api/blob/3215279a2d42e842052d367bb71596b02dccc23f/model/repository/todo_repository.go#L11-L23

podhmo commented 2 years ago

とりあえず、どうにかこうにか利用するdbを書き換えられるようにならなければいけない。 (1つだけで済むなら環境変数とかで切り替えでテスト時には環境変数を指定、それをMakefileで共有とかでも良い気がする)

podhmo commented 2 years ago

素直にテストを書いた