Merge pull request #3008 from oblakeerickson/log_out_return_success

Have log_out method return json

app/controllers/admin/users_controller.rb

@@ -66,10 +66,14 @@ class Admin::UsersController < Admin::AdminController
   end
 
   def log_out
-    @user.auth_token = nil
+    if @user
-    @user.save!
+      @user.auth_token = nil
-    MessageBus.publish "/logout", @user.id, user_ids: [@user.id]
+      @user.save!
-    render nothing: true
+      MessageBus.publish "/logout", @user.id, user_ids: [@user.id]
+      render json: success_json
+    else
+      render json: {error: I18n.t('admin_js.admin.users.id_not_found')}, status: 404
+    end
   end
 
   def refresh_browsers

config/locales/client.en.yml

@@ -1887,6 +1887,7 @@ en:
         create: 'Add Admin User'
         last_emailed: "Last Emailed"
         not_found: "Sorry, that username doesn't exist in our system."
+        id_not_found: "Sorry, that user id doesn't exist in our system."
         active: "Active"
         show_emails: "Show Emails"
         nav:

spec/controllers/admin/users_controller_spec.rb

@@ -360,6 +360,29 @@ describe Admin::UsersController do
       end
     end
 
+    context 'log_out' do
+      before do
+        @reg_user = Fabricate(:user)
+      end
+
+      it 'returns JSON' do
+        xhr :put, :log_out, user_id: @reg_user.id
+        ::JSON.parse(response.body).should be_present
+      end
+
+      it "returns success" do
+        xhr :put, :log_out, user_id: @reg_user.id
+        response.should be_success
+        json = ::JSON.parse(response.body)
+        json['success'].should == "OK"
+      end
+
+      it "returns 404 when user_id does not exist" do
+        xhr :put, :log_out, user_id: 123123
+        response.should_not be_success
+      end
+    end
+
     context 'block' do
       before do
         @reg_user = Fabricate(:user)