[Fix #164] Make the organisme field mandatory

app/models/procedure.rb

@@ -40,6 +40,7 @@ class Procedure < ActiveRecord::Base
 
   validates :libelle, presence: true, allow_blank: false, allow_nil: false
   validates :description, presence: true, allow_blank: false, allow_nil: false
+  validates :organisation, presence: true, allow_blank: false, allow_nil: false
 
   def hide!
     now = DateTime.now

app/views/admin/procedures/_informations.html.haml

@@ -2,7 +2,7 @@
   .alert.alert-info
     Cette procédure est publiée, certains éléments de la description ne sont plus modifiables
 
-- { libelle: 'Libellé*', description: 'Description*', organisation: 'Organisme', direction: 'Direction', lien_site_web: 'Lien site internet', lien_notice: 'Lien notice' }.each do |key, value|
+- { libelle: 'Libellé*', description: 'Description*', organisation: 'Organisme*', direction: 'Direction', lien_site_web: 'Lien site internet', lien_notice: 'Lien notice' }.each do |key, value|
   .form-group
     %h4
       = value

config/locales/models/procedure/fr.yml

@@ -1,5 +1,8 @@
 fr:
   activerecord:
+    attributes:
+      procedure:
+        organisation: Organisme
     errors:
       models:
         procedure:
@@ -10,3 +13,5 @@ fr:
               blank: Attribut manquant
             lien_demarche:
               blank: Attribut manquant
+            organisation:
+              blank: Attribut manquant

spec/features/admin/procedure_cloning_spec.rb

@@ -13,6 +13,7 @@ feature 'As an administrateur I wanna clone a procedure', js: true do
       page.find_by_id('new-procedure').click
       fill_in 'procedure_libelle', with: 'libelle de la procedure'
       page.execute_script("$('#procedure_description').data('wysihtml5').editor.setValue('description de la procedure')")
+      fill_in 'procedure_organisation', with: 'organisme de la procedure'
       page.find_by_id('save-procedure').click
     end
 

spec/features/admin/procedure_creation_spec.rb

@@ -37,6 +37,7 @@ feature 'As an administrateur I wanna create a new procedure', js: true do
       page.find_by_id('flash_message').visible?
       fill_in 'procedure_libelle', with: 'libelle de la procedure'
       page.execute_script("$('#procedure_description').data('wysihtml5').editor.setValue('description de la procedure')")
+      fill_in 'procedure_organisation', with: 'organisme de la procedure'
       page.find_by_id('save-procedure').click
       expect(page).to have_current_path(admin_procedure_types_de_champ_path(Procedure.first.id.to_s))
     end
@@ -47,6 +48,7 @@ feature 'As an administrateur I wanna create a new procedure', js: true do
       page.find_by_id('new-procedure').click
       fill_in 'procedure_libelle', with: 'libelle de la procedure'
       page.execute_script("$('#procedure_description').data('wysihtml5').editor.setValue('description de la procedure')")
+      fill_in 'procedure_organisation', with: 'organisme de la procedure'
       page.find_by_id('save-procedure').click
     end
 

spec/models/procedure_spec.rb

@@ -57,6 +57,12 @@ describe Procedure do
       it { is_expected.to allow_value('').for(:lien_demarche) }
       it { is_expected.to allow_value('http://localhost').for(:lien_demarche) }
     end
+
+    context 'organisation' do
+      it { is_expected.not_to allow_value(nil).for(:organisation) }
+      it { is_expected.not_to allow_value('').for(:organisation) }
+      it { is_expected.to allow_value('URRSAF').for(:organisation) }
+    end
   end
 
   describe '#types_de_champ_ordered' do