Improve logout method configuration by using first sign_out_via option on Devise. Thanks @carlesso

Author: jvortmann

app/helpers/rails_admin/application_helper.rb

@@ -48,7 +48,7 @@ def logout_path
     end
 
     def logout_method
-      return Devise.sign_out_via if defined?(Devise)
+      return [Devise.sign_out_via].flatten.first if defined?(Devise)
       :delete
     end
 

spec/helpers/rails_admin/application_helper_spec.rb

@@ -110,6 +110,21 @@ def initialize(_user)
       end
     end
 
+    describe '#logout_method' do
+      it 'defaults to :delete when Devise is not defined' do
+        allow(Object).to receive(:defined?).with(Devise).and_return(false)
+
+        expect(helper.logout_method).to eq(:delete)
+      end
+
+      it 'uses first sign out method from Devise when it is defined' do
+        allow(Object).to receive(:defined?).with(Devise).and_return(true)
+
+        expect(Devise).to receive(:sign_out_via).and_return([:whatever_defined_on_devise, :something_ignored])
+        expect(helper.logout_method).to eq(:whatever_defined_on_devise)
+      end
+    end
+
     describe '#wording_for' do
       it 'gives correct wording even if action is not visible' do
         RailsAdmin.config do |config|