From c2916e9d050daf25943e960a152473df9631d3dd Mon Sep 17 00:00:00 2001
From: clemkeirua <clement@keiruaprod.fr>
Date: Thu, 29 Aug 2019 16:01:35 +0200
Subject: [PATCH 01/14] remove possibility to create carte with parcelle
 agricole

---
 app/controllers/champs/carte_controller.rb                | 8 --------
 .../TypesDeChampEditor/components/TypeDeChamp.js          | 4 ----
 2 files changed, 12 deletions(-)

diff --git a/app/controllers/champs/carte_controller.rb b/app/controllers/champs/carte_controller.rb
index 47c15da93..059c4554f 100644
--- a/app/controllers/champs/carte_controller.rb
+++ b/app/controllers/champs/carte_controller.rb
@@ -47,14 +47,6 @@ class Champs::CarteController < ApplicationController
         end
       end
 
-      if @champ.parcelles_agricoles?
-        parcelles_agricoles = ApiCartoService.generate_rpg(coordinates)
-        geo_areas += parcelles_agricoles.map do |parcelle_agricole|
-          parcelle_agricole[:source] = GeoArea.sources.fetch(:parcelle_agricole)
-          parcelle_agricole
-        end
-      end
-
       selection_utilisateur = ApiCartoService.generate_selection_utilisateur(coordinates)
       selection_utilisateur[:source] = GeoArea.sources.fetch(:selection_utilisateur)
       geo_areas << selection_utilisateur
diff --git a/app/javascript/components/TypesDeChampEditor/components/TypeDeChamp.js b/app/javascript/components/TypesDeChampEditor/components/TypeDeChamp.js
index 3b5b01647..b158d51da 100644
--- a/app/javascript/components/TypesDeChampEditor/components/TypeDeChamp.js
+++ b/app/javascript/components/TypesDeChampEditor/components/TypeDeChamp.js
@@ -144,10 +144,6 @@ const TypeDeChamp = sortableElement(
               label="Cadastres"
               handler={updateHandlers.cadastres}
             />
-            <TypeDeChampCarteOption
-              label="Parcelles Agricoles"
-              handler={updateHandlers.parcelles_agricoles}
-            />
           </TypeDeChampCarteOptions>
           <TypeDeChampRepetitionOptions
             isVisible={isRepetition}

From f296318072fb2b66a86afd15f78ad5de9c9eee9a Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Mon, 19 Aug 2019 16:12:30 +0200
Subject: [PATCH 02/14] Create GroupeInstructeur

---
 app/models/groupe_instructeur.rb                     |  3 +++
 app/models/procedure.rb                              |  1 +
 .../20190819134252_create_groupe_instructeurs.rb     | 12 ++++++++++++
 db/schema.rb                                         | 12 +++++++++++-
 4 files changed, 27 insertions(+), 1 deletion(-)
 create mode 100644 app/models/groupe_instructeur.rb
 create mode 100644 db/migrate/20190819134252_create_groupe_instructeurs.rb

diff --git a/app/models/groupe_instructeur.rb b/app/models/groupe_instructeur.rb
new file mode 100644
index 000000000..6c8453f26
--- /dev/null
+++ b/app/models/groupe_instructeur.rb
@@ -0,0 +1,3 @@
+class GroupeInstructeur < ApplicationRecord
+  belongs_to :procedure
+end
diff --git a/app/models/procedure.rb b/app/models/procedure.rb
index 9536829fb..ef26e3153 100644
--- a/app/models/procedure.rb
+++ b/app/models/procedure.rb
@@ -18,6 +18,7 @@ class Procedure < ApplicationRecord
   has_many :administrateurs_procedures
   has_many :administrateurs, through: :administrateurs_procedures, after_remove: -> (procedure, _admin) { procedure.validate! }
   has_many :instructeurs, through: :assign_to
+  has_many :groupe_instructeurs, dependent: :destroy
 
   has_one :initiated_mail, class_name: "Mails::InitiatedMail", dependent: :destroy
   has_one :received_mail, class_name: "Mails::ReceivedMail", dependent: :destroy
diff --git a/db/migrate/20190819134252_create_groupe_instructeurs.rb b/db/migrate/20190819134252_create_groupe_instructeurs.rb
new file mode 100644
index 000000000..df90219b3
--- /dev/null
+++ b/db/migrate/20190819134252_create_groupe_instructeurs.rb
@@ -0,0 +1,12 @@
+class CreateGroupeInstructeurs < ActiveRecord::Migration[5.2]
+  def change
+    create_table :groupe_instructeurs do |t|
+      t.references :procedure, foreign_key: true, null: false
+      t.text :label, null: false
+
+      t.timestamps
+    end
+
+    add_index :groupe_instructeurs, [:procedure_id, :label], unique: true
+  end
+end
diff --git a/db/schema.rb b/db/schema.rb
index fdbf8f457..a6f024915 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2019_08_09_073736) do
+ActiveRecord::Schema.define(version: 2019_08_19_134252) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -391,6 +391,15 @@ ActiveRecord::Schema.define(version: 2019_08_09_073736) do
     t.index ["source"], name: "index_geo_areas_on_source"
   end
 
+  create_table "groupe_instructeurs", force: :cascade do |t|
+    t.bigint "procedure_id", null: false
+    t.text "label", null: false
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.index ["procedure_id", "label"], name: "index_groupe_instructeurs_on_procedure_id_and_label", unique: true
+    t.index ["procedure_id"], name: "index_groupe_instructeurs_on_procedure_id"
+  end
+
   create_table "individuals", id: :serial, force: :cascade do |t|
     t.string "nom"
     t.string "prenom"
@@ -638,6 +647,7 @@ ActiveRecord::Schema.define(version: 2019_08_09_073736) do
   add_foreign_key "dossiers", "users"
   add_foreign_key "feedbacks", "users"
   add_foreign_key "geo_areas", "champs"
+  add_foreign_key "groupe_instructeurs", "procedures"
   add_foreign_key "initiated_mails", "procedures"
   add_foreign_key "procedure_presentations", "assign_tos"
   add_foreign_key "procedures", "services"

From 115d91387c530f546b71cbc019179a016142f436 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Tue, 27 Aug 2019 15:28:02 +0200
Subject: [PATCH 03/14] A procedure has a default groupe instructeur

---
 app/models/groupe_instructeur.rb                     |  1 +
 app/models/procedure.rb                              | 11 +++++++++++
 ...0819142551_create_default_groupe_instructeur.rake | 12 ++++++++++++
 spec/models/procedure_spec.rb                        |  7 +++++++
 4 files changed, 31 insertions(+)
 create mode 100644 lib/tasks/deployment/20190819142551_create_default_groupe_instructeur.rake

diff --git a/app/models/groupe_instructeur.rb b/app/models/groupe_instructeur.rb
index 6c8453f26..ad4a1988b 100644
--- a/app/models/groupe_instructeur.rb
+++ b/app/models/groupe_instructeur.rb
@@ -1,3 +1,4 @@
 class GroupeInstructeur < ApplicationRecord
+  DEFAULT_LABEL = 'défaut'
   belongs_to :procedure
 end
diff --git a/app/models/procedure.rb b/app/models/procedure.rb
index ef26e3153..56a4637ec 100644
--- a/app/models/procedure.rb
+++ b/app/models/procedure.rb
@@ -74,6 +74,7 @@ class Procedure < ApplicationRecord
   before_save :update_juridique_required
   before_save :update_durees_conservation_required
   before_create :ensure_path_exists
+  after_create :ensure_default_groupe_instructeur
 
   include AASM
 
@@ -477,6 +478,10 @@ class Procedure < ApplicationRecord
     end
   end
 
+  def defaut_groupe_instructeur
+    groupe_instructeurs.find_by(label: GroupeInstructeur::DEFAULT_LABEL)
+  end
+
   private
 
   def move_type_de_champ_attributes(types_de_champ, type_de_champ, new_index)
@@ -566,4 +571,10 @@ class Procedure < ApplicationRecord
       self.path = SecureRandom.uuid
     end
   end
+
+  def ensure_default_groupe_instructeur
+    if self.groupe_instructeurs.empty?
+      groupe_instructeurs.create(label: GroupeInstructeur::DEFAULT_LABEL)
+    end
+  end
 end
diff --git a/lib/tasks/deployment/20190819142551_create_default_groupe_instructeur.rake b/lib/tasks/deployment/20190819142551_create_default_groupe_instructeur.rake
new file mode 100644
index 000000000..4e3021da7
--- /dev/null
+++ b/lib/tasks/deployment/20190819142551_create_default_groupe_instructeur.rake
@@ -0,0 +1,12 @@
+namespace :after_party do
+  desc 'Deployment task: create_default_groupe_instructeur'
+  task create_default_groupe_instructeur: :environment do
+    Procedure.find_each do |procedure|
+      procedure.groupe_instructeurs.create(label: GroupeInstructeur::DEFAULT_LABEL)
+    end
+
+    # Update task as completed.  If you remove the line below, the task will
+    # run with every deploy (or every time you call after_party:run).
+    AfterParty::TaskRecord.create version: '20190819142551'
+  end
+end
diff --git a/spec/models/procedure_spec.rb b/spec/models/procedure_spec.rb
index e81d11f25..741a87de7 100644
--- a/spec/models/procedure_spec.rb
+++ b/spec/models/procedure_spec.rb
@@ -882,4 +882,11 @@ describe Procedure do
       end
     end
   end
+
+  describe '.ensure_a_groupe_instructeur_exists' do
+    let!(:procedure) { create(:procedure) }
+
+    it { expect(procedure.groupe_instructeurs.count).to eq(1) }
+    it { expect(procedure.groupe_instructeurs.first.label).to eq(GroupeInstructeur::DEFAULT_LABEL) }
+  end
 end

From 97884c4349274bedbc32027a614ec79d14887ab4 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Tue, 20 Aug 2019 16:47:41 +0200
Subject: [PATCH 04/14] Link AssignTo et GroupeInstructeur

---
 app/models/assign_to.rb                           |  1 +
 app/models/groupe_instructeur.rb                  |  2 ++
 app/models/instructeur.rb                         |  1 +
 ...add_groupe_instructeur_column_to_assign_tos.rb |  6 ++++++
 db/schema.rb                                      |  6 +++++-
 ...145528_link_assign_and_groupe_instructeur.rake | 15 +++++++++++++++
 6 files changed, 30 insertions(+), 1 deletion(-)
 create mode 100644 db/migrate/20190819145355_add_groupe_instructeur_column_to_assign_tos.rb
 create mode 100644 lib/tasks/deployment/20190819145528_link_assign_and_groupe_instructeur.rake

diff --git a/app/models/assign_to.rb b/app/models/assign_to.rb
index 61802b8c0..33425d306 100644
--- a/app/models/assign_to.rb
+++ b/app/models/assign_to.rb
@@ -1,6 +1,7 @@
 class AssignTo < ApplicationRecord
   belongs_to :procedure
   belongs_to :instructeur
+  belongs_to :groupe_instructeur
   has_one :procedure_presentation, dependent: :destroy
 
   scope :with_email_notifications, -> { where(email_notifications_enabled: true) }
diff --git a/app/models/groupe_instructeur.rb b/app/models/groupe_instructeur.rb
index ad4a1988b..dcb8a14de 100644
--- a/app/models/groupe_instructeur.rb
+++ b/app/models/groupe_instructeur.rb
@@ -1,4 +1,6 @@
 class GroupeInstructeur < ApplicationRecord
   DEFAULT_LABEL = 'défaut'
   belongs_to :procedure
+  has_many :assign_tos
+  has_many :instructeurs, through: :assign_tos
 end
diff --git a/app/models/instructeur.rb b/app/models/instructeur.rb
index 5fcb1b205..e2e079b2d 100644
--- a/app/models/instructeur.rb
+++ b/app/models/instructeur.rb
@@ -7,6 +7,7 @@ class Instructeur < ApplicationRecord
 
   has_many :assign_to, dependent: :destroy
   has_many :procedures, through: :assign_to
+  has_many :groupe_instructeurs, through: :assign_to
 
   has_many :assign_to_with_email_notifications, -> { with_email_notifications }, class_name: 'AssignTo', inverse_of: :instructeur
   has_many :procedures_with_email_notifications, through: :assign_to_with_email_notifications, source: :procedure
diff --git a/db/migrate/20190819145355_add_groupe_instructeur_column_to_assign_tos.rb b/db/migrate/20190819145355_add_groupe_instructeur_column_to_assign_tos.rb
new file mode 100644
index 000000000..95168a8db
--- /dev/null
+++ b/db/migrate/20190819145355_add_groupe_instructeur_column_to_assign_tos.rb
@@ -0,0 +1,6 @@
+class AddGroupeInstructeurColumnToAssignTos < ActiveRecord::Migration[5.2]
+  def change
+    add_reference :assign_tos, :groupe_instructeur, foreign_key: true
+    add_index :assign_tos, [:groupe_instructeur_id, :instructeur_id], unique: true, name: 'unique_couple_groupe_instructeur_instructeur'
+  end
+end
diff --git a/db/schema.rb b/db/schema.rb
index a6f024915..a2ee405e7 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2019_08_19_134252) do
+ActiveRecord::Schema.define(version: 2019_08_19_145355) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -118,6 +118,9 @@ ActiveRecord::Schema.define(version: 2019_08_19_134252) do
     t.datetime "created_at"
     t.datetime "updated_at"
     t.boolean "email_notifications_enabled", default: false, null: false
+    t.bigint "groupe_instructeur_id"
+    t.index ["groupe_instructeur_id", "instructeur_id"], name: "unique_couple_groupe_instructeur_instructeur", unique: true
+    t.index ["groupe_instructeur_id"], name: "index_assign_tos_on_groupe_instructeur_id"
     t.index ["instructeur_id", "procedure_id"], name: "index_assign_tos_on_instructeur_id_and_procedure_id", unique: true
     t.index ["instructeur_id"], name: "index_assign_tos_on_instructeur_id"
     t.index ["procedure_id"], name: "index_assign_tos_on_procedure_id"
@@ -634,6 +637,7 @@ ActiveRecord::Schema.define(version: 2019_08_19_134252) do
     t.index ["procedure_id"], name: "index_without_continuation_mails_on_procedure_id"
   end
 
+  add_foreign_key "assign_tos", "groupe_instructeurs"
   add_foreign_key "attestation_templates", "procedures"
   add_foreign_key "attestations", "dossiers"
   add_foreign_key "avis", "instructeurs", column: "claimant_id"
diff --git a/lib/tasks/deployment/20190819145528_link_assign_and_groupe_instructeur.rake b/lib/tasks/deployment/20190819145528_link_assign_and_groupe_instructeur.rake
new file mode 100644
index 000000000..eaf745055
--- /dev/null
+++ b/lib/tasks/deployment/20190819145528_link_assign_and_groupe_instructeur.rake
@@ -0,0 +1,15 @@
+namespace :after_party do
+  desc 'Deployment task: link_assign_and_groupe_instructeur'
+  task link_assign_and_groupe_instructeur: :environment do
+    AssignTo.find_each do |at|
+      GroupeInstructeur
+        .find_by(procedure_id: at.procedure_id)
+        &.assign_tos
+        &.push(at)
+    end
+
+    # Update task as completed.  If you remove the line below, the task will
+    # run with every deploy (or every time you call after_party:run).
+    AfterParty::TaskRecord.create version: '20190819145528'
+  end
+end

From 25991c28921c3119a0a359effbdc53070fdb5007 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Tue, 20 Aug 2019 16:59:45 +0200
Subject: [PATCH 05/14] Assign a procedure assign a group as well

---
 app/models/instructeur.rb       | 5 ++++-
 spec/models/instructeur_spec.rb | 4 ++++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/app/models/instructeur.rb b/app/models/instructeur.rb
index e2e079b2d..6d4c93d4a 100644
--- a/app/models/instructeur.rb
+++ b/app/models/instructeur.rb
@@ -53,7 +53,10 @@ class Instructeur < ApplicationRecord
 
   def assign_to_procedure(procedure)
     begin
-      procedures << procedure
+      assign_to.create({
+        procedure: procedure,
+        groupe_instructeur: procedure.defaut_groupe_instructeur
+      })
       true
     rescue ActiveRecord::RecordNotUnique
       false
diff --git a/spec/models/instructeur_spec.rb b/spec/models/instructeur_spec.rb
index 5cc374a86..30b8d026a 100644
--- a/spec/models/instructeur_spec.rb
+++ b/spec/models/instructeur_spec.rb
@@ -97,6 +97,10 @@ describe Instructeur, type: :model do
 
       it { is_expected.to be_truthy }
       it { expect { subject }.to change(instructeur.procedures, :count) }
+      it do
+        subject
+        expect(instructeur.groupe_instructeurs).to include(procedure_to_assign.defaut_groupe_instructeur)
+      end
     end
 
     context "with an already assigned procedure" do

From 85499365a0290a4010ed564a09ff9f138160430b Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Tue, 20 Aug 2019 18:03:33 +0200
Subject: [PATCH 06/14] Instructeur has_many procedures through
 groupe_instructeur

---
 app/models/instructeur.rb                       |  4 ++--
 .../instructeurs/procedures_controller_spec.rb  | 10 +++++-----
 .../instructeurs/recherche_controller_spec.rb   |  2 +-
 spec/controllers/invites_controller_spec.rb     |  2 +-
 spec/controllers/root_controller_spec.rb        |  2 +-
 spec/models/dossier_spec.rb                     |  2 +-
 spec/models/instructeur_spec.rb                 | 17 +++++++++++------
 7 files changed, 22 insertions(+), 17 deletions(-)

diff --git a/app/models/instructeur.rb b/app/models/instructeur.rb
index 6d4c93d4a..df7cde3c2 100644
--- a/app/models/instructeur.rb
+++ b/app/models/instructeur.rb
@@ -6,8 +6,8 @@ class Instructeur < ApplicationRecord
   before_validation -> { sanitize_email(:email) }
 
   has_many :assign_to, dependent: :destroy
-  has_many :procedures, through: :assign_to
   has_many :groupe_instructeurs, through: :assign_to
+  has_many :procedures, through: :groupe_instructeurs
 
   has_many :assign_to_with_email_notifications, -> { with_email_notifications }, class_name: 'AssignTo', inverse_of: :instructeur
   has_many :procedures_with_email_notifications, through: :assign_to_with_email_notifications, source: :procedure
@@ -64,7 +64,7 @@ class Instructeur < ApplicationRecord
   end
 
   def remove_from_procedure(procedure)
-    !!(procedure.in?(procedures) && procedures.destroy(procedure))
+    !!(procedure.defaut_groupe_instructeur.in?(groupe_instructeurs) && groupe_instructeurs.destroy(procedure.defaut_groupe_instructeur))
   end
 
   def last_week_overview
diff --git a/spec/controllers/instructeurs/procedures_controller_spec.rb b/spec/controllers/instructeurs/procedures_controller_spec.rb
index 898dd568b..48a19ccef 100644
--- a/spec/controllers/instructeurs/procedures_controller_spec.rb
+++ b/spec/controllers/instructeurs/procedures_controller_spec.rb
@@ -103,9 +103,9 @@ describe Instructeurs::ProceduresController, type: :controller do
         let(:procedure3) { create(:procedure) }
 
         before do
-          instructeur.procedures << procedure1
-          instructeur.procedures << procedure2
-          instructeur.procedures << procedure3
+          instructeur.groupe_instructeurs << procedure1.defaut_groupe_instructeur
+          instructeur.groupe_instructeurs << procedure2.defaut_groupe_instructeur
+          instructeur.groupe_instructeurs << procedure3.defaut_groupe_instructeur
           subject
         end
 
@@ -117,7 +117,7 @@ describe Instructeurs::ProceduresController, type: :controller do
         let(:dossier) { create(:dossier, state: state, procedure: procedure) }
 
         before do
-          instructeur.procedures << procedure
+          instructeur.groupe_instructeurs << procedure.defaut_groupe_instructeur
           dossier
         end
 
@@ -141,7 +141,7 @@ describe Instructeurs::ProceduresController, type: :controller do
             create(:dossier, procedure: procedure, state: Dossier.states.fetch(:en_instruction))
             create(:dossier, procedure: procedure, state: Dossier.states.fetch(:sans_suite), archived: true)
 
-            instructeur.procedures << procedure2
+            instructeur.groupe_instructeurs << procedure2.defaut_groupe_instructeur
             create(:dossier, :followed, procedure: procedure2, state: Dossier.states.fetch(:en_construction))
             create(:dossier, procedure: procedure2, state: Dossier.states.fetch(:accepte))
             instructeur.followed_dossiers << create(:dossier, procedure: procedure2, state: Dossier.states.fetch(:en_instruction))
diff --git a/spec/controllers/instructeurs/recherche_controller_spec.rb b/spec/controllers/instructeurs/recherche_controller_spec.rb
index 739952847..415939a1d 100644
--- a/spec/controllers/instructeurs/recherche_controller_spec.rb
+++ b/spec/controllers/instructeurs/recherche_controller_spec.rb
@@ -5,7 +5,7 @@ describe Instructeurs::RechercheController, type: :controller do
   let(:dossier2) { create(:dossier, :en_construction, procedure: dossier.procedure) }
   let(:instructeur) { create(:instructeur) }
 
-  before { instructeur.procedures << dossier2.procedure }
+  before { instructeur.groupe_instructeurs << dossier2.procedure.defaut_groupe_instructeur }
 
   describe 'GET #index' do
     before { sign_in(instructeur.user) }
diff --git a/spec/controllers/invites_controller_spec.rb b/spec/controllers/invites_controller_spec.rb
index d52193623..3a78d9fba 100644
--- a/spec/controllers/invites_controller_spec.rb
+++ b/spec/controllers/invites_controller_spec.rb
@@ -32,7 +32,7 @@ describe InvitesController, type: :controller do
 
       context 'when instructeur has access to dossier' do
         before do
-          signed_in_profile.instructeur.procedures << dossier.procedure
+          signed_in_profile.instructeur.groupe_instructeurs << dossier.procedure.defaut_groupe_instructeur
         end
 
         it_behaves_like "he can not create invitation"
diff --git a/spec/controllers/root_controller_spec.rb b/spec/controllers/root_controller_spec.rb
index cfaf12a94..0afe43037 100644
--- a/spec/controllers/root_controller_spec.rb
+++ b/spec/controllers/root_controller_spec.rb
@@ -17,7 +17,7 @@ describe RootController, type: :controller do
     let(:dossier) { create(:dossier, :en_construction, procedure: procedure) }
 
     before do
-      instructeur.procedures << procedure
+      instructeur.groupe_instructeurs << procedure.defaut_groupe_instructeur
       sign_in(instructeur.user)
     end
 
diff --git a/spec/models/dossier_spec.rb b/spec/models/dossier_spec.rb
index 8d56632c3..d7aeabab7 100644
--- a/spec/models/dossier_spec.rb
+++ b/spec/models/dossier_spec.rb
@@ -615,7 +615,7 @@ describe Dossier do
       let(:dossier) { create(:dossier, :en_construction, :followed) }
       let!(:non_following_instructeur) do
         non_following_instructeur = create(:instructeur)
-        non_following_instructeur.procedures << dossier.procedure
+        non_following_instructeur.groupe_instructeurs << dossier.procedure.defaut_groupe_instructeur
         non_following_instructeur
       end
 
diff --git a/spec/models/instructeur_spec.rb b/spec/models/instructeur_spec.rb
index 30b8d026a..281123918 100644
--- a/spec/models/instructeur_spec.rb
+++ b/spec/models/instructeur_spec.rb
@@ -151,17 +151,21 @@ describe Instructeur, type: :model do
     after { Timecop.return }
 
     context 'when no procedure published was active last week' do
-      let!(:procedure) { create(:procedure, :published, instructeurs: [instructeur2], libelle: 'procedure') }
+      let!(:procedure) { create(:procedure, :published, libelle: 'procedure') }
+
+      before { instructeur2.assign_to_procedure(procedure) }
+
       context 'when the instructeur has no notifications' do
         it { is_expected.to eq(nil) }
       end
     end
 
     context 'when a procedure published was active' do
-      let!(:procedure) { create(:procedure, :published, instructeurs: [instructeur2], libelle: 'procedure') }
+      let!(:procedure) { create(:procedure, :published, libelle: 'procedure') }
       let(:procedure_overview) { double('procedure_overview', 'had_some_activities?'.to_sym => true) }
 
       before :each do
+        instructeur2.assign_to_procedure(procedure)
         expect_any_instance_of(Procedure).to receive(:procedure_overview).and_return(procedure_overview)
       end
 
@@ -169,10 +173,11 @@ describe Instructeur, type: :model do
     end
 
     context 'when a procedure not published was active with no notifications' do
-      let!(:procedure) { create(:procedure, instructeurs: [instructeur2], libelle: 'procedure') }
+      let!(:procedure) { create(:procedure, libelle: 'procedure') }
       let(:procedure_overview) { double('procedure_overview', 'had_some_activities?'.to_sym => true) }
 
       before :each do
+        instructeur2.assign_to_procedure(procedure)
         allow_any_instance_of(Procedure).to receive(:procedure_overview).and_return(procedure_overview)
       end
 
@@ -259,7 +264,7 @@ describe Instructeur, type: :model do
     let!(:dossier) { create(:dossier, :followed, state: Dossier.states.fetch(:en_construction)) }
     let(:instructeur) { dossier.follows.first.instructeur }
     let(:procedure) { dossier.procedure }
-    let!(:instructeur_2) { create(:instructeur, procedures: [procedure]) }
+    let!(:instructeur_2) { create(:instructeur, groupe_instructeurs: [procedure.defaut_groupe_instructeur]) }
 
     let!(:dossier_on_procedure_2) { create(:dossier, :followed, state: Dossier.states.fetch(:en_construction)) }
     let!(:instructeur_on_procedure_2) { dossier_on_procedure_2.follows.first.instructeur }
@@ -388,7 +393,7 @@ describe Instructeur, type: :model do
     let(:procedure_to_assign) { create(:procedure) }
 
     before do
-      create(:assign_to, instructeur: instructeur, procedure: procedure_to_assign, email_notifications_enabled: true)
+      create(:assign_to, instructeur: instructeur, procedure: procedure_to_assign, email_notifications_enabled: true, groupe_instructeur: procedure_to_assign.defaut_groupe_instructeur)
     end
 
     context 'when a dossier in construction exists' do
@@ -433,6 +438,6 @@ describe Instructeur, type: :model do
   private
 
   def assign(procedure_to_assign)
-    create :assign_to, instructeur: instructeur, procedure: procedure_to_assign
+    create :assign_to, instructeur: instructeur, procedure: procedure_to_assign, groupe_instructeur: procedure_to_assign.defaut_groupe_instructeur
   end
 end

From 64d28dba2f1829fc760a94d213c06ac3076703d9 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Wed, 21 Aug 2019 13:53:27 +0200
Subject: [PATCH 07/14] cleaning : use missing_steps

---
 app/views/admin/procedures/show.html.haml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/views/admin/procedures/show.html.haml b/app/views/admin/procedures/show.html.haml
index c88467779..94ae06481 100644
--- a/app/views/admin/procedures/show.html.haml
+++ b/app/views/admin/procedures/show.html.haml
@@ -5,7 +5,7 @@
     = render partial: '/admin/procedures/modal_transfer'
 
     - if @procedure.brouillon?
-      - if @procedure.instructeurs.empty? || @procedure.service.nil?
+      - if @procedure.missing_steps.present?
         - missing_elements = []
         - if @procedure.instructeurs.empty?
           - missing_elements << 'des instructeurs'
@@ -59,7 +59,7 @@
           %br
           Attention, diffusez toujours le <strong>lien complet</strong> affiché ci-dessus, et non pas un lien générique vers demarches-simplifiees.fr. Ne dites pas non plus aux usagers de se rendre sur le site générique demarches-simplifiees.fr, donnez-leur toujours le lien complet.
         - elsif @procedure.brouillon_avec_lien?
-          - if @procedure.instructeurs.present? && @procedure.service.present?
+          - if @procedure.missing_steps.empty?
             %p
               Cette démarche est actuellement <strong>en test</strong>,
               pour y accéder vous pouvez utiliser le lien :

From ba24958035309e1120095f149ce257e7346dbd39 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Wed, 21 Aug 2019 13:53:53 +0200
Subject: [PATCH 08/14] Procedure add missing_instructeurs?

---
 app/models/procedure.rb                   |  6 +++++-
 app/views/admin/procedures/show.html.haml |  6 +++---
 spec/models/procedure_spec.rb             | 16 ++++++++++++++++
 3 files changed, 24 insertions(+), 4 deletions(-)

diff --git a/app/models/procedure.rb b/app/models/procedure.rb
index 56a4637ec..0a82131df 100644
--- a/app/models/procedure.rb
+++ b/app/models/procedure.rb
@@ -411,7 +411,7 @@ class Procedure < ApplicationRecord
       result << :service
     end
 
-    if instructeurs.empty?
+    if missing_instructeurs?
       result << :instructeurs
     end
 
@@ -482,6 +482,10 @@ class Procedure < ApplicationRecord
     groupe_instructeurs.find_by(label: GroupeInstructeur::DEFAULT_LABEL)
   end
 
+  def missing_instructeurs?
+    !AssignTo.exists?(groupe_instructeur: groupe_instructeurs)
+  end
+
   private
 
   def move_type_de_champ_attributes(types_de_champ, type_de_champ, new_index)
diff --git a/app/views/admin/procedures/show.html.haml b/app/views/admin/procedures/show.html.haml
index 94ae06481..f1e353499 100644
--- a/app/views/admin/procedures/show.html.haml
+++ b/app/views/admin/procedures/show.html.haml
@@ -7,7 +7,7 @@
     - if @procedure.brouillon?
       - if @procedure.missing_steps.present?
         - missing_elements = []
-        - if @procedure.instructeurs.empty?
+        - if @procedure.missing_instructeurs?
           - missing_elements << 'des instructeurs'
         - if @procedure.service.nil?
           - missing_elements << 'un service'
@@ -106,9 +106,9 @@
           - else
             .alert.alert-info
               Pour pouvoir tester cette démarche, vous devez d’abord lui affecter
-              - if @procedure.instructeurs.empty?
+              - if @procedure.missing_instructeurs?
                 = link_to("des instructeurs", admin_procedure_assigns_path(@procedure))
-              - if @procedure.instructeurs.empty? && @procedure.service.nil?
+              - if @procedure.missing_instructeurs? && @procedure.service.nil?
                 et
               - if @procedure.service.nil?
                 = link_to("un service", services_path(procedure_id: @procedure))
diff --git a/spec/models/procedure_spec.rb b/spec/models/procedure_spec.rb
index 741a87de7..ab4dad111 100644
--- a/spec/models/procedure_spec.rb
+++ b/spec/models/procedure_spec.rb
@@ -889,4 +889,20 @@ describe Procedure do
     it { expect(procedure.groupe_instructeurs.count).to eq(1) }
     it { expect(procedure.groupe_instructeurs.first.label).to eq(GroupeInstructeur::DEFAULT_LABEL) }
   end
+
+  describe '.missing_instructeurs?' do
+    let!(:procedure) { create(:procedure) }
+
+    subject { procedure.missing_instructeurs? }
+
+    it { is_expected.to be true }
+
+    context 'when an instructeur is assign to this procedure' do
+      let!(:instructeur) { create(:instructeur) }
+
+      before { instructeur.assign_to_procedure(procedure) }
+
+      it { is_expected.to be false }
+    end
+  end
 end

From b541ef0b41483cb6550c92946da806af8f5c7b6d Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Wed, 21 Aug 2019 14:09:56 +0200
Subject: [PATCH 09/14] Procedure has many instructeurs with
 defaut_groupe_instructeur

---
 app/controllers/admin/assigns_controller.rb           | 2 +-
 app/controllers/instructeurs/dossiers_controller.rb   | 2 +-
 app/controllers/instructeurs/procedures_controller.rb | 2 +-
 app/helpers/dossier_link_helper.rb                    | 2 +-
 spec/controllers/admin/procedures_controller_spec.rb  | 2 +-
 spec/factories/procedure.rb                           | 4 ++--
 spec/features/instructeurs/instructeur_spec.rb        | 2 +-
 spec/helpers/dossier_link_helper_spec.rb              | 2 +-
 spec/models/procedure_spec.rb                         | 2 +-
 spec/views/admin/assigns/show.html.haml_spec.rb       | 2 +-
 10 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/app/controllers/admin/assigns_controller.rb b/app/controllers/admin/assigns_controller.rb
index c295076af..fca3aabc6 100644
--- a/app/controllers/admin/assigns_controller.rb
+++ b/app/controllers/admin/assigns_controller.rb
@@ -8,7 +8,7 @@ class Admin::AssignsController < AdminController
   NOT_ASSIGN = 'not_assign'
 
   def show
-    assign_scope = @procedure.instructeurs
+    assign_scope = @procedure.defaut_groupe_instructeur.instructeurs
 
     @instructeurs_assign = smart_listing_create :instructeurs_assign,
       assign_scope,
diff --git a/app/controllers/instructeurs/dossiers_controller.rb b/app/controllers/instructeurs/dossiers_controller.rb
index 2f676a371..cdfb3f60a 100644
--- a/app/controllers/instructeurs/dossiers_controller.rb
+++ b/app/controllers/instructeurs/dossiers_controller.rb
@@ -56,7 +56,7 @@ module Instructeurs
       @previous_following_instructeurs_emails = previous_followers.pluck(:email)
       @avis_emails = dossier.avis.includes(:instructeur).map(&:email_to_display)
       @invites_emails = dossier.invites.map(&:email)
-      @potential_recipients = procedure.instructeurs.reject { |g| g == current_instructeur }
+      @potential_recipients = procedure.defaut_groupe_instructeur.instructeurs.reject { |g| g == current_instructeur }
     end
 
     def send_to_instructeurs
diff --git a/app/controllers/instructeurs/procedures_controller.rb b/app/controllers/instructeurs/procedures_controller.rb
index 71624ab2e..f900a5bf0 100644
--- a/app/controllers/instructeurs/procedures_controller.rb
+++ b/app/controllers/instructeurs/procedures_controller.rb
@@ -221,7 +221,7 @@ module Instructeurs
     end
 
     def ensure_ownership!
-      if !procedure.instructeurs.include?(current_instructeur)
+      if !procedure.defaut_groupe_instructeur.instructeurs.include?(current_instructeur)
         flash[:alert] = "Vous n'avez pas accès à cette démarche"
         redirect_to root_path
       end
diff --git a/app/helpers/dossier_link_helper.rb b/app/helpers/dossier_link_helper.rb
index 15ad75f12..1a3986771 100644
--- a/app/helpers/dossier_link_helper.rb
+++ b/app/helpers/dossier_link_helper.rb
@@ -1,7 +1,7 @@
 module DossierLinkHelper
   def dossier_linked_path(user, dossier)
     if user.is_a?(Instructeur)
-      if dossier.procedure.instructeurs.include?(user)
+      if dossier.procedure.defaut_groupe_instructeur.instructeurs.include?(user)
         instructeur_dossier_path(dossier.procedure, dossier)
       else
         avis = dossier.avis.find_by(instructeur: user)
diff --git a/spec/controllers/admin/procedures_controller_spec.rb b/spec/controllers/admin/procedures_controller_spec.rb
index e32b5ab32..0fe3420d4 100644
--- a/spec/controllers/admin/procedures_controller_spec.rb
+++ b/spec/controllers/admin/procedures_controller_spec.rb
@@ -218,7 +218,7 @@ describe Admin::ProceduresController, type: :controller do
 
         describe "admin can also instruct the procedure as a instructeur" do
           subject { Procedure.last }
-          it { expect(subject.instructeurs).to include(instructeur) }
+          it { expect(subject.defaut_groupe_instructeur.instructeurs).to include(instructeur) }
         end
       end
     end
diff --git a/spec/factories/procedure.rb b/spec/factories/procedure.rb
index ef845b6d7..39f3583c2 100644
--- a/spec/factories/procedure.rb
+++ b/spec/factories/procedure.rb
@@ -61,8 +61,8 @@ FactoryBot.define do
     end
 
     trait :with_instructeur do
-      after(:build) do |procedure, _evaluator|
-        procedure.instructeurs << create(:instructeur)
+      after(:create) do |procedure, _evaluator|
+        procedure.defaut_groupe_instructeur.instructeurs << create(:instructeur)
       end
     end
 
diff --git a/spec/features/instructeurs/instructeur_spec.rb b/spec/features/instructeurs/instructeur_spec.rb
index ddef26937..d6ce007a5 100644
--- a/spec/features/instructeurs/instructeur_spec.rb
+++ b/spec/features/instructeurs/instructeur_spec.rb
@@ -171,7 +171,7 @@ feature 'The instructeur part' do
   scenario 'A instructeur can send a dossier to several instructeurs', js: true do
     instructeur_2 = FactoryBot.create(:instructeur)
     instructeur_3 = FactoryBot.create(:instructeur)
