Surveys during proposal submission

.haml-lint_todo.yml

@@ -38,6 +38,7 @@ linters:
       - "app/views/admin/emails/index.html.haml"
       - "app/views/admin/events/_proposal.html.haml"
       - "app/views/admin/events/index.html.haml"
+      - "app/views/admin/events/show.html.haml"
       - "app/views/admin/tracks/index.html.haml"
       - "app/views/admin/tracks/show.html.haml"
       - "app/views/admin/versions/_object_desc_and_link.html.haml"

.rubocop_todo.yml

@@ -442,7 +442,7 @@ Metrics/BlockNesting:
 # Offense count: 14
 # Configuration parameters: CountComments, CountAsOne.
 Metrics/ClassLength:
-  Max: 652
+  Max: 655
 
 # Offense count: 29
 # Configuration parameters: IgnoredMethods.

app/controllers/proposals_controller.rb

@@ -12,6 +12,7 @@ def index
     @event = @program.events.new
     @event.event_users.new(user: current_user, event_role: 'submitter')
     @events = current_user.proposals(@conference)
+    @surveys = @conference.surveys.during_proposal.select(&:active?)
   end
 
   def show

app/models/conference.rb

@@ -57,6 +57,10 @@ class Conference < ApplicationRecord
   has_many :event_types, through: :program
 
   has_many :surveys, as: :surveyable, dependent: :destroy do
+    def during_proposal
+      where(target: targets[:during_proposal])
+    end
+
     def for_registration
       where(target: targets[:during_registration])
     end

app/models/event.rb

@@ -228,8 +228,11 @@ def speaker_emails
   #
   # Returns +Hash+
   def progress_status
+    surveys = conference.surveys.during_proposal.select(&:active?)
+
     {
       registered:       speakers.all? { |speaker| program.conference.user_registered? speaker },
+      surveys:          (surveys.all? { |survey| survey.replied?(submitter) } if surveys.present?),
       commercials:      commercials.any?,
       biographies:      speakers.all? { |speaker| !speaker.biography.blank? },
       subtitle:         !subtitle.blank?,

app/models/survey.rb

@@ -5,7 +5,7 @@ class Survey < ActiveRecord::Base
   has_many :survey_questions, dependent: :destroy
   has_many :survey_submissions, dependent: :destroy
 
-  enum target: [:after_conference, :during_registration, :after_event]
+  enum target: [:after_conference, :during_registration, :after_event, :during_proposal]
   validates :title, presence: true
 
   ##
@@ -36,4 +36,8 @@ def active?
   def closed?
     !active?
   end
+
+  def replied?(user_id)
+    survey_submissions.where(user_id: user_id).any?
+  end
 end

app/views/admin/events/show.html.haml

@@ -67,6 +67,11 @@
             %td= link_to "#{'Speaker'.pluralize(@event.speakers.count)} must be registered to the conference", admin_conference_registrations_path(@event.program.conference.short_title)
             %td{ 'class' => class_for_todo(progress_status['registered']) }
               %span{ 'class' => [icon_for_todo(progress_status['registered']), 'fa-lg'] }
+          - unless progress_status['surveys'].nil?
+            %tr
+              %td= link_to 'Respond to surveys', admin_conference_surveys_path(@event.program.conference.short_title)
+              %td{ 'class' => class_for_todo(progress_status['surveys']) }
+                %span{ 'class' => [icon_for_todo(progress_status['surveys']), 'fa-lg'] }
           %tr
             %td
               - speakers_count = @event.speakers.count

app/views/proposals/_tooltip.html.haml

@@ -7,6 +7,10 @@
           Speaker(s) registered to the conference
         - else
           = link_to 'Speaker(s) not registered to the conference', new_conference_conference_registration_path(event.program.conference.short_title)
+  - unless progress_status['surveys'].nil?
+    %li{'class'=>class_for_todo(progress_status['surveys'])}
+      %span{'class'=>icon_for_todo(progress_status['surveys'])}
+      = link_to 'Respond to surveys', '#surveys'
   %li{'class'=>class_for_todo(progress_status['biographies'])}
     %span{'class'=>icon_for_todo(progress_status['biographies'])}
       - if progress_status['biographies']

app/views/proposals/index.html.haml

@@ -117,3 +117,10 @@
     .col-md-12
       - if can? :create, @event
         = link_to "New Proposal", new_conference_program_proposal_path(@conference.short_title), class: 'btn btn-success pull-right'
+
+  - if @surveys.any?
+    .row
+      .col-md-12
+        %h2#surveys Surveys
+
+        = render partial: 'surveys/list', locals: { surveys: @surveys, conference: @conference }

app/views/surveys/_list.html.haml

@@ -1,5 +1,5 @@
 - surveys.each do |survey|
-  - if survey.survey_submissions.find_by(user: current_user)
+  - if survey.replied?(current_user)
     %i.fa-solid.fa-square-check.text-success{ title: 'Thank you for filling out the survey' }
   - else
     %i.fa-solid.fa-square-minus.text-danger{ title: 'Please fill out the survey' }

spec/features/surveys_spec.rb

@@ -52,4 +52,36 @@
       expect(find(:link, survey.title).sibling('.fa-solid')[:title]).to eq('Thank you for filling out the survey')
     end
   end
+
+  context 'as a speaker' do
+    let(:speaker) { create(:user) }
+
+    before :each do
+      sign_in speaker
+    end
+
+    scenario 'respond to a survey during proposal submission', feature: true, js: true do
+      create :cfp, program: conference.program
+      create :event, program: conference.program, submitter: speaker
+      survey = create(:survey, surveyable: conference, target: :during_proposal)
+      create :boolean_mandatory, survey: survey
+
+      visit conference_program_proposals_path(conference.short_title)
+      within('.progress') { expect(page).to have_text '4 left' }
+      click_on 'Complete your proposal'
+      within('.popover') { expect(find(:link, 'Respond to surveys')).to have_sibling('.fa-xmark') }
+      expect(find(:link, survey.title).sibling('.fa-solid')[:title]).to eq('Please fill out the survey')
+
+      click_link survey.title
+      choose 'Yes'
+      click_button 'Submit'
+      expect(flash).to eq('Successfully responded to survey.')
+
+      visit conference_program_proposals_path(conference.short_title)
+      within('.progress') { expect(page).to have_text '3 left' }
+      click_on 'Complete your proposal'
+      within('.popover') { expect(find(:link, 'Respond to surveys')).to have_sibling('.fa-check') }
+      expect(find(:link, survey.title).sibling('.fa-solid')[:title]).to eq('Thank you for filling out the survey')
+    end
+  end
 end