Resolve Comments

app/controllers/partner_groups_controller.rb

@@ -1,5 +1,5 @@
 class PartnerGroupsController < ApplicationController
-  before_action :set_partner_groups, only: %i[edit destroy]
+  before_action :set_partner_group, only: %i[edit destroy]
 
   def new
     @partner_group = current_organization.partner_groups.new
@@ -41,7 +41,7 @@ def destroy
 
   private
 
-  def set_partner_groups
+  def set_partner_group
     @partner_group = current_organization.partner_groups.find(params[:id])
   end
 

app/views/partners/_partner_groups_table.html.erb

@@ -54,7 +54,7 @@
                     </td>
                     <td class="text-left">
                       <%= edit_button_to edit_partner_group_path(pg) %>
-                      <%= delete_button_to(partner_group_path(pg),{confirm: confirm_delete_msg(pg.name)}) unless pg.partners.any? %>
+                      <%= delete_button_to(partner_group_path(pg),{confirm: confirm_delete_msg(pg.name)}) if pg.partners.none? %>
                     </td>
                   </tr>
                 <% end %>

spec/system/partner_system_spec.rb

@@ -540,8 +540,8 @@
           it 'should not display the delete link' do
             visit partners_path
             click_on 'Groups'
-            expect(page).to have_content(first_partner_group.name)
-            expect(page).to have_content(first_partner.name)
+            expect(page).to have_content('Group 2')
+            expect(page).to have_content('Leslie Sue, the 1')
             expect(page).not_to have_link('Delete')
           end
         end
@@ -554,7 +554,7 @@
           it 'should display the delete link and allow deletion' do
             visit partners_path
             click_on 'Groups'
-            expect(page).to have_content(second_partner_group.name)
+            expect(page).to have_content('Group 1')
             expect(page).to have_link('Delete')
             expect do
               accept_confirm do