Merge pull request #3385 from consul/welcome_page_not_logged_in_user

Require logged in user if navigate to /welcome
This commit is contained in:
Julian Nicolas Herrero
2019-03-22 17:35:29 +01:00
committed by GitHub
2 changed files with 6 additions and 0 deletions

View File

@@ -1,6 +1,7 @@
class WelcomeController < ApplicationController
skip_authorization_check
before_action :set_user_recommendations, only: :index, if: :current_user
before_action :authenticate_user!, only: :welcome
layout "devise", only: [:welcome, :verification]

View File

@@ -4,6 +4,11 @@ feature "Welcome screen" do
let(:budget) { create(:budget) }
scenario "requires a logged in user" do
visit welcome_path
expect(page).to have_content "You must sign in or register to continue."
end
scenario "for a not verified user" do
user = create(:user)
login_through_form_as(user)