Improve renew action

app/controllers/repp/v1/domains/renews_controller.rb

@@ -40,7 +40,9 @@ module Repp
           @domain = Epp::Domain.find_by(registrar: registrar, name: params[:domain_id])
           @domain ||= Epp::Domain.find_by!(registrar: registrar, name_puny: params[:domain_id])
 
-          @domain
+          return @domain if @domain
+
+          raise ActiveRecord::RecordNotFound
         end
 
         def renew_params

app/models/actions/domain_renew.rb

@@ -15,19 +15,12 @@ module Actions
     end
 
     def renew
-      period = params[:period]
-      unit = params[:period_unit]
-
       task = Domains::BulkRenew::SingleDomainRenew.run(domain: domain,
         period: params[:period],
         unit: params[:period_unit],
         registrar: user)
 
-      return true if task
+      task.valid?
-
-      puts task.errors
-
-      false
     end
   end
 end