Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Aug 30, 2022

単体テスト・結合テスト

単体テスト・結合テスト

#やった事・レビュー観点
単体テスト・結合テストの実装。

参考ファイルの共有頂きありがとうございます。
修正しました。
宜しくお願い致します。

@cha-han851
Copy link

単体テストに関してですが、色んなところで使われてるコードって何を示していますか?
ページネーションは次のページだけでなく前のページいけることも確認してください。
コードが500文字以上、、、500文字以下のテストもお願いします。
エラーメッセージの内容も記載をお願いします。
管理者マイページには管理者編集ページへの動線以外は何もないのでしょうか?
編集ページのコードを入力し更新ボタンを押下するっていうのは何を更新してるのでしょうか?
とりあえず上記ご確認お願いいたします。

@ghost
Copy link
Author

ghost commented Aug 30, 2022

単体テスト・結合テスト

単体テスト・結合テスト

#やった事・レビュー観点
単体テスト・結合テストの実装。

レビューありがとうございます!
管理者編集ページの件ですが、すみません。
管理者編集ページへの動線以外は何もないです。
管理者は全てのユーザーの記事を閲覧、編集、削除できる権限が有る実装をしてます。

@tatsuki-uchiyama
Copy link

tatsuki-uchiyama commented Aug 30, 2022

記事の投稿、編集機能では入力の必須チェックはないのでしょうか。
500文字以上のエラー文言が必須チェックのようになっているのは仕様通りなのでしょうか。

ご確認お願いします。

余談ですがマークダウンは記号と文字の間に半角スペースがないと機能しないので

# やったこと

このように記載してください。
こういった部分もPreviewを見るなどして正しく出力されていることを確認するようにしてください。

あと過去のプルリクエストはマージ後クローズしていってください。

@ghost
Copy link
Author

ghost commented Aug 30, 2022

単体テスト・結合テスト

単体テスト・結合テスト

やった事・レビュー観点

単体テスト・結合テストの実装。

レビューありがとうございます!
失礼致しました。
500文字以上に別で空チェックは必要です。
修正致しました。

かしこまりました。
ファイルの件も含め、コメント前にチェック意識します。

なぜかアルゴリズムだけプルリクエストクローズ出来ないです。
これから調査したいと思います。

@cha-han851
Copy link

ページネーションを押下という手順が二つあります。
切り分けしてください。
トップページに遷移するも二つありますが、この場合は一回の手順なので期待する結果をまとめてください。
xssやSQLなどの基本的なセキュリティのテスト項目も追加してください。

選択したユーザーのフォームを編集し、更新ボタンを押下する。この時のバリデーションはないのでしょうか?

@ghost
Copy link
Author

ghost commented Aug 30, 2022

単体テスト・結合テスト

単体テスト・結合テスト

やった事・レビュー観点

単体テスト・結合テストの実装。
度々レビュー頂きありがとうございます。

再修正して提出させて頂きました。
宜しくお願い致します。

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