Merge pull request #869 from CHSS-IT/fix_local_logout_routing

Fix local logout routing

app/controllers/sessions_controller.rb

@@ -9,6 +9,10 @@ def end_session
     redirect_to '/logout'
   end
 
+  def destroy
+    redirect_to root_path, notice: t(".success")
+  end
+
   private
 
   def find_user

config/locales/en.yml

@@ -47,6 +47,9 @@ en:
   settings:
     update:
       success: 'Setting was successfully updated.'
+  sessions:
+    destroy:
+      success: 'You have logged out successfully.'
   users:
     create:
       success: 'User was successfully created'

config/routes.rb

@@ -25,4 +25,6 @@
   get '/login', to: 'sections#index'
 
   get 'exit', to: 'sessions#end_session', as: :logout
+
+  get '/logout', to: 'sessions#destroy'
 end

test/controllers/sessions_controller_test.rb

@@ -24,4 +24,9 @@ class SessionsControllerTest < ActionDispatch::IntegrationTest
     get logout_url
     assert_in_delta @user.reload.updated_at, old_updated_at, 1.second
   end
+
+  test 'destroy action redirects to the root path' do
+    get '/logout'
+    assert_redirected_to root_path
+  end
 end