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

テストの追加 #616

Merged
merged 24 commits into from
Dec 28, 2021
Merged

テストの追加 #616

merged 24 commits into from
Dec 28, 2021

Conversation

TOMOFUMI-KONDO
Copy link
Collaborator

close #441

やったこと

  • chat.ts というファイルにテストを追加した
    • 実際のリポジトリクラスやデリバリクラスを使い、擬似的なクライアントをからのリクエストに正しくレスポンスを返すかのテスト。おそらく統合テストの立ち位置。

やっていないこと

  • アプリケーション側のエラーハンドリング等が未実装の箇所のテストをいくつかskipにしてます

動作確認手順

  • /app/server 直下で yarn test

参考リンク・補足など

Copy link
Collaborator

@Eri-0910 Eri-0910 left a comment

Choose a reason for hiding this comment

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

ちょっとだけコメントしたけど大きな問題はないのでいいと思います

)
})

test("異常系_存在しないtopicには投稿できない", (resolve) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

異常系、存在しないトピックだけじゃなくて開始していないトピックに送れないのもチェックしていいかもしれない。

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

d516ce7
追加しました〜

)
})

test("正常系_Reactionの投稿", (resolve) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

存在しないMessageへのリアクション(&Question)……は考えなくていいか……

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

まあ統合テストなのでそこまで細かくテストしなくてもいいかなと...
ここの機能は単体テストでカバーしているので。(リポジトリのテストも後で書きます)

Comment on lines +917 to +920
// TODO: 結果が不安定でたまに失敗する(DBへのインサートの順序が実行時依存なのがおそらくの原因)のでskipにしている。
// このエンドポイントのレスポンスの方を仕様通りに直せばchatItemsのパラメータにexpect.arrayContaining()が使えるので、
// おそらくはそれで解決できそう。
test.skip("正常系_終了したルームのチャットアイテムの履歴を見れる", async () => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

気が向いたらissueにしておいてください

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

作っておきました
#645

@TOMOFUMI-KONDO TOMOFUMI-KONDO merged commit 0c9784b into develop Dec 28, 2021
@TOMOFUMI-KONDO TOMOFUMI-KONDO deleted the TOMOFUMI-KONDO/update-e2e-test branch December 28, 2021 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

server側のテストを更新する
2 participants