Skip to content

Commit 55ed101

Browse files
committed
Show validation errors also on people_manageds aside form
1 parent 85b69d2 commit 55ed101

File tree

2 files changed

+12
-1
lines changed

2 files changed

+12
-1
lines changed

app/views/person/manageds/new.html.haml

+1
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88

99
= render layout: 'people_managers/section' do
1010
= standard_form(entry, url: person_manageds_path(person), html: {class: 'p-2'}) do |f|
11+
= f.error_messages
1112
- if create_managed? && params[:create]
1213
= render "new_person_fields", f: f
1314
- else

spec/features/people_managers_spec.rb

+11-1
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ def find_person(name, retries = 0)
6767
end
6868
end
6969

70-
it 'shows validation errors in form' do
70+
it 'shows manager validation errors in form' do
7171
within_turbo_frame do
7272
person.people_managers.create!(manager: people(:bottom_member))
7373
click_dropdown('Verwalter*in zuweisen')
@@ -77,6 +77,16 @@ def find_person(name, retries = 0)
7777
end
7878
end
7979

80+
it 'shows managed validation errors in form' do
81+
within_turbo_frame do
82+
person.people_manageds.create!(managed: people(:bottom_member))
83+
click_dropdown('Kind zuweisen')
84+
find_person 'Bottom Member'
85+
click_on 'Speichern'
86+
expect(page).to have_css('.alert-danger', text: 'Verwalter*in ist bereits gesetzt.')
87+
end
88+
end
89+
8090
context 'without write permission on other persons' do
8191
let(:person) { current_user }
8292
let(:current_user) { Fabricate(Group::TopGroup::LocalSecretary.sti_name, group: top_group).person }

0 commit comments

Comments
 (0)