-    procedure.instructeurs << [instructeur_2, instructeur_3]
+    procedure.defaut_groupe_instructeur.instructeurs << [instructeur_2, instructeur_3]
 
     send_dossier = double()
     expect(InstructeurMailer).to receive(:send_dossier).and_return(send_dossier).twice
diff --git a/spec/helpers/dossier_link_helper_spec.rb b/spec/helpers/dossier_link_helper_spec.rb
index 7190603a7..2c7db4af1 100644
--- a/spec/helpers/dossier_link_helper_spec.rb
+++ b/spec/helpers/dossier_link_helper_spec.rb
@@ -18,7 +18,7 @@ describe DossierLinkHelper do
       let(:dossier) { create(:dossier) }
       let(:instructeur) { create(:instructeur) }
 
-      before { dossier.procedure.instructeurs << instructeur }
+      before { dossier.procedure.defaut_groupe_instructeur.instructeurs << instructeur }
 
       it { expect(helper.dossier_linked_path(instructeur, dossier)).to eq(instructeur_dossier_path(dossier.procedure, dossier)) }
     end
diff --git a/spec/models/procedure_spec.rb b/spec/models/procedure_spec.rb
index ab4dad111..0ddc65d61 100644
--- a/spec/models/procedure_spec.rb
+++ b/spec/models/procedure_spec.rb
@@ -401,7 +401,7 @@ describe Procedure do
     subject { @procedure }
 
     it { expect(subject.parent_procedure).to eq(procedure) }
-    it { expect(subject.instructeurs.pluck(:email)).to eq([administrateur.email]) }
+    it { expect(subject.defaut_groupe_instructeur.instructeurs.pluck(:email)).to eq([administrateur.email]) }
 
     it 'should duplicate specific objects with different id' do
       expect(subject.id).not_to eq(procedure.id)
diff --git a/spec/views/admin/assigns/show.html.haml_spec.rb b/spec/views/admin/assigns/show.html.haml_spec.rb
index 3136d94a7..6c0342922 100644
--- a/spec/views/admin/assigns/show.html.haml_spec.rb
+++ b/spec/views/admin/assigns/show.html.haml_spec.rb
@@ -4,7 +4,7 @@ describe 'admin/assigns/show.html.haml', type: :view do
   let(:admin) { create(:administrateur) }
   let(:procedure) { create :procedure, administrateur: admin }
 
-  let(:assign_instructeurs) { procedure.instructeurs }
+  let(:assign_instructeurs) { procedure.defaut_groupe_instructeur.instructeurs }
   let(:not_assign_instructeurs) { admin.instructeurs.where.not(id: assign_instructeurs.ids) }
 
   before do

From 31cb056351a3dc01eb34939eb8328803205db922 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Wed, 21 Aug 2019 14:25:21 +0200
Subject: [PATCH 10/14] Procedure: remove direct instructeurs access

Add shortcut for the spec to assign instructeurs directly to the defaut groupe
---
 app/models/procedure.rb     | 2 --
 spec/factories/procedure.rb | 5 +++++
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/app/models/procedure.rb b/app/models/procedure.rb
index 0a82131df..9ea61980b 100644
--- a/app/models/procedure.rb
+++ b/app/models/procedure.rb
@@ -14,10 +14,8 @@ class Procedure < ApplicationRecord
   belongs_to :parent_procedure, class_name: 'Procedure'
   belongs_to :service
 
-  has_many :assign_to, dependent: :destroy
   has_many :administrateurs_procedures
   has_many :administrateurs, through: :administrateurs_procedures, after_remove: -> (procedure, _admin) { procedure.validate! }
-  has_many :instructeurs, through: :assign_to
   has_many :groupe_instructeurs, dependent: :destroy
 
   has_one :initiated_mail, class_name: "Mails::InitiatedMail", dependent: :destroy
diff --git a/spec/factories/procedure.rb b/spec/factories/procedure.rb
index 39f3583c2..6136d005b 100644
--- a/spec/factories/procedure.rb
+++ b/spec/factories/procedure.rb
@@ -14,6 +14,7 @@ FactoryBot.define do
 
     transient do
       administrateur {}
+      instructeurs { [] }
     end
 
     after(:build) do |procedure, evaluator|
@@ -24,6 +25,10 @@ FactoryBot.define do
       end
     end
 
+    after(:create) do |procedure, evaluator|
+      evaluator.instructeurs.each { |i| i.assign_to_procedure(procedure) }
+    end
+
     factory :procedure_with_dossiers do
       transient do
         dossiers_count { 1 }

From caf1fbca4415e6148b2d72b2a8bbeae773527d64 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Wed, 21 Aug 2019 16:16:53 +0200
Subject: [PATCH 11/14] Use assign_to

---
 spec/features/instructeurs/procedure_filters_spec.rb | 4 ++--
 spec/models/dossier_spec.rb                          | 4 ++--
 spec/services/dossier_search_service_spec.rb         | 4 ++--
 spec/views/admin/procedures/show.html.haml_spec.rb   | 2 +-
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/spec/features/instructeurs/procedure_filters_spec.rb b/spec/features/instructeurs/procedure_filters_spec.rb
index 544c0a9c3..5753111bd 100644
--- a/spec/features/instructeurs/procedure_filters_spec.rb
+++ b/spec/features/instructeurs/procedure_filters_spec.rb
@@ -1,8 +1,8 @@
 require "spec_helper"
 
 feature "procedure filters" do
-  let(:procedure) { create(:procedure, :published, :with_type_de_champ) }
-  let(:instructeur) { create(:instructeur, procedures: [procedure]) }
+  let(:instructeur) { create(:instructeur) }
+  let(:procedure) { create(:procedure, :published, :with_type_de_champ, instructeurs: [instructeur]) }
   let!(:type_de_champ) { procedure.types_de_champ.first }
   let!(:new_unfollow_dossier) { create(:dossier, procedure: procedure, state: Dossier.states.fetch(:en_instruction)) }
   let!(:champ) { Champ.find_by(type_de_champ_id: type_de_champ.id, dossier_id: new_unfollow_dossier.id) }
diff --git a/spec/models/dossier_spec.rb b/spec/models/dossier_spec.rb
index d7aeabab7..eceea782a 100644
--- a/spec/models/dossier_spec.rb
+++ b/spec/models/dossier_spec.rb
@@ -235,10 +235,10 @@ describe Dossier do
   end
 
   describe '#avis_for' do
