search

yutokyokutyo / sample_app

Rails Tutorial 研修用
2 stars 0 forks source link

[Exercises9_6_3] Sign in していないユーザーに対し、“Profile”,“Settings” のリンクタグが見えないことを確認するテストを追加しました。 #18

Closed yutokyokutyo closed 10 years ago

yutokyokutyo commented 10 years ago

Rails Tutorial 【Exercises9_6_3】

内容

サインインしていないユーザーに対し、“Profile”,“Settings” のリンクタグが見えないことを確認するテストを追加しました。 最初にサインインページに訪れた場合と、サインインに失敗して再度サインインページにリダイレクトされた場合についてのテストとなります。 以下の手順により、この実装が正しく動作しているかをチェックしました。

全体テストでGreenを確認しました。

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

Finished in 3.69 seconds
90 examples, 0 failures

Randomized with seed 16257

ローカルサーバーチェック

最初にサインインをする場合

2014-09-04 19 01 03

サインインに失敗した場合

2014-09-04 19 10 00

演習内容

http://www.railstutorial.org/book/updating_and_deleting_users#sec-updating_deleting_exercises

この内容でMasterにマージしたいと思います。 レビューをお願いいたします。

@tacahilo @kitak @gs3 @keokent

hfm commented 10 years ago

「ローカルサーバーチェック」のところ，このPRに必要かな？

画像はインパクトが強いから，「Invalid email/password combination」の部分が先に目に入って， この表示を確認したのかな？とか，違うこと考えちゃうかも

hfm commented 10 years ago

局長は文章を足して何かを稼ごうとしている気配をたまに感じるのだけど，別のPRにも書いたとおり，お互いの時間は有限だから簡潔で分かりやすい文章が欲しいぞう :space_invader:

hfm commented 10 years ago

あ，内容はLGTMでござんす

keokent commented 10 years ago

良さそうだ！ :dog:

yutokyokutyo commented 10 years ago

@tacahilo @keokent

「ローカルサーバーチェック」のところ，このPRに必要かな？

情報は多いほうが良いのかなと考えて、画像を貼り付けてみました。しかし、確かに本来伝えたい目的を果たすどころか、誤解を招くような画像になってしまいましたね。 あまり意味がないであろうものはどんどん省略していこうと思います！

局長は文章を足して何かを稼ごうとしている気配をたまに感じるのだけど，別のPRにも書いたとおり，お互いの時間は有限だから簡潔で分かりやすい文章が欲しいぞう :space_invader:

PRにかける時間が長くなってしまうことは問題だと思っていて、自分が思いついたものをズラーっと書いてしまい、その結果レビューをしていただきたい相手には伝わりづらくなる文章が出来上がってしまうという次第です。 自分がPRを読みやすいものにするためにかける時間と、相手が読むのにかかる時間の両方のバランスを考えて、総合的に一番時間が短くなるようなPRを書いていこうと思います。 お互いの時間は有限だからというお言葉！とても響きました！ご指摘ありがとうございます！

内容は良さそうとのことなのでMasterにマージしたいと思います！ レビューありがとうございました:octocat:

keokent commented 10 years ago

:+1:

yutokyokutyo commented 10 years ago

:sparkling_heart:

hfm commented 10 years ago

PRにかける時間が長くなってしまうことは問題だと思っていて、自分が思いついたものをズラーっと書いてしまい、その結果レビューをしていただきたい相手には伝わりづらくなる文章が出来上がってしまうという次第です。 自分がPRを読みやすいものにするためにかける時間と、相手が読むのにかかる時間の両方のバランスを考えて、総合的に一番時間が短くなるようなPRを書いていこうと思います。

ほむほむ．期待してるー