現場で使えるRuby on Rails速習実践ガイド RSpecのlet共通化でテストが通らない

Question

letを使用した共通化がテキスト通りに打ってもテストが通りません。
ユーザーAが作ったタスクをユーザーBが見れないテストなのですが、見れてしまうようです。

require 'rails_helper'

describe 'タスク管理機能' ,type: :system do
	describe '一覧表示機能' do
		let(:user_a) { FactoryBot.create(:user, name: 'ユーザーA', email: '[e-mail]')}
		let(:user_b) { FactoryBot.create(:user, name: 'ユーザーB', email: '[e-mail]')}

		before do
			FactoryBot.create(:task, name: '最初のタスク', user: user_a)
			visit login_path
			fill_in 'メールアドレス', with: '[e-mail]'
			fill_in 'パスワード', with: 'password'
			click_button 'ログインする'
		end

		context 'ユーザーAがログインしている時' do
			let(:login_user) { user_a }
			it 'ユーザーAが作成したタスクが表示される' do
				expect(page).to have_content '最初のタスク'
			end
		end

		context 'ユーザーBがログインしている時' do
			let(:login_user) { user_b }
			it 'ユーザAが作成したタスクが表示されない' do
				expect(page).to have_no_content '最初のタスク'
			end
		end
	end
end

Failures:

  1) タスク管理機能 一覧表示機能 ユーザーHがログインしている時 ユーザDが作成したタスクが表示されない
     Failure/Error: expect(page).to have_no_content '最初のタスク'
       expected not to find text "最初のタスク" in "Taskleaf\nタスク一覧\nログアウト\nログインしました\nタスク一覧\n新規登録\n名称 登録日時\n最初のタスク 2019-03-05 02:39:31 UTC 編集削除"
     
     [Screenshot]: tmp/screenshots/failures_r_spec_example_groups_nested_nested_h_ユーザdが作成したタスクが表示されない_308.png

     
     # ./spec/system/tasks_spec.rb:27:in `block (4 levels) in <top (required)>'

Finished in 8.03 seconds (files took 5.41 seconds to load)
2 examples, 1 failure

Failed examples:

rspec ./spec/system/tasks_spec.rb:26 # タスク管理機能 一覧表示機能 ユーザーHがログインしている時 ユーザDが作成したタスクが表示されない

ご教授お願いいたします。

Answer 1

その書籍を持ってないのですが

fill_in 'メールアドレス', with: '[e-mail]'
fill_in 'パスワード', with: 'password'

が気になります。

実際、サポートサイトからチャプター5完成形を覗いてみると違っています。

Comments

[yuriko]

サポートセンターまで見ていただきましてありがとうございます。
実は、サポートサイトではそのチャプターごとの"最終形"が見れるだけであり、その道のりは見れないのです。

共通化する前のコードはこちらで、パスします。
なので、fill_in の箇所は問題ないと思っております。

```

require 'rails_helper'

describe 'タスク管理機能' ,type: :system do
describe '一覧表示機能' do
before do
user_a = FactoryBot.create(:user, name: 'ユーザーA', email: '[e-mail]')
FactoryBot.create(:task, name: '最初のタスク', user: user_a)
end

context 'ユーザーAがログインしている時' do
before do
visit login_path
fill_in 'メールアドレス', with: '[e-mail]'
fill_in 'パスワード', with: 'password'
click_button 'ログインする'
end
end

context 'ユーザーBがログインしている時' do
before do
FactoryBot.create(:user, name: 'ユーザーB', email: '[e-mail]')
visit login_path
fill_in 'メールアドレス', with: '[e-mail]'
fill_in 'パスワード', with: 'password'
click_button 'ログインする'
end

it 'ユーザーAが作成したタスクが表示されない' do
expect(page).to have_no_content '最初のタスク'
end
end
end
end
```

もし再度お気付きの点がございましたらよろしくお願いいたします。

[asm]

ですからfill_in 'メールアドレス'に注目してAでログインする場合とBでログインする場合を見比べ
そして、共通化した後を見てください。

ユーザーAでしかログインできていません。

これがあなたが共通化する際に失敗したのか
それとも、テキスト通りで失敗した理由などを説明する文章が後にあるのか
は知りませんが
完成形では、きちんとログインユーザーをcontext毎に振り分けられています。

[yuriko]

どうもありがとうございました。
仰る通りの箇所の私の確認ミスでした。