SecureNet: Allow production transactions

The TEST parameter was always being sent as true!  Now we only pass that
parameter at the right time.

Closes activemerchant#805.

CHANGELOG

@@ -2,6 +2,7 @@
 
 * MerchantWarrior: Fix handling of amounts [duff]
 * MerchantWarrior: Send the CVV to the gateway [duff]
+* SecureNet: Allow production transactions [duff]
 
 == Version 1.36.0 (August 2, 2013)
 

lib/active_merchant/billing/gateways/secure_net.rb

@@ -69,9 +69,8 @@ def commit(request)
         data = ssl_post(url, xml, "Content-Type" => "text/xml")
         response = parse(data)
 
-        test_mode = test?
         Response.new(success?(response), message_from(response), response,
-          :test => test_mode,
+          :test => test?,
           :authorization => build_authorization(response),
           :avs_result => { :code => response[:avs_result_code] },
           :cvv_result => response[:card_code_response_code]
@@ -208,7 +207,7 @@ def add_required_params(xml, action, options)
 
       def add_more_required_params(xml, options)
         xml.tag! 'RETAIL_LANENUM', '0'
-        xml.tag! 'TEST', 'TRUE'
+        xml.tag! 'TEST', 'TRUE' if test?
         xml.tag! 'TOTAL_INSTALLMENTCOUNT', 0
         xml.tag! 'TRANSACTION_SERVICE', 0
       end