Chapter12のエクササイズをやる #26

Merged
merged 8 commits into from Jul 2, 2015

Projects

None yet

3 participants

@orzup
Owner
orzup commented Jul 1, 2015

やること

  • ホームとプロフィールのstatsに関するテストを書こう
  • ホームでフィードが表示されるか確認するテストを書こう

完了条件

  • 2人にLGTMもらう
  • テストが通る
added some commits Jul 1, 2015
@orzup [WIP]
4e466ff
@orzup プロフィール画面のインテグレーションテストを追加 9ffdea0
@orzup フィードが確認できるかどうかテストを追加
955d2ce
@orzup statsに関するテストに修正
f5a7a8a
@orzup ホームにフィードが表示されるかテストを追加
118f1b3
@orzup テストの修正
1ページ目に表示されるべきフィードかどうかに関するテスト>に修正した。また、正しいフィードの値を担保するテスト項目をインテグレーションテストから削除した(モデルにすでにあった)。
27b0fed
@ku00 ku00 and 1 other commented on an outdated diff Jul 2, 2015
test/integration/users_profile_test.rb
- # assert_match @user.microposts.count.to_s, response.body
- # assert_select 'div.pagination'
- # @user.microposts.paginate(page: 1).each do |micropost|
- # assert_match micropost.content, response.body
- # end
- # end
+
+ test "profile display" do
+ get user_path(@user)
+ assert_template 'users/show'
+ assert_select 'title', full_title(@user.name)
+ assert_select 'img.gravatar'
+ assert_select 'h1', text: @user.name
+ assert_match @user.microposts.count.to_s, response.body
+ assert_match @user.following.count.to_s, response.body
+ assert_match @user.followers.count.to_s, response.body
@ku00
ku00 Jul 2, 2015

このテストだと、数字が一致しただけで通ってしまうのでよくなさそう

@orzup
orzup Jul 2, 2015 Owner

#26 (comment) と同じ意味だよね?

@hanazuki hanazuki and 1 other commented on an outdated diff Jul 2, 2015
test/integration/site_home_test.rb
@@ -0,0 +1,14 @@
+require 'test_helper'
+
+class SiteHomeTest < ActionDispatch::IntegrationTest
+ def setup
+ @user = users(:michael)
+ end
+
+ test "layout stuts" do
+ log_in_as(@user)
+ get root_path
+ assert_match @user.following.count.to_s, response.body
+ assert_match @user.followers.count.to_s, response.body
@hanazuki
hanazuki Jul 2, 2015

michaelのfollowersとfollowingが同じ人数のとき,どちらかの表示が壊れていてもテスト通ってしまいませんか?
そもそも2という一文字がレスポンスに含まれていても,それがフォロワーの人数を表しているのかどうか,わからない…

@orzup
orzup Jul 2, 2015 Owner

ハ! きちんとフォロワーとフォローの数をみなきゃダメだ。修正します!

@hanazuki hanazuki and 1 other commented on an outdated diff Jul 2, 2015
test/integration/site_home_test.rb
@@ -0,0 +1,14 @@
+require 'test_helper'
+
+class SiteHomeTest < ActionDispatch::IntegrationTest
+ def setup
+ @user = users(:michael)
+ end
+
+ test "layout stuts" do
@hanazuki
hanazuki Jul 2, 2015

statsのtypo 😁

@orzup
orzup Jul 2, 2015 Owner

!!!

added some commits Jul 2, 2015
@orzup typoを修正
07ff8af
@orzup きちんとフォローとフォロワーとの数をテストするように修正
3eba9e9
@orzup orzup changed the title from [WIP]Chapter12のエクササイズをやる to Chapter12のエクササイズをやる Jul 2, 2015
@ku00
ku00 commented Jul 2, 2015

よさそう 👍

@hanazuki
hanazuki commented Jul 2, 2015

👍

@orzup orzup merged commit 42ce28c into sample_app Jul 2, 2015

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@orzup orzup deleted the exercises12 branch Jul 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment