diff --git a/app/views/admin/events/_form.html.haml b/app/views/admin/events/_form.html.haml index 231089153..59efdc084 100644 --- a/app/views/admin/events/_form.html.haml +++ b/app/views/admin/events/_form.html.haml @@ -46,7 +46,7 @@ .col-12.col-md-6.col-lg-4 = f.association :gold_sponsors, input_html: { data: { placeholder: 'Select gold sponsors' }}, collection: all_sponsors .col-12 - = f.input :organisers, collection: all_managers, value_method: :id, label_method: :full_name, selected: @event.organisers.pluck(&:id), input_html: { multiple: true } + = f.input :organisers, collection: all_managers, value_method: :id, label_method: :full_name, selected: @event.organisers.pluck(:id), input_html: { multiple: true } .col-12 = f.input :announce_only, as: :boolean, hint: 'Events where invitations are not handled via our application' .col-12 diff --git a/spec/features/admin/manage_event_spec.rb b/spec/features/admin/manage_event_spec.rb index 6f6170eed..d0104cba5 100644 --- a/spec/features/admin/manage_event_spec.rb +++ b/spec/features/admin/manage_event_spec.rb @@ -25,7 +25,9 @@ find_by_id('event_chapter_ids_chosen').click find('.add-all-chapters', text: 'Add to all').click - expect(page).to have_css('.search-choice', count: Chapter.count) + within('#event_chapter_ids_chosen') do + expect(page).to have_css('.search-choice', count: Chapter.count) + end click_on 'Save' @@ -33,6 +35,15 @@ expect(event.reload.chapter_ids).to match_array(Chapter.ids) end + scenario 'editing an event keeps its existing organisers' do + visit edit_admin_event_path(event) + + click_on 'Save' + + expect(page).to have_text('You have just updated the event') + expect(event.reload.organisers).to include(member) + end + scenario 'verifying an attendance' do invitation = Fabricate(:invitation, event:, attending: true) visit admin_event_path(event) diff --git a/spec/models/concerns/listable_spec.rb b/spec/models/concerns/listable_spec.rb index ad0b19ed6..d54a86a08 100644 --- a/spec/models/concerns/listable_spec.rb +++ b/spec/models/concerns/listable_spec.rb @@ -6,7 +6,7 @@ context 'with scopes' do describe '#today_and_upcoming' do it 'returns a list of all today and upcoming workshops' do - travel_to(Time.current) do + travel_to(Time.current.middle_of_day) do Fabricate.times(2, :past_workshop) future_workshops = Fabricate.times(1, :workshop)