form_for から form tag に変更し、リファクタリングをしました。

[yutokyokutyo]

Rails Tutorial 【Exercises8_5_1】

@tacahilo @kitak @gs3 @keokent

演習内容

http://www.railstutorial.org/book/sign_in_out#sec-sign_in_out_exercises

内容

paramsに注意し form_forからform tagに変更しリファクタリングをしましょう。という問題内容でしたので、以下のプロセスでリファクタリングをしました。

• [x] app/views/sessions/new.html.erbについて、form_forからform tagに変更する。
• [x] app/controllers/sessions_controller.rb について、不要なparams[:session]を削除する。

  テスト

• Greenを確認しました。

% bundle exec rspec spec/                                                                               (git)-[Exercises8_5_1]
....................................................

Finished in 0.97332 seconds
52 examples, 0 failures

Randomized with seed 37781

動作確認

変更前(form_forの場合)

Sign in ページ

HTMLのソースコード表示結果

    <form accept-charset="UTF-8" action="/sessions" method="post"><div style="margin:0;padding:0;display:inline"><input name="utf8" type="hidden" value="&#x2713;" /><input name="authenticity_token" type="hidden" value="7fDizIaz4gqxp9c9YpVE3hQ0qUGWNwFSgkIFU2zrIIU=" /></div>

      <label for="session_email">Email</label>
      <input id="session_email" name="session[email]" type="text" />

      <label for="session_password">Password</label>
      <input id="session_password" name="session[password]" type="password" />

      <input class="btn btn-large btn-primary" name="commit" type="submit" value="Sign in" />
</form>

変更後(form_tagの場合)

Sign in ページ

HTMLのソースコード表示結果

    <form accept-charset="UTF-8" action="/sessions" method="post"><div style="margin:0;padding:0;display:inline"><input name="utf8" type="hidden" value="&#x2713;" /><input name="authenticity_token" type="hidden" value="7fDizIaz4gqxp9c9YpVE3hQ0qUGWNwFSgkIFU2zrIIU=" /></div>

      <label for="email">Email</label>
      <input id="email" name="email" type="text" />

      <label for="password">Password</label>
      <input id="password" name="password" type="password" />

      <input class="btn btn-large btn-primary" name="commit" type="submit" value="Sign in" />
</form>

---

この内容でMasterにMergeしたいと思っております。 レビューをお願いいたします。

[hfm]

通ったけど， @ryoma123 とテストの数違うね :ghost:

usr0600296@okkun [~/src/github.com/yutokyokutyo/sample_app] (Exercises8_5_1[-] @0) [perl v5.18.2] [ruby 2.1.1p76] [python 2.7.7]
$ bundle exec rake spec
/Users/usr0600296/.rbenv/versions/2.1.1/bin/ruby -S rspec ./spec/helpers/application_helper_spec.rb ./spec/models/user_spec.rb ./spec/requests/authentication_pages_spec.rb ./spec/requests/static_pages_spec.rb ./spec/requests/user_pages_spec.rb
....................................................

Finished in 4.1 seconds
52 examples, 0 failures

Randomized with seed 63389

[yutokyokutyo]

@tacahilo

通ったけど, @ryoma123 とテストの数違うね

そうなんです。 自分のテストが不足しているのか、それとも@ryoma123 のテストが重複してしまっているのかがまだ分かっていないので時間があるときに確認してみます。(自分のテストが不足していた場合、テストの信憑性が失われてしまうので早めに確認をしようと思います。)

[yutokyokutyo]

@keokent

この謎の赤いマークは ファイルの末尾に 改行文字がないと出てしまいます。

そういうことだったのですね！ commitする際に見かけない変更点がありましたので、少し困惑しておりました。 今後気をつけていきたいと思います。 記事のリンクありがとうございます！

[yutokyokutyo]

内容自体は大丈夫そうなのでmasterにマージしたいと思います。 レビューありがとうございました。

[yutokyokutyo]

@gs3 なるほど！自分の書き方ではDeprecatedなのですね。 Rubyの読書会でやったばかりの内容だあ。 基本的にDeprecatedなものは非推奨なので使わないようにしたほうが良いと教えていただきましたので、変更します！ レビューありがとうございました！