Merge pull request #273 from psu-libraries/269-admin-delete

Removes the delete buttons for users and jobs in rails admin

Author: ajkiessl

.rubocop.yml

@@ -9,6 +9,7 @@ Metrics/BlockLength:
     - 'config/routes.rb'
     - 'app/models/gui_user.rb'
     - 'app/models/unit.rb'
+    - 'config/initializers/rails_admin.rb'
 
 Metrics/CyclomaticComplexity:
   Exclude:

config/initializers/rails_admin.rb

@@ -11,6 +11,26 @@
     @current_gui_user ||= GUIUser.find_by(id: session[:user_id]) if session[:user_id]
   end
 
+  config.actions do
+    dashboard                     # mandatory
+    index                         # mandatory
+    new
+    export
+    show
+    edit
+    show_in_app
+
+    delete do
+      visible do
+        bindings[:abstract_model].model == Unit
+      end
+    end
+
+    ## With an audit adapter, you can add:
+    # history_index
+    # history_show
+  end
+
   config.navigation_static_label = 'Tools'
   config.navigation_static_links = {
     'Sidekiq' => '/sidekiq'

spec/requests/admin/rails_admin_content_spec.rb

@@ -50,6 +50,7 @@
     expect(response.body).to include('123')
     expect(response.body).to include('45678')
     expect(response.body).to include(unit.total_pages_processed.to_s)
+    expect(response.body).to include('Delete')
   end
 
   it 'shows GuiUser list and show content' do
@@ -62,6 +63,7 @@
     expect(response.body).to include('gui.user@example.com')
     expect(response.body).to include('Accessibility Services Unit')
     expect(response.body).to include(gui_user_record.total_pages_processed_last_24_hours.to_s)
+    expect(response.body).not_to include('Delete')
   end
 
   it 'shows ApiUser list and show content including' do
@@ -77,6 +79,7 @@
     expect(response.body).to include('https://example.com/webhook')
     expect(response.body).to include('Accessibility Services Unit')
     expect(response.body).to include(api_user_record.total_pages_processed_last_24_hours.to_s)
+    expect(response.body).not_to include('Delete')
   end
 
   it 'shows Job list and show content' do