Skip to content

Conversation

@KOUTYA47
Copy link
Contributor

チケットへのリンク

やったこと

  • servie_score_test.goのテストケースの実装

やらないこと

  • なし

できるようになること(ユーザ目線)

  • 何ができるようになるのか?(あれば。無いなら「無し」で OK)

できなくなること(ユーザ目線)

  • 何ができなくなるのか?(あれば。無いなら「無し」で OK)

動作確認

  • どのような動作確認を行ったのか? 結果はどうか?

その他

  • レビュワーへの参考情報(実装上の懸念点や注意点などあれば記載)

s: &ScoreService{
scoreRepo: &mockScoreRepository{
getMaxScores: func(ctx context.Context, userID uuid.UUID) (*model.Score, *model.Score, error) {
return nil, nil, errors.New("db error")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

repositoryErrorなのにdbとしているところは気になります。
どうでも良いですが。

Copy link
Member

@KinjiKawaguchi KinjiKawaguchi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

境界値のテストがほしいです。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants