Allow criteria to be deleted

app/forms/need_criteria_form.rb

@@ -1,6 +1,11 @@
 class NeedCriteriaForm < Rectify::Form
   class CriterionForm < Rectify::Form
     attribute :value, String
+    attribute :delete, Boolean
+
+    def deleted?
+      self.delete == true
+    end
   end
 
   mimic :need_criteria
@@ -15,7 +20,7 @@ def map_model(model)
   def criteria_attributes=(attributes)
     self.criteria = attributes.map {|_, item|
       CriterionForm.from_params(item)
-    }
+    }.reject(&:deleted?)
   end
 
   def add_extra_criteria!

app/views/need_criteria/edit.html.erb

@@ -11,9 +11,14 @@
 
     <%= semantic_form_for @form, url: need_criteria_path(need), as: :need_criteria do |f| %>
 
+      <ul>
       <%= f.fields_for :criteria, f.object.criteria do |cf| %>
-        <%= cf.input :value, as: :string, label: "Criteria #{cf.index+1}" %>
+        <li>
+          <%= cf.input :value, as: :string, label: "Criteria #{cf.index+1}" %>
+          <%= cf.input :delete, as: :boolean %>
+        </li>
       <% end %>
+      </ul>
 
       <%= f.action :submit, button_html: { class: 'btn btn-primary' } %>
       <%= f.action :submit, button_html: { class: 'btn btn-primary', value: t('formtastic.actions.need_criteria.save_add_more') } %>

spec/features/editing_need_criteria_spec.rb

@@ -47,4 +47,18 @@
     expect(inputs.size).to eq(expected_criteria.size + 1)
   end
 
+  it 'can delete a criteria' do
+    visit need_criteria_path(need)
+    click_on 'Edit'
+
+    input_container = page.all('form li').first
+    within(input_container) do
+      check 'Delete'
+    end
+    click_on 'Save criteria'
+
+    criteria = page.all('.need-criteria li').map(&:text)
+    expect(criteria).to contain_exactly(expected_criteria.last)
+  end
+
 end