update routee? method
This commit is contained in:
parent
110953e450
commit
5578115acf
6 changed files with 10 additions and 20 deletions
|
@ -1,7 +1,7 @@
|
||||||
.fr-col-6.fr-col-md-4.fr-col-lg-3
|
.fr-col-6.fr-col-md-4.fr-col-lg-3
|
||||||
= link_to admin_procedure_groupe_instructeurs_path(@procedure), id: 'groupe-instructeurs', class: 'fr-tile fr-enlarge-link' do
|
= link_to admin_procedure_groupe_instructeurs_path(@procedure), id: 'groupe-instructeurs', class: 'fr-tile fr-enlarge-link' do
|
||||||
.fr-tile__body.flex.justify-between
|
.fr-tile__body.flex.justify-between
|
||||||
- if @procedure.routee? || @procedure.instructeurs.size > 1
|
- if @procedure.routee?
|
||||||
%div
|
%div
|
||||||
%span.icon.accept
|
%span.icon.accept
|
||||||
%p.fr-tile-status-accept Validé
|
%p.fr-tile-status-accept Validé
|
||||||
|
@ -17,6 +17,6 @@
|
||||||
%p.fr-tag= @procedure.instructeurs.size
|
%p.fr-tag= @procedure.instructeurs.size
|
||||||
|
|
||||||
%h3.fr-h6
|
%h3.fr-h6
|
||||||
= @procedure.routee? ? t('.routee.title', count: @procedure.groupe_instructeurs.size) : t('.title', count: @procedure.instructeurs.size)
|
= @procedure.groupe_instructeurs.many? ? t('.routee.title', count: @procedure.groupe_instructeurs.size) : t('.title', count: @procedure.instructeurs.size)
|
||||||
%p.fr-tile-subtitle Suivi des dossiers
|
%p.fr-tile-subtitle Suivi des dossiers
|
||||||
%p.fr-btn.fr-btn--tertiary= t('views.shared.actions.edit')
|
%p.fr-btn.fr-btn--tertiary= t('views.shared.actions.edit')
|
||||||
|
|
|
@ -432,7 +432,7 @@ module Users
|
||||||
end
|
end
|
||||||
|
|
||||||
def should_fill_groupe_instructeur?
|
def should_fill_groupe_instructeur?
|
||||||
(!@dossier.procedure.routee? || @dossier.procedure.groupe_instructeurs.actif.one?) && @dossier.groupe_instructeur_id.nil?
|
!@dossier.procedure.routee? && @dossier.groupe_instructeur_id.nil?
|
||||||
end
|
end
|
||||||
|
|
||||||
def defaut_groupe_instructeur
|
def defaut_groupe_instructeur
|
||||||
|
|
|
@ -619,7 +619,7 @@ class Dossier < ApplicationRecord
|
||||||
end
|
end
|
||||||
|
|
||||||
def show_groupe_instructeur_selector?
|
def show_groupe_instructeur_selector?
|
||||||
procedure.routee? && !procedure.feature_enabled?(:procedure_routage_api) && procedure.groupe_instructeurs.actif.many?
|
procedure.routee? && !procedure.feature_enabled?(:procedure_routage_api)
|
||||||
end
|
end
|
||||||
|
|
||||||
def assign_to_groupe_instructeur(groupe_instructeur, author = nil)
|
def assign_to_groupe_instructeur(groupe_instructeur, author = nil)
|
||||||
|
|
|
@ -695,7 +695,7 @@ class Procedure < ApplicationRecord
|
||||||
end
|
end
|
||||||
|
|
||||||
def routee?
|
def routee?
|
||||||
routing_enabled? || groupe_instructeurs.size > 1
|
routing_enabled?
|
||||||
end
|
end
|
||||||
|
|
||||||
def instructeurs_self_management?
|
def instructeurs_self_management?
|
||||||
|
@ -703,7 +703,7 @@ class Procedure < ApplicationRecord
|
||||||
end
|
end
|
||||||
|
|
||||||
def defaut_groupe_instructeur_for_new_dossier
|
def defaut_groupe_instructeur_for_new_dossier
|
||||||
if !routee? || feature_enabled?(:procedure_routage_api) || (routee? && self.groupe_instructeurs.actif.size == 1)
|
if !routee? || feature_enabled?(:procedure_routage_api)
|
||||||
defaut_groupe_instructeur
|
defaut_groupe_instructeur
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -83,6 +83,7 @@ describe TagsSubstitutionConcern, type: :model do
|
||||||
gi.dossiers << dossier
|
gi.dossiers << dossier
|
||||||
dossier.update(groupe_instructeur: gi)
|
dossier.update(groupe_instructeur: gi)
|
||||||
dossier.reload
|
dossier.reload
|
||||||
|
procedure.reload
|
||||||
end
|
end
|
||||||
|
|
||||||
it { expect(procedure.routee?).to eq(true) }
|
it { expect(procedure.routee?).to eq(true) }
|
||||||
|
|
|
@ -62,20 +62,9 @@ describe 'shared/dossiers/champs.html.haml', type: :view do
|
||||||
let(:dossier) { create(:dossier, procedure: procedure) }
|
let(:dossier) { create(:dossier, procedure: procedure) }
|
||||||
let(:champs) { [] }
|
let(:champs) { [] }
|
||||||
|
|
||||||
it "does not render the routing criteria name and its value" do
|
it "renders the routing criteria name and its value" do
|
||||||
expect(subject).not_to include(procedure.routing_criteria_name)
|
expect(subject).to include(procedure.routing_criteria_name)
|
||||||
expect(subject).not_to include(dossier.procedure.defaut_groupe_instructeur.label)
|
expect(subject).to include(dossier.groupe_instructeur.label)
|
||||||
end
|
|
||||||
|
|
||||||
context "with selected groupe instructeur" do
|
|
||||||
before do
|
|
||||||
dossier.groupe_instructeur = dossier.procedure.defaut_groupe_instructeur
|
|
||||||
end
|
|
||||||
|
|
||||||
it "renders the routing criteria name and its value" do
|
|
||||||
expect(subject).to include(procedure.routing_criteria_name)
|
|
||||||
expect(subject).to include(dossier.groupe_instructeur.label)
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
|
|
||||||
context "with seen_at" do
|
context "with seen_at" do
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue