サインアップ機能#1
Conversation
|
black(フォーマッタ)のチェックに失敗しました。CI実行のログを確認して修正し，再度コミット・プッシュしてください。 |
|
black(フォーマッタ)のチェックに失敗しました。CI実行のログを確認して修正し，再度コミット・プッシュしてください。 |
1 similar comment
|
black(フォーマッタ)のチェックに失敗しました。CI実行のログを確認して修正し，再度コミット・プッシュしてください。 |
|
Django Unit Testが失敗しました。実行ログを確認して修正し，再度コミット・プッシュしてください。 |
|
Django Unit Testが失敗しました。実行ログを確認して修正し，再度コミット・プッシュしてください。 |
|
Django Unit Testが失敗しました。実行ログを確認して修正し，再度コミット・プッシュしてください。 |
harune-pg
left a comment
harune-pg
left a comment
There was a problem hiding this comment.
最終課題の最初のステップ、画面と機能が要件通り実装できていてよかったです👍
いくつか修正点がありますので、ご確認お願いします🙏
| @@ -0,0 +1,11 @@ | |||
| {% extends "base.html" %} ← base.htmlを継承 | |||
There was a problem hiding this comment.
← コメント の形式のコメントを削除するか、HTMLのコメントの記法 <!-- コメント --> に修正するかしてください
| def test_failure_post_with_duplicated_user(self): | ||
| User.objects.create_user(username="testuser", password="testpassword") | ||
| invalid_data = { | ||
| "username": "testuser", | ||
| "email": "test@test.com", | ||
| "password1": "testpassword", | ||
| "password2": "testpassword", | ||
| } | ||
| response = self.client.post(self.url, invalid_data) | ||
| form = response.context["form"] | ||
| self.assertEqual(response.status_code, 200) | ||
| self.assertTrue(User.objects.filter(username=invalid_data["username"]).exists()) | ||
| self.assertFalse(form.is_valid()) | ||
| self.assertIn("同じユーザー名が既に登録済みです。", form.errors["username"]) |
There was a problem hiding this comment.
「DBにレコードが追加されていない」ことを検証するために、113行目の検証は少し不十分かもしれません。
exists() は対象のユーザーが 1 件以上存在するかどうかを確認するため、103行目で作成したユーザー以外に、意図せずフォームで新たに作成されたユーザーが存在する可能性を完全に排除できていない可能性があります。代わりに、count() を使用して、該当ユーザーのレコード数が1件であることを確認するのはいかがでしょうか？そうすることで、より確実に検証できるかと思います。
|
LGTM！ |
|
少々遅れながら、見ました。 |
3 participants
受講生の確認事項
１次レビュアーの確認事項