DGNum/demarches-normaliennes
15
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

Merge pull request #5256 from tchak/remove-dossiers-procedure_id-1

Browse source 
Prepare to remove dossier procedure_id
This commit is contained in:
Paul Chavard 2020-06-11 16:32:51 +02:00 committed by GitHub
commit d3fc810d23
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
6 changed files with 4 additions and 55 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/controllers/champs/siret_controller.rb
View file

@ -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

2
app/controllers/instructeurs/recherche_controller.rb
View file

@ -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

2
app/jobs/etablissement_update_job.rb
View file

@ -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

2
app/models/dossier.rb
View file

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

15
lib/tasks/deployment/20190826153115_link_dossier_and_groupe_instructeur.rake
View file

@ -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

36
spec/lib/tasks/deployment/20190826153115_link_dossier_and_groupe_instructeur.rake_spec.rb
View file

@ -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