Make sure degree requirements row doesn't appear for undergraduate

app/components/candidate_interface/course_choices_review_component.rb

@@ -158,7 +158,7 @@ def start_date_row(application_choice)
     end
 
     def degree_required_row(application_choice)
-      return unless application_choice.current_course.degree_grade?
+      return if application_choice.current_course.does_not_require_degree?
 
       {
         key: 'Degree requirements',

app/models/course.rb

@@ -210,6 +210,10 @@ def course_type
     I18n.t('course_types.postgraduate')
   end
 
+  def does_not_require_degree?
+    undergraduate? || !degree_grade?
+  end
+
 private
 
   def touch_application_choices_and_forms

spec/components/candidate_interface/course_review_component_spec.rb

@@ -60,4 +60,41 @@
       end
     end
   end
+
+  describe 'degree requirements' do
+    let(:application_choice) { create(:application_choice, course_option: create(:course_option, course:)) }
+
+    context 'when course is undergraduate but incorrectly has a degree_grade' do
+      let(:course) do
+        create(
+          :course,
+          :teacher_degree_apprenticeship,
+          degree_grade: 'two_one',
+        )
+      end
+
+      it 'does not render row' do
+        result = render_inline(described_class.new(application_choice:))
+        expect(result.text).not_to include('Degree requirements')
+      end
+    end
+
+    context 'when course is postgraduate and degree grade is present' do
+      let(:course) { create(:course, degree_grade: 'two_one') }
+
+      it 'does render row' do
+        result = render_inline(described_class.new(application_choice:))
+        expect(result.text).to include('Degree requirements')
+      end
+    end
+
+    context 'when course is postgraduate and degree grade is blank' do
+      let(:course) { create(:course, degree_grade: nil) }
+
+      it 'does not render row' do
+        result = render_inline(described_class.new(application_choice:))
+        expect(result.text).not_to include('Degree requirements')
+      end
+    end
+  end
 end

spec/models/course_spec.rb

@@ -349,4 +349,35 @@
       end
     end
   end
+
+  describe '#does_not_require_degree?' do
+    let(:course) { build(:course, program_type:, degree_grade:) }
+
+    context 'when the course is undergraduate' do
+      let(:program_type) { 'teacher_degree_apprenticeship' }
+      let(:degree_grade) { nil }
+
+      it 'returns true' do
+        expect(course.does_not_require_degree?).to be(true)
+      end
+    end
+
+    context 'when the course is postgraduate and does not require a degree grade' do
+      let(:program_type) { 'higher_education_programme' }
+      let(:degree_grade) { nil }
+
+      it 'returns true' do
+        expect(course.does_not_require_degree?).to be(true)
+      end
+    end
+
+    context 'when the course is postgraduate and requires a degree grade' do
+      let(:program_type) { 'higher_education_programme' }
+      let(:degree_grade) { 'two_one' }
+
+      it 'returns false' do
+        expect(course.does_not_require_degree?).to be(false)
+      end
+    end
+  end
 end