From c49dee0a630024bbd29d162010152042a2008faa Mon Sep 17 00:00:00 2001 From: Nelson Jovel Date: Thu, 21 Apr 2022 12:06:50 -0700 Subject: [PATCH] Do not show question prompts for measure 1B-1. Finishes #181921933 --- app/presenters/teacher_survey_presenter.rb | 2 ++ .../teacher_survey_presenter_spec.rb | 35 +++++++++++++++++++ 2 files changed, 37 insertions(+) create mode 100644 spec/presenters/data_item_presenters/teacher_survey_presenter_spec.rb diff --git a/app/presenters/teacher_survey_presenter.rb b/app/presenters/teacher_survey_presenter.rb index ef46afaa..22ab9992 100644 --- a/app/presenters/teacher_survey_presenter.rb +++ b/app/presenters/teacher_survey_presenter.rb @@ -13,6 +13,8 @@ class TeacherSurveyPresenter < DataItemPresenter end def item_descriptions + return ['Items available upon request to MCIEA.'] if @measure_id == '1B-i' + @survey_items.map(&:prompt) end diff --git a/spec/presenters/data_item_presenters/teacher_survey_presenter_spec.rb b/spec/presenters/data_item_presenters/teacher_survey_presenter_spec.rb new file mode 100644 index 00000000..2d80d20b --- /dev/null +++ b/spec/presenters/data_item_presenters/teacher_survey_presenter_spec.rb @@ -0,0 +1,35 @@ +require 'rails_helper' + +describe TeacherSurveyPresenter do + let(:measure_1A_i) { Measure.find_by_measure_id '1A-i' } + let(:measure_1B_i) { Measure.find_by_measure_id '1B-i' } + describe '#item_description' do + before :each do + Rails.application.load_seed + end + + after :each do + DatabaseCleaner.clean + end + + context 'When the presenter is based on measure 1A-1' do + it 'returns a list of survey prompts for teacher survey items' do + expect(TeacherSurveyPresenter.new(measure_id: measure_1A_i.measure_id, survey_items: measure_1A_i.teacher_survey_items, + has_sufficient_data: true).item_descriptions).to eq [ + 'Given your preparation for teaching how comfortable are you teaching at the grade-level you have been assigned?', + 'How prepared are you for teaching the topics that you are expected to teach in your assignment?', + 'How confident are you in working with the student body at your school?' + ] + end + end + + context 'When the presenter is based on measure 1B-i' do + it 'returns a message hiding the actual prompts. Instead it presents a message telling the user they can ask for more information' do + expect(TeacherSurveyPresenter.new(measure_id: measure_1B_i.measure_id, survey_items: measure_1B_i.teacher_survey_items, + has_sufficient_data: true).item_descriptions).to eq [ + 'Items available upon request to MCIEA.' + ] + end + end + end +end