small refactoring - 3

app/controllers/schemes_controller.rb

@@ -49,7 +49,7 @@ class SchemesController < ApplicationController
         if confirm_secondary_page? page
           redirect_to scheme_secondary_client_group_path(@scheme, check_answers: "true")
         else
-          @scheme.update!(secondary_client_group: nil) unless !@scheme.has_other_client_group == "No"
+          @scheme.update!(secondary_client_group: nil) if @scheme.has_other_client_group == "No"
           redirect_to scheme_check_answers_path(@scheme)
         end
       else

spec/requests/schemes_controller_spec.rb

@@ -205,7 +205,6 @@ RSpec.describe SchemesController, type: :request do
         expect(page).to have_content(specific_scheme.sensitive)
         expect(page).to have_content(specific_scheme.scheme_type)
         expect(page).to have_content(specific_scheme.registered_under_care_act)
-        expect(page).to have_content(specific_scheme.total_units)
         expect(page).to have_content(specific_scheme.primary_client_group)
         expect(page).to have_content(specific_scheme.secondary_client_group)
         expect(page).to have_content(specific_scheme.support_type)
@@ -239,7 +238,6 @@ RSpec.describe SchemesController, type: :request do
         expect(page).to have_content(specific_scheme.sensitive)
         expect(page).to have_content(specific_scheme.scheme_type)
         expect(page).to have_content(specific_scheme.registered_under_care_act)
-        expect(page).to have_content(specific_scheme.total_units)
         expect(page).to have_content(specific_scheme.primary_client_group)
         expect(page).to have_content(specific_scheme.secondary_client_group)
         expect(page).to have_content(specific_scheme.support_type)