Merge pull request #2780 from DFE-Digital/AQTS-1017-bug-fix

AQTS 1017 work history bug fix

Author: syed87

app/controllers/teacher_interface/work_histories_controller.rb

@@ -227,7 +227,16 @@ def destroy
         )
 
       if @form.save(validate: true)
-        redirect_to %i[check teacher_interface application_form work_histories]
+        if application_form.work_histories.none?
+          redirect_to %i[new teacher_interface application_form work_history]
+        else
+          redirect_to %i[
+                        check
+                        teacher_interface
+                        application_form
+                        work_histories
+                      ]
+        end
       else
         send_errors_to_big_query(@form)
 

spec/system/teacher_interface/work_history_spec.rb

@@ -71,6 +71,19 @@
     when_i_delete_work_history
     then_i_see_the(:teacher_check_work_histories_page)
     and_i_see_two_work_histories
+
+    when_i_click_delete_work_history
+    then_i_see_the(:teacher_delete_work_history_page)
+
+    when_i_delete_work_history
+    then_i_see_the(:teacher_check_work_histories_page)
+    and_i_see_one_work_history
+
+    when_i_click_delete_work_history
+    then_i_see_the(:teacher_delete_work_history_page)
+
+    when_i_delete_work_history
+    then_i_see_the(:teacher_new_work_history_page)
   end
 
   private
@@ -262,6 +275,10 @@ def and_i_see_two_work_histories
     expect(teacher_check_work_histories_page.summary_cards.count).to eq(2)
   end
 
+  def and_i_see_one_work_history
+    expect(teacher_check_work_histories_page.summary_cards.count).to eq(1)
+  end
+
   def teacher
     @teacher ||= create(:teacher)
   end