rubocop

app/controllers/registration/training_emails_controller.rb

@@ -9,12 +9,10 @@ def update
     if @user_form.save
       if ENV['EARLY_YEARS_EMAILS']
         redirect_to edit_registration_early_years_emails_path
+      elsif current_user.registration_complete?
+        redirect_to my_modules_path, notice: t('.complete_update')
       else
-        if current_user.registration_complete?
-          redirect_to my_modules_path, notice: t('.complete_update')
-        else
-          complete_registration
-        end
+        complete_registration
       end
     else
       render :edit, status: :unprocessable_entity

app/forms/users/early_years_emails_form.rb

@@ -1,19 +1,19 @@
 module Users
-    class EarlyYearsEmailsForm < BaseForm
-      attr_accessor :early_years_emails
-
-      validates :early_years_emails, presence: true
-
-      def name
-        'early_years_emails'
-      end
-
-      def save
-        if valid?
-          user.update!(
-            early_years_emails: early_years_emails,
-          )
-        end
+  class EarlyYearsEmailsForm < BaseForm
+    attr_accessor :early_years_emails
+
+    validates :early_years_emails, presence: true
+
+    def name
+      'early_years_emails'
+    end
+
+    def save
+      if valid?
+        user.update!(
+          early_years_emails: early_years_emails,
+        )
       end
     end
-  end
+  end
+end

spec/system/confirmed_user/completing_registration_spec.rb

@@ -81,7 +81,7 @@
       choose 'Send me early years email updates'
 
       click_button 'Continue'
-    
+
     end
 
     expect(page).to have_text('Thank you for creating an Early years child development training account. You can now start the first module.')