Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conflicting tests: 'redirects to home page if not logged in' #385

Open
taoliu12 opened this issue Apr 19, 2019 · 0 comments
Open

Conflicting tests: 'redirects to home page if not logged in' #385

taoliu12 opened this issue Apr 19, 2019 · 0 comments

Comments

@taoliu12
Copy link
Contributor

taoliu12 commented Apr 19, 2019

users_features_spec 48:

it 'prevents user from viewing user show page and redirects to home page if not logged in' do
    create_standard_user
    visit '/users/1'
    expect(current_path).to eq('/')
    expect(page).to have_content("Sign Up")
  end

The previous test requires creating a user to also log them in, but now this test doesn't log them out before visiting '/users/1', even though it is testing what happens if a user is logged out.

@taoliu12 taoliu12 changed the title conflicting tests Conflicting tests: 'redirects to home page if not logged in' Apr 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant