Merge pull request #5260 from betagouv/dev

2020-06-11-03

app/controllers/champs/siret_controller.rb

@@ -46,7 +46,7 @@ class Champs::SiretController < ApplicationController
       @champ = policy_scope(Champ).find(params[:champ_id])
       @etablissement = @champ&.etablissement
     end
-    @procedure_id = @champ&.dossier&.procedure_id || 'aperçu'
+    @procedure_id = @champ&.dossier&.procedure&.id || 'aperçu'
   end
 
   def find_etablissement_with_siret

app/controllers/instructeurs/recherche_controller.rb

@@ -5,7 +5,7 @@ module Instructeurs
       @dossiers = DossierSearchService.matching_dossiers_for_instructeur(@search_terms, current_instructeur)
       @followed_dossiers_id = current_instructeur
         .followed_dossiers
-        .where(procedure_id: @dossiers.pluck(:procedure_id))
+        .where(groupe_instructeur_id: @dossiers.pluck(:groupe_instructeur_id))
         .pluck(:id)
     end
   end

app/jobs/etablissement_update_job.rb

@@ -1,7 +1,7 @@
 class EtablissementUpdateJob < ApplicationJob
   def perform(dossier, siret)
     begin
-      etablissement_attributes = ApiEntrepriseService.get_etablissement_params_for_siret(siret, dossier.procedure_id)
+      etablissement_attributes = ApiEntrepriseService.get_etablissement_params_for_siret(siret, dossier.procedure.id)
     rescue
       return
     end

app/lib/api_entreprise/api.rb

@@ -78,7 +78,7 @@ class ApiEntreprise::API
     elsif response.code == 400
       raise BadFormatRequest, "url:  #{url}"
     else
-      raise RequestFailed, "HTTP Error Code: #{response.code} for #{url}"
+      raise RequestFailed, "HTTP Error Code: #{response.code} for #{url}\nheaders: #{response.headers}\nbody: #{response.body}"
     end
   end
 

app/models/dossier.rb

@@ -1,5 +1,5 @@
 class Dossier < ApplicationRecord
-  self.ignored_columns = ['json_latlngs']
+  self.ignored_columns = ['procedure_id']
   include DossierFilteringConcern
 
   include Discard::Model

lib/tasks/deployment/20190826153115_link_dossier_and_groupe_instructeur.rake

@@ -1,15 +0,0 @@
-namespace :after_party do
-  desc 'Deployment task: link_dossier_and_groupe_instructeur'
-  task link_dossier_and_groupe_instructeur: :environment do
-    sql = <<~SQL
-      UPDATE dossiers AS d1 SET groupe_instructeur_id = g.id
-      FROM groupe_instructeurs AS g
-      WHERE g.procedure_id = d1.procedure_id
-        and d1.groupe_instructeur_id is null;
-    SQL
-
-    ActiveRecord::Base.connection.execute(sql)
-
-    AfterParty::TaskRecord.create version: '20190826153115'
-  end
-end

spec/lib/tasks/deployment/20190826153115_link_dossier_and_groupe_instructeur.rake_spec.rb

@@ -1,36 +0,0 @@
-describe '20190826153115_link_dossier_and_groupe_instructeur.rake' do
-  let(:rake_task) { Rake::Task['after_party:link_dossier_and_groupe_instructeur'] }
-
-  subject { rake_task.invoke }
-  after { rake_task.reenable }
-
-  context 'with 3 dossiers' do
-    let!(:procedure) { create(:procedure) }
-    let!(:procedure2) { create(:procedure) }
-    let!(:other_procedure_needed_to_create_dossier) { create(:procedure) }
-    let!(:other_gi) { other_procedure_needed_to_create_dossier.defaut_groupe_instructeur }
-    let!(:dossier) { Dossier.create(user: create(:user), procedure_id: procedure.id, groupe_instructeur: other_gi) }
-    let!(:dossier2) { Dossier.create(user: create(:user), procedure_id: procedure2.id, groupe_instructeur: other_gi) }
-    let!(:dossier3) { Dossier.create(user: create(:user), procedure_id: procedure2.id, groupe_instructeur: other_gi) }
-
-    before do
-      [dossier, dossier2, dossier3].each do |d|
-        d.update_column('groupe_instructeur_id', nil)
-      end
-
-      other_procedure_needed_to_create_dossier.groupe_instructeurs.destroy_all
-      other_procedure_needed_to_create_dossier.destroy
-    end
-
-    it do
-      expect(dossier.reload.groupe_instructeur_id).to be_nil
-      subject
-      expect(Dossier.count).to eq(3)
-      expect(Procedure.count).to eq(2)
-      expect(GroupeInstructeur.count).to eq(2)
-      expect(dossier.reload.groupe_instructeur_id).to eq(procedure.defaut_groupe_instructeur.id)
-      expect(dossier2.reload.groupe_instructeur_id).to eq(procedure2.defaut_groupe_instructeur.id)
-      expect(dossier3.reload.groupe_instructeur_id).to eq(procedure2.defaut_groupe_instructeur.id)
-    end
-  end
-end