Skip to content

Commit 04e5ba0

Browse files
authored
Merge pull request #2634 from DFE-Digital/AQTS-818-dev-elp-summary-not-visible-on-check-your-answers-page-before-submitting-your-application
[AQTS-818] Introduce "Your English language proficiency" section on "Check your answers" page for applicants
2 parents c3e4c81 + 52bab1e commit 04e5ba0

File tree

3 files changed

+28
-0
lines changed

3 files changed

+28
-0
lines changed

app/views/teacher_interface/application_forms/edit.html.erb

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,13 @@
2020
<%= render "shared/application_form/subjects_summary", application_form: @application_form, changeable: true %>
2121
</section>
2222

23+
<% unless @application_form.created_under_old_regulations? %>
24+
<section id="app-application-form-your-english-language-proficiency">
25+
<h2 class="govuk-heading-m">Your English language proficiency</h2>
26+
<%= render "shared/application_form/english_language_summary", application_form: @application_form, changeable: true %>
27+
</section>
28+
<% end %>
29+
2330
<% if @application_form.needs_work_history %>
2431
<section id="app-application-form-work-history">
2532
<h2 class="govuk-heading-m">Your work history</h2>

spec/support/autoload/page_objects/teacher_interface/check_your_answers.rb

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,13 @@ class CheckYourAnswers < SitePrism::Page
3232
"#app-check-your-answers-summary-subjects"
3333
end
3434

35+
section :your_english_language_proficiency,
36+
"#app-application-form-your-english-language-proficiency" do
37+
section :english_language_proficiency_summary_list,
38+
GovukSummaryList,
39+
"#app-check-your-answers-summary-english-language"
40+
end
41+
3542
section :work_history, "section#app-application-form-work-history" do
3643
section :add_summary_list,
3744
GovukSummaryList,

spec/system/teacher_interface/check_answers_spec.rb

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,11 @@
4444
end
4545
end
4646

47+
it "english language proficiency section" do
48+
when_i_click_change_link_for_english_language_proficiency
49+
then_i_see_the(:teacher_english_language_exemption_page)
50+
end
51+
4752
it "work history section" do
4853
when_i_click_change_links(:work_history, :work_history) do
4954
if teacher_check_document_page.displayed?(0)
@@ -159,6 +164,15 @@ def and_i_dont_need_to_upload_another_file
159164
teacher_check_document_page.form.continue_button.click
160165
end
161166

167+
def when_i_click_change_link_for_english_language_proficiency
168+
find_change_link_summary_list(
169+
:your_english_language_proficiency,
170+
:english_language_proficiency,
171+
).rows[
172+
1
173+
].actions.link.click
174+
end
175+
162176
def teacher
163177
@teacher ||= create(:teacher)
164178
end

0 commit comments

Comments
 (0)