Disable Unit destuction for base units with subunits

Closes #40

app/controllers/default/units_controller.rb

@@ -12,7 +12,7 @@ class Default::UnitsController < ApplicationController
   end
 
   def index
-    @units = current_user.units.defaults_diff.includes(:base).ordered
+    @units = current_user.units.defaults_diff.includes(:base).includes(:subunits).ordered
   end
 
   def import

app/controllers/units_controller.rb

@@ -9,7 +9,7 @@ class UnitsController < ApplicationController
   end
 
   def index
-    @units = current_user.units.includes(:subunits).ordered
+    @units = current_user.units.includes(:base).includes(:subunits).ordered
   end
 
   def new

app/views/default/units/_unit.html.erb

@@ -12,7 +12,7 @@
       <% if current_user.at_least(:admin) %>
         <% if unit.default? %>
           <%= image_button_to t('.delete'), 'delete-outline', default_unit_path(unit),
-            method: :delete %>
+            method: :delete, **disabled_attributes(!unit.movable?) %>
         <% else %>
           <%= image_button_to t('.export'), 'upload-outline', export_default_unit_path(unit),
             disabled_attributes(!unit.portable?) %>

app/views/units/_unit.html.erb

@@ -17,8 +17,8 @@
           id: dom_id(unit, :new, :link), onclick: 'this.blur();', data: {turbo_stream: true} %>
       <% end %>
 
-      <%= image_button_to t('.delete_unit'), 'delete-outline', unit_path(unit),
+      <%= image_button_to t('.destroy'), 'delete-outline', unit_path(unit),
-        method: :delete %>
+        method: :delete, **disabled_attributes(!unit.movable?) %>
     </td>
     <% if unit.movable? %>
       <td class="handle" draggable="true">&#x283F</td>

config/locales/en.yml

@@ -59,7 +59,7 @@ en:
     no_items: There are no configured units. You can Add some or Import from defaults.
     unit:
       new_subunit: Subunit
-      delete_unit: Delete
+      destroy: Delete
     index:
       new_unit: Add unit
       import_units: Import

test/system/units_test.rb

@@ -162,6 +162,13 @@ class UnitsTest < ApplicationSystemTestCase
     assert_selector 'tbody > tr:has(input, textarea)'
   end
 
-  #test "destroy" do
+  test "destroy" do
-  #end
+    link = all(:link_or_button, exact_text: t('units.unit.destroy')).sample
+    label = link.ancestor('tr').first(:link)[:text]
+    assert_difference ->{ @user.units.count }, -1 do
+      link.click
+    end
+    assert_selector 'tbody tr', count: [@user.units.count, 1].max
+    assert_selector '.flash.notice', text: t('units.destroy.success', unit: label)
+  end
 end