#1/user sign up

[Kaito-a-bit]

○issue 　 #1

○実装方針

• ユーザ登録画面に飛べる画面（base.html）
• ユーザ登録画面（sign_up.html）
• ユーザ登録後に遷移する画面（home.html） を作成した。

○実行画面

• ログイン画面

  

• ログイン後ホーム画面

  

• 管理画面（ユーザ登録が確認できる画面）

  

○質問・その他

• 現状、アドレスバーよりユーザ登録を経なくてもホーム画面に遷移できる仕様になってしまっています。 これを防ぐ方法を探したのですが、参考になるサイトを見つけることできませんでした。 これを防ぐ手段の名前などを教えて頂けたら幸いです。 →次のログイン・ログアウト機能の作成と同時に修正します。

• プロジェクトの作成時点で.gitignoreに記載した内容が不十分であったことにより、変更ファイルが多くなってしまっています。ご迷惑おかけします。

[Kaito-a-bit]

○課題 　・ユーザ登録には成功したが、その後のリダイアレクトが成功しないことがある。 　　（まだバグかどうか判然としない部分がある）

○解決策 　・htmlの<form>を閉じていなかったので適切な形にすることで望ましい挙動をするようになった。

<成功時のログ>

[Kaito-a-bit]

○現状 ・URLを打ち込めば、ログインしていなくてもホーム画面に行けてしまうようになっている

[Akinori13]

gitignoreを後から反映する方法があるので、それを実施してください。 次のprでログイン・ログアウト機能を実装しますが、ご質問の点についてはこれ が参考になるかと思います。

[Kaito-a-bit]

gitignoreを後から反映する方法があるので、それを実施してください。

この記事を参考に.gitignoreファイルを編集し、キャッシュを削除しました。

[Kaito-a-bit]

○修正点　

• 差分に含まれてしまっていた.pycをgit管理の対象から外しました。
• base.htmlを拡張する形でユーザ登録画面・ホーム画面を作り替えました。 尚、上記に加えてログイン画面・ログアウト画面を、base.htmlを拡張する形で実装しようと考えています。
• 上記に伴って不要になったcssファイルを削除しました。

[Kaito-a-bit]

○変更点

• [ ] ユーザ登録時にauthenticate()を通すようにした
• [ ] UserのモデルをAbstractBaseUserを拡張する形でカスタムのものに変更した
• [ ] フォームにhelp_text, errorsを表示させるようにした
• [ ] ユーザ登録に関わるテストを作成した

[Kaito-a-bit]

• TO DO
• [x] カスタムユーザモデルへの変更
• [x] ヘルプテキストの表示
• [x] エラーメッセージの表示
• [x] Djangoの認証バックエンドへの変更
• [x] 管理者サイトへのカスタムモデルの適用・表示
• [ ] テストの作成

[Kaito-a-bit]

デフォルトでblank=FALSE, null=FALSEらしいので必須フィールドに対する指定を削除した

[Kaito-a-bit]

• requirements.txtを作成しました
• 必要のないimportを削除、空行・インデント調整をしました
• migrationsディレクトリ内にある__pycache__ファイル以外のものをgit管理の対象に入れました
• UserCreationFormを使用してフォームを作成しました
• djangoの認証バックエンドにemailを用いて通すようにしました

PRが大きくなってすみません😞🙇‍♂️

[Kaito-a-bit]

user = authenticate(username=email, password=password)で既に試行していたが、passwordの取得ができていなかった。

• Userモデルに既にpasswordというカラムが存在しているが、password = form.cleaned_data.get('passsword')で取得できていない

• Cleaned_dataはフォームの中でバリデーションを通過したデータが格納されていく。バリデーション自体は通過しているはずなのに取得できない理由がわからないです。

[Akinori13]

user = authenticate(username=email, password=password)で既に試行していたが、passwordの取得ができていなかった。

• Userモデルに既にpasswordというカラムが存在しているが、password = form.cleaned_data.get('passsword')で取得できていない
• Cleaned_dataはフォームの中でバリデーションを通過したデータが格納されていく。バリデーション自体は通過しているはずなのに取得できない理由がわからないです。

UserCreationFormは、password1とpassword2というフォームアイテムを持ちます。このpassword1とpassword2のバリデーションチェックが完了した後に、password1がハッシュ化されてUserモデルのpasswordに格納されます。 したがってユーザーが入力した内容を確認する時は、password1を取得します。

form.cleaned_data.get('password1')

[Kaito-a-bit]

• フォームの入力に対して異常系と正常系エラーを作成。 status_code, リダイレクト先、エラー文、DBで生成されていないかをそれぞれ確認。

[Kaito-a-bit]

テスト作成の参考記事

• Djangoでテストを書こう【前編】
• はじめての Django アプリ作成、その 5
• テストツール

[Kaito-a-bit]

• 各ページ（top, signup, home）のステータスコードを確認するテストを作成
• 短すぎるパスワードを弾くテストを作成
• 長すぎるユーザネームを弾くテストを作成
• メールアドレスが固有であることを確かめるテストを作成
• 誤ったフォーマットのメールを弾くテストを作成

[Kaito-a-bit]

○リダイレクト先のテスト リダイレクト先の検証については下記のサイトを参考に作成した。

• 【Django】テストでリダイレクト先のページを検証する方法
• Django testing how to assert Redirect

def test_redirect_to_home(self):
    data = {
      'username': 'Gym Motivation3',
      'email': 'nanaMotive@gmail.com',
      'birthday': '2002-1-1',
      'password1': 'kjhd1245',
      'password2': 'kjhd1245',
    }
    self.response = self.client.post('/signup/', data)
    self.assertRedirects(self.response, '/home/', status_code=302, target_status_code=200, msg_prefix='', fetch_redirect_response=True)

上記のテストの実行結果は以下の通りとなった。 以下の結果を302が返ってくるはずだけど200が返ってきた。という趣旨のエラーだと解釈した。

ここでshellで同様の処理を実行してみることにした。 shell内ではpostした結果返ってきたのは302だった。 （あるページに遷移したことを示すコード）

• ここで何故テストが成功しないのかがわからないです。
• 指定の仕方があっているか曖昧ですが、redirect_chainについて確認してみました。 ここで、結果を返す配列が空になっているのが何故かについても理解ができませんでした。

[fumiharun11]

サインアップ処理自体に失敗しているため生じているエラーです。 以下のコメント箇所が原因と考えられます。

def test_redirect_to_home(self):
    data = {
      'username': 'Gym Motivation3', # 半角スペースがある
      'email': 'nanaMotive@gmail.com',
      'birthday': '2002-1-1',
      'password1': 'kjhd1245',
      'password2': 'kjhd1245',
    }
    self.response = self.client.post('/signup/', data)
    self.assertRedirects(self.response, '/home/', status_code=302, target_status_code=200, msg_prefix='', fetch_redirect_response=True)