Streamline response handling in payment processing

Remove some indirection and unnecessary meta-programming.
Have it return either true or false leaving the "success" state call
as a responsibility of the caller (the "failure_state" was always the
same).

Also remove one nesting level in the implementation.

Author: elia

core/app/models/spree/payment/processing.rb

@@ -45,7 +45,7 @@ def authorize!
             source,
             gateway_options,
           )
-          handle_response(response, :pend, :failure)
+          pend! if handle_response(response)
         end
       end
 
@@ -61,7 +61,7 @@ def purchase!
             source,
             gateway_options,
           )
-          handle_response(response, :complete, :failure)
+          complete! if handle_response(response)
         end
 
         # This won't be called if `payment_method.purchase` raises a GatewayError
@@ -87,7 +87,7 @@ def capture!(capture_amount = nil)
           money = ::Money.new(capture_amount, currency)
           capture_events.create!(amount: money.to_d)
           update!(amount: captured_amount)
-          handle_response(response, :complete, :failure)
+          complete! if handle_response(response)
         end
       end
 
@@ -182,24 +182,28 @@ def check_payment_preconditions!
         true
       end
 
-      def handle_response(response, success_state, failure_state)
+      # @returns true if the response is successful
+      # @returns false (and calls #failure) if the response is not successful
+      def handle_response(response)
         record_response(response)
 
-        if response.success?
-          unless response.authorization.nil?
-            self.response_code = response.authorization
-            self.avs_response = response.avs_result['code']
-
-            if response.cvv_result
-              self.cvv_response_code = response.cvv_result['code']
-              self.cvv_response_message = response.cvv_result['message']
-            end
-          end
-          send("#{success_state}!")
-        else
-          send(failure_state)
+        unless response.success?
+          failure
           gateway_error(response)
+          return false
+        end
+
+        unless response.authorization.nil?
+          self.response_code = response.authorization
+          self.avs_response = response.avs_result['code']
+
+          if response.cvv_result
+            self.cvv_response_code = response.cvv_result['code']
+            self.cvv_response_message = response.cvv_result['message']
+          end
         end
+
+        true
       end
 
       def record_response(response)