Build interface

Author: stevepolitodesign

app/controllers/active_sessions_controller.rb

@@ -1,2 +1,25 @@
 class ActiveSessionsController < ApplicationController
+  before_action :authenticate_user!
+
+  def destroy
+    @active_session = current_user.active_sessions.find(params[:id])
+
+    @active_session.destroy
+
+    if current_user
+      redirect_to account_path, notice: "Session deleted."
+    else
+      reset_session
+      redirect_to root_path, notice: "Signed out."
+    end
+  end
+
+  def destroy_all
+    user = current_user
+
+    current_user.active_sessions.destroy_all
+    reset_session
+
+    redirect_to root_path, notice: "Signed out."
+  end
 end

app/views/users/edit.html.erb

@@ -25,19 +25,18 @@
 <% end %>
 <h2>Current Logins</h2>
 <% if @active_sessions.any? %> 
-  <%= link_to "Log out of all other sessions", destroy_all_active_sessions_path %>
+  <%= button_to "Log out of all other sessions", destroy_all_active_sessions_path, method: :delete %>
   <table>
     <thead>
       <tr>
         <th>User Agent</th>
         <th>IP Address</th>
         <th>Signed In At</th>
+        <th>Sign Out</th>
       </tr>
     </thead>
     <tbody>
-      <tr>
-        <%= render @active_sessions %>
-      </tr>
+      <%= render @active_sessions %>
     </tbody>
   </table>
 <% end %>

test/controllers/active_sessions_controller_test.rb

@@ -1,7 +1,45 @@
 require "test_helper"
 
 class ActiveSessionsControllerTest < ActionDispatch::IntegrationTest
-  # test "the truth" do
-  #   assert true
-  # end
+  setup do
+    @confirmed_user = User.create!(email: "confirmed_user@example.com", password: "password", password_confirmation: "password", confirmed_at: Time.current)
+  end
+
+  test "should destroy all active sessions" do
+    login @confirmed_user
+    @confirmed_user.active_sessions.create!
+
+    assert_difference("ActiveSession.count", -2) do
+      delete destroy_all_active_sessions_path
+    end
+
+    assert_redirected_to root_path
+    assert_nil current_user
+    assert_not_nil flash[:notice]
+  end
+
+  test "should destroy another session" do
+    login @confirmed_user
+    @confirmed_user.active_sessions.create!
+
+    assert_difference("ActiveSession.count", -1) do
+      delete active_session_path(@confirmed_user.active_sessions.last)
+    end
+
+    assert_redirected_to account_path
+    assert_not_nil current_user
+    assert_not_nil flash[:notice]
+  end
+
+  test "should destroy current session" do
+    login @confirmed_user
+
+    assert_difference("ActiveSession.count", -1) do
+      delete active_session_path(@confirmed_user.active_sessions.last)
+    end
+
+    assert_redirected_to root_path
+    assert_nil current_user
+    assert_not_nil flash[:notice]
+  end
 end

test/integration/user_interface_test.rb

@@ -20,12 +20,16 @@ class UserInterfaceTest < ActionDispatch::IntegrationTest
 
     get account_path
 
-    assert_select "a", "Log out of all other sessions"
+    assert_select "input[type='submit']" do
+      assert_select "[value=?]", "Log out of all other sessions"
+    end
     assert_match destroy_all_active_sessions_path, @response.body
 
     assert_select "table" do
-      assert_select "a", "Sign Out"
+      assert_select "input[type='submit']" do
+        assert_select "[value=?]", "Sign Out"
+      end
     end
-    assert_match destroy_active_session_path(@confirmed_user.active_sessions.last), @response.body
+    assert_match active_session_path(@confirmed_user.active_sessions.last), @response.body
   end
 end