-    let!(:procedure) { create(:procedure, :published) }
+    let!(:instructeur) { create(:instructeur) }
+    let!(:procedure) { create(:procedure, :published, instructeurs: [instructeur]) }
     let!(:dossier) { create(:dossier, procedure: procedure, state: Dossier.states.fetch(:en_construction)) }
 
-    let!(:instructeur) { create(:instructeur, procedures: [procedure]) }
     let!(:expert_1) { create(:instructeur) }
     let!(:expert_2) { create(:instructeur) }
 
diff --git a/spec/services/dossier_search_service_spec.rb b/spec/services/dossier_search_service_spec.rb
index c89473b85..248b84e2e 100644
--- a/spec/services/dossier_search_service_spec.rb
+++ b/spec/services/dossier_search_service_spec.rb
@@ -15,8 +15,8 @@ describe DossierSearchService do
     let(:instructeur_2) { create(:instructeur, administrateurs: [administrateur_2]) }
 
     before do
-      create :assign_to, instructeur: instructeur_1, procedure: procedure_1
-      create :assign_to, instructeur: instructeur_2, procedure: procedure_2
+      instructeur_1.assign_to_procedure(procedure_1)
+      instructeur_2.assign_to_procedure(procedure_2)
     end
 
     let(:procedure_1) { create(:procedure, :published, administrateur: administrateur_1) }
diff --git a/spec/views/admin/procedures/show.html.haml_spec.rb b/spec/views/admin/procedures/show.html.haml_spec.rb
index b2fb5dc17..d1cc03946 100644
--- a/spec/views/admin/procedures/show.html.haml_spec.rb
+++ b/spec/views/admin/procedures/show.html.haml_spec.rb
@@ -23,7 +23,7 @@ describe 'admin/procedures/show.html.haml', type: :view do
 
     context 'when procedure have a gestionnare affected' do
       before do
-        create :assign_to, instructeur: create(:instructeur), procedure: procedure
+        create(:instructeur).assign_to_procedure(procedure)
         render
       end
 

From a2061fba794b469325091cc67666f46da1eb08ad Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Wed, 21 Aug 2019 16:17:02 +0200
Subject: [PATCH 12/14] wording

---
 spec/views/admin/procedures/show.html.haml_spec.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/spec/views/admin/procedures/show.html.haml_spec.rb b/spec/views/admin/procedures/show.html.haml_spec.rb
index d1cc03946..f98d1eb91 100644
--- a/spec/views/admin/procedures/show.html.haml_spec.rb
+++ b/spec/views/admin/procedures/show.html.haml_spec.rb
@@ -9,7 +9,7 @@ describe 'admin/procedures/show.html.haml', type: :view do
   end
 
   describe 'procedure is draft' do
-    context 'when procedure does not have a gestionnare affected' do
+    context 'when procedure does not have a instructeur affected' do
       before do
         render
       end
@@ -21,7 +21,7 @@ describe 'admin/procedures/show.html.haml', type: :view do
       end
     end
 
-    context 'when procedure have a gestionnare affected' do
+    context 'when procedure have a instructeur affected' do
       before do
         create(:instructeur).assign_to_procedure(procedure)
         render

From a82d49aa607ee6abfbf9f875ee3f405053cc7359 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Wed, 21 Aug 2019 16:33:26 +0200
Subject: [PATCH 13/14] Procedure: save procedure while cloning

This is required to be able to assign the administrateur.instructeur to the defaut_groupe_instructeur of the new procedure
---
 app/controllers/admin/procedures_controller.rb | 6 ++----
 app/models/procedure.rb                        | 2 ++
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/controllers/admin/procedures_controller.rb b/app/controllers/admin/procedures_controller.rb
index 31032bcb5..0b05d2824 100644
--- a/app/controllers/admin/procedures_controller.rb
+++ b/app/controllers/admin/procedures_controller.rb
@@ -136,9 +136,7 @@ class Admin::ProceduresController < AdminController
       render '/admin/procedures/transfer', formats: 'js', status: 404
     else
       procedure = current_administrateur.procedures.find(params[:procedure_id])
-      clone_procedure = procedure.clone(admin, false)
-
-      clone_procedure.save
+      procedure.clone(admin, false)
 
       flash.now.notice = "La démarche a correctement été clonée vers le nouvel administrateur."
 
@@ -162,7 +160,7 @@ class Admin::ProceduresController < AdminController
     procedure = Procedure.find(params[:procedure_id])
     new_procedure = procedure.clone(current_administrateur, cloned_from_library?)
 
-    if new_procedure.save
+    if new_procedure.valid?
       flash.notice = 'Démarche clonée'
       redirect_to edit_admin_procedure_path(id: new_procedure.id)
     else
diff --git a/app/models/procedure.rb b/app/models/procedure.rb
index 9ea61980b..a340bf53a 100644
--- a/app/models/procedure.rb
+++ b/app/models/procedure.rb
@@ -247,6 +247,8 @@ class Procedure < ApplicationRecord
       procedure.service = self.service.clone_and_assign_to_administrateur(admin)
     end
 
+    procedure.save
+
     admin.instructeur.assign_to_procedure(procedure)
 
     procedure

From eb75a563b874b26082d00f3ffa1d1454f0d60803 Mon Sep 17 00:00:00 2001
From: simon lehericey <mail@simon.lehericey.net>
Date: Wed, 21 Aug 2019 16:51:30 +0200
Subject: [PATCH 14/14] Fix old after_party spec

---
 ...20190117154829_delete_dossiers_without_procedure.rake_spec.rb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/spec/lib/tasks/deployment/20190117154829_delete_dossiers_without_procedure.rake_spec.rb b/spec/lib/tasks/deployment/20190117154829_delete_dossiers_without_procedure.rake_spec.rb
index 89cdf9b52..fd530c1e1 100644
--- a/spec/lib/tasks/deployment/20190117154829_delete_dossiers_without_procedure.rake_spec.rb
+++ b/spec/lib/tasks/deployment/20190117154829_delete_dossiers_without_procedure.rake_spec.rb
@@ -19,6 +19,7 @@ describe '20190117154829_delete_dossiers_without_procedure.rake' do
 
     before do
       procedure_2_dossier_ids
+      procedure2.defaut_groupe_instructeur.delete
       procedure2.delete
       expect(procedure_2_dossier_ids.count).to eq(2)
       expect(Dossier.find_by(id: procedure_2_dossier_ids.first).procedure).to be nil