Browse Source

displays managing agents of merged organisation selected as owning org

pull/1819/head
Kat 3 years ago
parent
commit
ae9ea5707b
  1. 6
      app/models/form/lettings/questions/managing_organisation.rb
  2. 19
      spec/models/form/lettings/questions/managing_organisation_spec.rb

6
app/models/form/lettings/questions/managing_organisation.rb

@ -30,7 +30,11 @@ class Form::Lettings::Questions::ManagingOrganisation < ::Form::Question
orgs = if user.support? orgs = if user.support?
log.owning_organisation.managing_agents log.owning_organisation.managing_agents
else else
user.organisation.managing_agents if user.organisation.absorbed_organisations.include?(log.owning_organisation)
user.organisation.managing_agents + log.owning_organisation.managing_agents
else
user.organisation.managing_agents
end
end.pluck(:id, :name).to_h end.pluck(:id, :name).to_h
user.organisation.absorbed_organisations.each do |absorbed_org| user.organisation.absorbed_organisations.each do |absorbed_org|

19
spec/models/form/lettings/questions/managing_organisation_spec.rb

@ -172,15 +172,28 @@ RSpec.describe Form::Lettings::Questions::ManagingOrganisation, type: :model do
create(:lettings_log, owning_organisation: absorbing_org, managing_organisation: nil) create(:lettings_log, owning_organisation: absorbing_org, managing_organisation: nil)
end end
let(:options) do it "displays merged organisation on the list of choices" do
{ options = {
"" => "Select an option", "" => "Select an option",
absorbing_org.id => "Absorbing org (Your organisation)", absorbing_org.id => "Absorbing org (Your organisation)",
merged_org.id => "Merged org (active until 2 August 2023)", merged_org.id => "Merged org (active until 2 August 2023)",
merged_org.id => "Merged org (active until 2 August 2023)",
} }
expect(question.displayed_answer_options(log, user)).to eq(options)
end end
it "displays merged organisation on the list of choices" do it "displays managing agents of merged organisation selected as owning org" do
managing_agent = create(:organisation, name: "Managing org 1")
create(:organisation_relationship, parent_organisation: merged_org, child_organisation: managing_agent)
options = {
"" => "Select an option",
merged_org.id => "Merged org (active until 2 August 2023)",
absorbing_org.id => "Absorbing org (Your organisation)",
managing_agent.id => "Managing org 1",
}
log.update!(owning_organisation: merged_org)
expect(question.displayed_answer_options(log, user)).to eq(options) expect(question.displayed_answer_options(log, user)).to eq(options)
end end
end end

Loading…
Cancel
Save