killbill-orbital 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a9d2b43b5b0f77588cd43c1c2dbe7b1fbd179e06
-  data.tar.gz: 89c9027236eda7d63b0e08204b71b0da28fe704b
+  metadata.gz: caf9e96477c8998f8775bafa6ac5880e9864ff52
+  data.tar.gz: a2db465f2f8dbbba3f2f348ad04946147fe62acd
 SHA512:
-  metadata.gz: b6fcd695a3cc200e28065e546ac34ce7dc16751fa9a57f36d86d9aeb5dd9a2d527e094bb9e0f1cf97b5fb797a6472b9fc904c369ad61b7d2ff9d7e1ae35523d4
-  data.tar.gz: f7d816f206329ee0e30ed68ee4e92e45e29281db1244accfcc7a6bacb11e803e964ea1075cb48f0534e57d458cf8ff60378037b024dca800052222e360d4cf7b
+  metadata.gz: 63322da8cd5cd93f014c19a8f160641f34847a22ceac9cb8dabf06efb25353755cf75839ed8a6aacc542b69dca4b2b85e00209fbd5dc8d1baba3469642808064
+  data.tar.gz: 50548d3981b5fba7f051c1feda33ff772666826392967265a79bead2ef987eaf02102599cda739450c2cbde5868d30631c17adb9fa71ab4e206227e6e93bbe76

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    killbill-orbital (0.1.1)
+    killbill-orbital (0.1.2)
       actionpack (~> 4.1.0)
       actionview (~> 4.1.0)
       activemerchant (~> 1.55.0)

data/NEWS

@@ -1,3 +1,8 @@
+0.1.2
+    Upgrade Orbital API version to 7.0.1
+    Add network tokenization support
+    Add support for credit operation
+
 0.1.1
     Return processor response in PaymentTransactionInfoPlugin
 

data/VERSION

@@ -1 +1 @@
-0.1.1
+0.1.2

data/lib/orbital/ext/active_merchant/active_merchant.rb

@@ -5,6 +5,8 @@ module ActiveMerchant
 
     class OrbitalGateway
 
+      API_VERSION = '7.0.1'
+
       def store(creditcard, options = {})
         response = add_customer_profile(creditcard, options)
 
@@ -36,7 +38,8 @@ module ActiveMerchant
 
         headers = POST_HEADERS.merge('Content-length' => order.size.to_s,
                                      'User-Agent' => user_agent,
-                                     'Interface-Version' => 'Ruby|KillBill|Open-Source Gateway')
+                                     'Interface-Version' => 'Ruby|KillBill|Open-Source Gateway',
+                                     'Content-Type' => 'application/PTI70')
         headers['X-Request-Id'] = x_r_id unless x_r_id.blank?
         headers.merge!('Trace-number' => trace_number.to_s,
                        'Merchant-Id' => @options[:merchant_id]) if @options[:retry_logic] && trace_number
@@ -60,32 +63,94 @@ module ActiveMerchant
                      })
       end
 
+      def build_new_order_xml(action, money, parameters = {})
+        requires!(parameters, :order_id)
+        xml = xml_envelope
+        xml.tag! :Request do
+          xml.tag! :NewOrder do
+            add_xml_credentials(xml)
+            # EC - Ecommerce transaction
+            # RC - Recurring Payment transaction
+            # MO - Mail Order Telephone Order transaction
+            # IV - Interactive Voice Response
+            # IN - Interactive Voice Response
+            xml.tag! :IndustryType, parameters[:industry_type] || ECOMMERCE_TRANSACTION
+            # A  - Auth Only No Capture
+            # AC - Auth and Capture
+            # F  - Force Auth No Capture and no online authorization
+            # FR - Force Auth No Capture and no online authorization
+            # FC - Force Auth and Capture no online authorization
+            # R  - Refund and Capture no online authorization
+            xml.tag! :MessageType, action
+            add_bin_merchant_and_terminal(xml, parameters)
+
+            yield xml if block_given?
+
+            xml.tag! :OrderID, format_order_id(parameters[:order_id])
+            xml.tag! :Amount, amount(money)
+            xml.tag! :Comments, parameters[:comments] if parameters[:comments]
+
+            # Add additional card information for tokenized credit card that must be placed after the above three elements
+            if action == AUTH_ONLY || action == AUTH_AND_CAPTURE
+              add_additional_network_tokenization(xml, parameters[:creditcard]) unless parameters[:creditcard].nil?
+            end
+
+            if parameters[:soft_descriptors].is_a?(OrbitalSoftDescriptors)
+              add_soft_descriptors(xml, parameters[:soft_descriptors])
+            end
+
+            set_recurring_ind(xml, parameters)
+
+            # Append Transaction Reference Number at the end for Refund transactions
+            if action == REFUND && !parameters[:authorization].nil?
+              tx_ref_num, _ = split_authorization(parameters[:authorization])
+              xml.tag! :TxRefNum, tx_ref_num
+            end
+          end
+        end
+        xml.target!
+      end
+
       # A – Authorization request
       def authorize(money, creditcard, options = {})
-        order = build_new_order_xml(AUTH_ONLY, money, options) do |xml|
+        order = build_new_order_xml(AUTH_ONLY, money, options.merge(:creditcard=>creditcard)) do |xml|
           add_creditcard(xml, creditcard, options)
           add_address(xml, creditcard, options)
           if @options[:customer_profiles]
             add_customer_data(xml, creditcard, options)
             add_managed_billing(xml, options)
           end
+          add_network_tokenization(xml, creditcard)
         end
         commit(order, :authorize, options[:trace_number])
       end
 
       # AC – Authorization and Capture
       def purchase(money, creditcard, options = {})
-        order = build_new_order_xml(AUTH_AND_CAPTURE, money, options) do |xml|
+        order = build_new_order_xml(AUTH_AND_CAPTURE, money, options.merge(:creditcard=>creditcard)) do |xml|
           add_creditcard(xml, creditcard, options)
           add_address(xml, creditcard, options)
           if @options[:customer_profiles]
             add_customer_data(xml, creditcard, options)
             add_managed_billing(xml, options)
           end
+          add_network_tokenization(xml, creditcard)
         end
         commit(order, :purchase, options[:trace_number])
       end
 
+      def credit(money, creditcard, options= {})
+        order = build_new_order_xml(REFUND, money, options) do |xml|
+          add_creditcard(xml, creditcard, options)
+          add_address(xml, creditcard, options)
+          if @options[:customer_profiles]
+            add_customer_data(xml, creditcard, options)
+            add_managed_billing(xml, options)
+          end
+        end
+        commit(order, :credit, options[:trace_number])
+      end
+
       def add_creditcard(xml, creditcard, options = {})
         currency = options[:currency]
         cvv_indicator_visa_discover = options[:cvv_indicator_visa_discover]
@@ -111,6 +176,41 @@ module ActiveMerchant
         end
       end
 
+      def add_network_tokenization(xml, payment_method)
+        return unless network_tokenization?(payment_method)
+        card_brand = card_brand(payment_method).to_sym
+
+        # The elements must follow a specific sequence
+        xml.tag!('AuthenticationECIInd', payment_method.eci) unless payment_method.eci.nil?
+        xml.tag!('CAVV', payment_method.payment_cryptogram) if card_brand == :visa
+      end
+
+      def add_additional_network_tokenization(xml, payment_method)
+        return unless network_tokenization?(payment_method)
+        card_brand = card_brand(payment_method).to_sym
+
+        # The elements must follow a specific sequence
+        xml.tag!('AAV', payment_method.payment_cryptogram) if card_brand == :master
+        xml.tag!('DPANInd', 'Y')
+        xml.tag!('AEVV', payment_method.payment_cryptogram) if card_brand == :american_express
+        xml.tag!('DigitalTokenCryptogram', payment_method.payment_cryptogram)
+      end
+
+      def network_tokenization?(payment_method)
+        payment_method.is_a?(NetworkTokenizationCreditCard)
+      end
+
+      def success?(response, message_type)
+        if [:refund, :void, :credit].include?(message_type)
+          response[:proc_status] == SUCCESS
+        elsif response[:customer_profile_action]
+          response[:profile_proc_status] == SUCCESS
+        else
+          response[:proc_status] == SUCCESS &&
+              APPROVED.include?(response[:resp_code])
+        end
+      end
+
     end
   end
 end

data/orbital.yml

@@ -3,7 +3,7 @@
     :test: true
     :login: <%= ENV['LOGIN'] %>
     :password: <%= ENV['PASSWORD'] %>
-    :merchant_id: <%= ENV['MERCHANT_ID'] %>
+    :merchant_id: "<%= ENV['MERCHANT_ID'] %>"
 
 :database:
 # SQLite (development)

data/pom.xml

@@ -25,7 +25,7 @@
     <groupId>org.kill-bill.billing.plugin.ruby</groupId>
     <artifactId>orbital-plugin</artifactId>
     <packaging>pom</packaging>
-    <version>0.1.1</version>
+    <version>0.1.2</version>
     <name>orbital-plugin</name>
     <url>http://github.com/killbill/killbill-orbital-plugin</url>
     <description>Plugin for accessing Orbital as a payment gateway</description>

data/spec/orbital/cvv_indicator_spec.rb

@@ -0,0 +1,171 @@
+require 'spec_helper'
+
+ActiveMerchant::Billing::Base.mode = :test
+
+describe Killbill::Orbital::PaymentPlugin do
+
+  include ::Killbill::Plugin::ActiveMerchant::RSpec
+
+  before(:each) do
+    @plugin = build_plugin(::Killbill::Orbital::PaymentPlugin, 'orbital')
+    @plugin.start_plugin
+    @call_context = build_call_context
+
+    ::Killbill::Orbital::OrbitalPaymentMethod.delete_all
+    ::Killbill::Orbital::OrbitalResponse.delete_all
+    ::Killbill::Orbital::OrbitalTransaction.delete_all
+  end
+
+  after(:each) do
+    @plugin.stop_plugin
+  end
+
+  context 'cvv indicator spec' do
+    before(:each) do
+      @amount     = BigDecimal.new('100')
+      @currency   = 'USD'
+    end
+
+    it 'should set correct indicator for visa and discover if cvv value is present regardless of cvv_indicator_visa_discover' do
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454' })
+      validate_cvv_indicator_field 1
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'discover' })
+      validate_cvv_indicator_field 1
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cvv_indicator_visa_discover => true })
+      validate_cvv_indicator_field 1
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'discover', :cvv_indicator_visa_discover => true  })
+      validate_cvv_indicator_field 1
+    end
+
+    it 'should set correct indicator for visa and discover if cvv value is not present and cvv_indicator_visa_discover is true' do
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cvv_indicator_visa_discover => true })
+      @properties.reject! {|property| property.key == 'ccVerificationValue' }
+      validate_cvv_indicator_field 9
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'discover', :cvv_indicator_visa_discover => true })
+      @properties.reject! {|property| property.key == 'ccVerificationValue' }
+      validate_cvv_indicator_field 9
+    end
+
+    it 'should set customized indicator for visa and discover if cvv value is not present and cvv_indicator_visa_discover is true and cvv_indicator_override_visa_discover is given' do
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cvv_indicator_visa_discover => true, :cvv_indicator_override_visa_discover => '2' })
+      @properties.reject! {|property| property.key == 'ccVerificationValue' }
+      validate_cvv_indicator_field 2
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'discover', :cvv_indicator_visa_discover => true, :cvv_indicator_override_visa_discover => '2' })
+      @properties.reject! {|property| property.key == 'ccVerificationValue' }
+      validate_cvv_indicator_field 2
+    end
+
+    it 'should set correct indicator for visa and discover if cvv value is not present and cvv_indicator_visa_discover is nil or false' do
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454'})
+      @properties.reject! {|property| property.key == 'ccVerificationValue' }
+      validate_cvv_indicator_field
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'discover'})
+      @properties.reject! {|property| property.key == 'ccVerificationValue' }
+      validate_cvv_indicator_field
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cvv_indicator_visa_discover => false })
+      @properties.reject! {|property| property.key == 'ccVerificationValue' }
+      validate_cvv_indicator_field
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'discover', :cvv_indicator_visa_discover => false })
+      @properties.reject! {|property| property.key == 'ccVerificationValue' }
+      validate_cvv_indicator_field
+    end
+
+    it 'should not include indicator except visa and discover for all cases' do
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'master' })
+      validate_cvv_indicator_field
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'american_express' })
+      validate_cvv_indicator_field
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'master', :cvv_indicator_visa_discover => false })
+      validate_cvv_indicator_field
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'american_express', :cvv_indicator_visa_discover => false })
+      validate_cvv_indicator_field
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'master', :cvv_indicator_visa_discover => true})
+      validate_cvv_indicator_field
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'american_express', :cvv_indicator_visa_discover => true })
+      validate_cvv_indicator_field
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'master' })
+      @properties.reject! {|property| property.key == 'ccVerificationValue' }
+      validate_cvv_indicator_field
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'american_express' })
+      @properties.reject! {|property| property.key == 'ccVerificationValue' }
+      validate_cvv_indicator_field
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'master', :cvv_indicator_visa_discover => false})
+      @properties.reject! {|property| property.key == 'ccVerificationValue' }
+      validate_cvv_indicator_field
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'american_express', :cvv_indicator_visa_discover => false })
+      @properties.reject! {|property| property.key == 'ccVerificationValue' }
+      validate_cvv_indicator_field
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'master', :cvv_indicator_visa_discover => true})
+      @properties.reject! {|property| property.key == 'ccVerificationValue' }
+      validate_cvv_indicator_field
+
+      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'american_express', :cvv_indicator_visa_discover => true })
+      @properties.reject! {|property| property.key == 'ccVerificationValue' }
+      validate_cvv_indicator_field
+    end
+  end
+
+  def successful_authorize_response
+    <<-XML
+<?xml version=\"1.0\" encoding=\"UTF-8\"?><Response><NewOrderResp><IndustryType></IndustryType><MessageType>A</MessageType><MerchantID>1111111</MerchantID><TerminalID>001</TerminalID><CardBrand>CC</CardBrand><AccountNum>XXXXXXXXXXXX5454</AccountNum><OrderID>5b257b31-1f84-44bc-b32</OrderID><TxRefNum>5834AA75E4466AEA59512165057C37DD810053C2</TxRefNum><TxRefIdx>0</TxRefIdx><ProcStatus>0</ProcStatus><ApprovalStatus>1</ApprovalStatus><RespCode>00</RespCode><AVSRespCode>B </AVSRespCode><CVV2RespCode> </CVV2RespCode><AuthCode>tst424</AuthCode><RecurringAdviceCd></RecurringAdviceCd><CAVVRespCode></CAVVRespCode><StatusMsg>Approved</StatusMsg><RespMsg></RespMsg><HostRespCode>100</HostRespCode><HostAVSRespCode>I3</HostAVSRespCode><HostCVV2RespCode>  </HostCVV2RespCode><CustomerRefNum></CustomerRefNum><CustomerName></CustomerName><ProfileProcStatus></ProfileProcStatus><CustomerProfileMessage></CustomerProfileMessage><RespTime>152837</RespTime><PartialAuthOccurred></PartialAuthOccurred><RequestedAmount></RequestedAmount><RedeemedAmount></RedeemedAmount><RemainingBalance></RemainingBalance><CountryFraudFilterStatus></CountryFraudFilterStatus><IsoCountryCode></IsoCountryCode></NewOrderResp></Response>
+    XML
+  end
+
+  def successful_purchase_response
+    <<-XML
+<?xml version=\"1.0\" encoding=\"UTF-8\"?><Response><NewOrderResp><IndustryType></IndustryType><MessageType>AC</MessageType><MerchantID>1111111</MerchantID><TerminalID>001</TerminalID><CardBrand>CC</CardBrand><AccountNum>XXXXXXXXXXXX5454</AccountNum><OrderID>88132d30-f4f7-4028-949</OrderID><TxRefNum>5834EAC9C7A53FEB600A479629FB6C6427A2532C</TxRefNum><TxRefIdx>1</TxRefIdx><ProcStatus>0</ProcStatus><ApprovalStatus>1</ApprovalStatus><RespCode>00</RespCode><AVSRespCode>3 </AVSRespCode><CVV2RespCode>I</CVV2RespCode><AuthCode>tst703</AuthCode><RecurringAdviceCd></RecurringAdviceCd><CAVVRespCode></CAVVRespCode><StatusMsg>Approved</StatusMsg><RespMsg></RespMsg><HostRespCode>100</HostRespCode><HostAVSRespCode>  </HostAVSRespCode><HostCVV2RespCode>I</HostCVV2RespCode><CustomerRefNum></CustomerRefNum><CustomerName></CustomerName><ProfileProcStatus></ProfileProcStatus><CustomerProfileMessage></CustomerProfileMessage><RespTime>200306</RespTime><PartialAuthOccurred></PartialAuthOccurred><RequestedAmount></RequestedAmount><RedeemedAmount></RedeemedAmount><RemainingBalance></RemainingBalance><CountryFraudFilterStatus></CountryFraudFilterStatus><IsoCountryCode></IsoCountryCode></NewOrderResp></Response>    XML
+    XML
+  end
+
+  def validate_cvv_indicator_field(expected_field = nil)
+    ::ActiveMerchant::Billing::OrbitalGateway.any_instance.stub(:ssl_post) do |host, request_body|
+      if expected_field.nil?
+        request_body.should_not match('<CardSecValInd>')
+      else
+        request_body.should match("<CardSecValInd>#{expected_field}</CardSecValInd>")
+      end
+      if request_body.include? '<MessageType>A</MessageType>'
+        successful_authorize_response
+      else
+        successful_purchase_response
+      end
+    end
+
+    authorize
+    purchase
+  end
+
+  def authorize
+    kb_payment_id, kb_transaction_id = create_payment
+    payment_response = @plugin.authorize_payment(SecureRandom.uuid, kb_payment_id, kb_transaction_id, SecureRandom.uuid, @amount, @currency, @properties, @call_context)
+    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
+    payment_response.amount.should == @amount
+    payment_response.transaction_type.should == :AUTHORIZE
+  end
+
+  def purchase
+    kb_payment_id, kb_transaction_id = create_payment
+    payment_response = @plugin.purchase_payment(SecureRandom.uuid, kb_payment_id, kb_transaction_id, SecureRandom.uuid, @amount, @currency, @properties, @call_context)
+    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
+    payment_response.amount.should == @amount
+    payment_response.transaction_type.should == :PURCHASE
+  end
+end

data/spec/orbital/remote/integration_spec.rb

@@ -1,222 +1,8 @@
 require 'spec_helper'
+require_relative 'shared_examples_for_payment_flow'
 
 ActiveMerchant::Billing::Base.mode = :test
 
-shared_examples 'common_specs' do
-  before(:each) do
-    create_payment
-  end
-
-  after(:each) do
-    @plugin.stop_plugin
-  end
-
-  it 'should be able to purchase' do
-    Killbill::Orbital::OrbitalResponse.all.size.should == 1
-    Killbill::Orbital::OrbitalTransaction.all.size.should == 0
-
-    payment_response = @plugin.purchase_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[0].id, @pm.kb_payment_method_id, @amount, @currency, @properties, @call_context)
-    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
-    payment_response.amount.should == @amount
-    payment_response.transaction_type.should == :PURCHASE
-    find_value_from_properties(payment_response.properties, 'processorResponse').should == '100'
-
-    responses = Killbill::Orbital::OrbitalResponse.all
-    responses.size.should == 2
-    responses[0].api_call.should == 'add_payment_method'
-    responses[0].message.should == 'Profile Request Processed'
-    responses[1].api_call.should == 'purchase'
-    responses[1].message.should == 'Approved'
-    transactions = Killbill::Orbital::OrbitalTransaction.all
-    transactions.size.should == 1
-    transactions[0].api_call.should == 'purchase'
-  end
-
-  it 'should be able to charge and refund' do
-    payment_response = @plugin.purchase_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[0].id, @pm.kb_payment_method_id, @amount, @currency, @properties, @call_context)
-    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
-    payment_response.amount.should == @amount
-    payment_response.transaction_type.should == :PURCHASE
-    find_value_from_properties(payment_response.properties, 'processorResponse').should == '100'
-
-    # Try a full refund
-    refund_response = @plugin.refund_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[1].id, @pm.kb_payment_method_id, @amount, @currency, @properties, @call_context)
-    refund_response.status.should eq(:PROCESSED), refund_response.gateway_error
-    refund_response.amount.should == @amount
-    refund_response.transaction_type.should == :REFUND
-  end
-
-  it 'should be able to auth, capture and refund' do
-    payment_response = @plugin.authorize_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[0].id, @pm.kb_payment_method_id, @amount, @currency, @properties, @call_context)
-    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
-    payment_response.amount.should == @amount
-    payment_response.transaction_type.should == :AUTHORIZE
-    find_value_from_properties(payment_response.properties, 'processorResponse').should == '100'
-
-    # Try multiple partial captures
-    partial_capture_amount = BigDecimal.new('10')
-    1.upto(3) do |i|
-      payment_response = @plugin.capture_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[i].id, @pm.kb_payment_method_id, partial_capture_amount, @currency, @properties, @call_context)
-      payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
-      payment_response.amount.should == partial_capture_amount
-      payment_response.transaction_type.should == :CAPTURE
-    end
-
-    # Try a partial refund
-    refund_response = @plugin.refund_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[4].id, @pm.kb_payment_method_id, partial_capture_amount, @currency, @properties, @call_context)
-    refund_response.status.should eq(:PROCESSED), refund_response.gateway_error
-    refund_response.amount.should == partial_capture_amount
-    refund_response.transaction_type.should == :REFUND
-
-    # Try to capture again
-    payment_response = @plugin.capture_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[5].id, @pm.kb_payment_method_id, partial_capture_amount, @currency, @properties, @call_context)
-    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
-    payment_response.amount.should == partial_capture_amount
-    payment_response.transaction_type.should == :CAPTURE
-  end
-
-  it 'should be able to auth and void' do
-    payment_response = @plugin.authorize_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[0].id, @pm.kb_payment_method_id, @amount, @currency, @properties, @call_context)
-    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
-    payment_response.amount.should == @amount
-    payment_response.transaction_type.should == :AUTHORIZE
-    find_value_from_properties(payment_response.properties, 'processorResponse').should == '100'
-
-    payment_response = @plugin.void_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[1].id, @pm.kb_payment_method_id, @properties, @call_context)
-    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
-    payment_response.transaction_type.should == :VOID
-  end
-
-  it 'should be able to auth, partial capture and void' do
-    payment_response = @plugin.authorize_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[0].id, @pm.kb_payment_method_id, @amount, @currency, @properties, @call_context)
-    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
-    payment_response.amount.should == @amount
-    payment_response.transaction_type.should == :AUTHORIZE
-    find_value_from_properties(payment_response.properties, 'processorResponse').should == '100'
-
-    partial_capture_amount = BigDecimal.new('10')
-    payment_response       = @plugin.capture_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[1].id, @pm.kb_payment_method_id, partial_capture_amount, @currency, @properties, @call_context)
-    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
-    payment_response.amount.should == partial_capture_amount
-    payment_response.transaction_type.should == :CAPTURE
-
-    payment_response = @plugin.void_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[2].id, @pm.kb_payment_method_id, @properties, @call_context)
-    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
-    payment_response.transaction_type.should == :VOID
-  end
-
-  it 'should include host response code' do
-    # Sending a specific amount of 530 will trigger the Do Not Honor error.
-    payment_response = @plugin.authorize_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[0].id, @pm.kb_payment_method_id, BigDecimal.new('530'), @currency, @properties, @call_context)
-    payment_response.status.should eq(:ERROR), payment_response.gateway_error
-    payment_response.transaction_type.should == :AUTHORIZE
-    payment_response.amount.should be_nil
-    find_value_from_properties(payment_response.properties, 'processorResponse').should == '530'
-  end
-end
-
-shared_examples 'cvv_indicator_specs' do
-  after(:each) do
-    @plugin.stop_plugin
-  end
-
-  it 'should set correct indicator for visa and discover if cvv value is present regardless of cvv_indicator_visa_discover' do
-    validate_cvv_indicator_field 1
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'discover' })
-    validate_cvv_indicator_field 1
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cvv_indicator_visa_discover => true })
-    validate_cvv_indicator_field 1
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'discover', :cvv_indicator_visa_discover => true  })
-    validate_cvv_indicator_field 1
-  end
-
-  it 'should set correct indicator for visa and discover if cvv value is not present and cvv_indicator_visa_discover is true' do
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cvv_indicator_visa_discover => true })
-    @properties.reject! {|property| property.key == 'ccVerificationValue' }
-    validate_cvv_indicator_field 9
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'discover', :cvv_indicator_visa_discover => true })
-    @properties.reject! {|property| property.key == 'ccVerificationValue' }
-    validate_cvv_indicator_field 9
-  end
-
-  it 'should set customized indicator for visa and discover if cvv value is not present and cvv_indicator_visa_discover is true and cvv_indicator_override_visa_discover is given' do
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cvv_indicator_visa_discover => true, :cvv_indicator_override_visa_discover => '2' })
-    @properties.reject! {|property| property.key == 'ccVerificationValue' }
-    validate_cvv_indicator_field 2
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'discover', :cvv_indicator_visa_discover => true, :cvv_indicator_override_visa_discover => '2' })
-    @properties.reject! {|property| property.key == 'ccVerificationValue' }
-    validate_cvv_indicator_field 2
-  end
-
-  it 'should set correct indicator for visa and discover if cvv value is not present and cvv_indicator_visa_discover is nil or false' do
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454'})
-    @properties.reject! {|property| property.key == 'ccVerificationValue' }
-    validate_cvv_indicator_field
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'discover'})
-    @properties.reject! {|property| property.key == 'ccVerificationValue' }
-    validate_cvv_indicator_field
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cvv_indicator_visa_discover => false })
-    @properties.reject! {|property| property.key == 'ccVerificationValue' }
-    validate_cvv_indicator_field
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'discover', :cvv_indicator_visa_discover => false })
-    @properties.reject! {|property| property.key == 'ccVerificationValue' }
-    validate_cvv_indicator_field
-  end
-
-  it 'should not include indicator except visa and discover for all cases' do
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'master' })
-    validate_cvv_indicator_field
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'american_express' })
-    validate_cvv_indicator_field
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'master', :cvv_indicator_visa_discover => false })
-    validate_cvv_indicator_field
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'american_express', :cvv_indicator_visa_discover => false })
-    validate_cvv_indicator_field
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'master', :cvv_indicator_visa_discover => true})
-    validate_cvv_indicator_field
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'american_express', :cvv_indicator_visa_discover => true })
-    validate_cvv_indicator_field
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'master' })
-    @properties.reject! {|property| property.key == 'ccVerificationValue' }
-    validate_cvv_indicator_field
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'american_express' })
-    @properties.reject! {|property| property.key == 'ccVerificationValue' }
-    validate_cvv_indicator_field
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'master', :cvv_indicator_visa_discover => false})
-    @properties.reject! {|property| property.key == 'ccVerificationValue' }
-    validate_cvv_indicator_field
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'american_express', :cvv_indicator_visa_discover => false })
-    @properties.reject! {|property| property.key == 'ccVerificationValue' }
-    validate_cvv_indicator_field
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'master', :cvv_indicator_visa_discover => true})
-    @properties.reject! {|property| property.key == 'ccVerificationValue' }
-    validate_cvv_indicator_field
-
-    @properties = build_pm_properties(nil, { :cc_number => '5454545454545454', :cc_type => 'american_express', :cvv_indicator_visa_discover => true })
-    @properties.reject! {|property| property.key == 'ccVerificationValue' }
-    validate_cvv_indicator_field
-  end
-end
-
-
 describe Killbill::Orbital::PaymentPlugin do
 
   include ::Killbill::Plugin::ActiveMerchant::RSpec
@@ -241,7 +27,7 @@ describe Killbill::Orbital::PaymentPlugin do
       @currency   = 'USD'
     end
 
-    include_examples 'common_specs'
+    include_examples 'payment_flow_spec'
   end
 
   context 'custom profile flow' do
@@ -252,63 +38,74 @@ describe Killbill::Orbital::PaymentPlugin do
       @currency   = 'USD'
     end
 
-    include_examples 'common_specs'
+    include_examples 'payment_flow_spec'
   end
 
-  context 'cvv indicator spec' do
+  context 'tokenized credit card flow amex' do
     before(:each) do
-      @properties = build_pm_properties(nil, { :cc_number => '5454545454545454' })
+      cryptogram  = 'EHuWW9PiBkWvqE5juRwDzAUFBAk='
+      @properties = build_pm_properties(nil,
+                                        {
+                                            :cc_number => 378282246310005,
+                                            :cc_type => 'american_express',
+                                            :payment_cryptogram => cryptogram
+                                        })
       @pm         = create_payment_method(::Killbill::Orbital::OrbitalPaymentMethod, nil, @call_context.tenant_id, @properties, {})
       @amount     = BigDecimal.new('100')
       @currency   = 'USD'
     end
 
-    include_examples 'cvv_indicator_specs'
+    include_examples 'payment_flow_spec'
   end
 
-  def successful_authorize_response
-    <<-XML
-<?xml version=\"1.0\" encoding=\"UTF-8\"?><Response><NewOrderResp><IndustryType></IndustryType><MessageType>A</MessageType><MerchantID>1111111</MerchantID><TerminalID>001</TerminalID><CardBrand>CC</CardBrand><AccountNum>XXXXXXXXXXXX5454</AccountNum><OrderID>5b257b31-1f84-44bc-b32</OrderID><TxRefNum>5834AA75E4466AEA59512165057C37DD810053C2</TxRefNum><TxRefIdx>0</TxRefIdx><ProcStatus>0</ProcStatus><ApprovalStatus>1</ApprovalStatus><RespCode>00</RespCode><AVSRespCode>B </AVSRespCode><CVV2RespCode> </CVV2RespCode><AuthCode>tst424</AuthCode><RecurringAdviceCd></RecurringAdviceCd><CAVVRespCode></CAVVRespCode><StatusMsg>Approved</StatusMsg><RespMsg></RespMsg><HostRespCode>100</HostRespCode><HostAVSRespCode>I3</HostAVSRespCode><HostCVV2RespCode>  </HostCVV2RespCode><CustomerRefNum></CustomerRefNum><CustomerName></CustomerName><ProfileProcStatus></ProfileProcStatus><CustomerProfileMessage></CustomerProfileMessage><RespTime>152837</RespTime><PartialAuthOccurred></PartialAuthOccurred><RequestedAmount></RequestedAmount><RedeemedAmount></RedeemedAmount><RemainingBalance></RemainingBalance><CountryFraudFilterStatus></CountryFraudFilterStatus><IsoCountryCode></IsoCountryCode></NewOrderResp></Response>
-    XML
-  end
+  context 'tokenized credit card flow master' do
+    before(:each) do
+      cryptogram  = 'EHuWW9PiBkWvqE5juRwDzAUFBAk='
+      @properties = build_pm_properties(nil,
+                                        {
+                                            :cc_number => 5555555555554444,
+                                            :cc_type => 'master',
+                                            :payment_cryptogram => cryptogram
+                                        })
+      @pm         = create_payment_method(::Killbill::Orbital::OrbitalPaymentMethod, nil, @call_context.tenant_id, @properties, {})
+      @amount     = BigDecimal.new('100')
+      @currency   = 'USD'
+    end
 
-  def successful_purchase_response
-    <<-XML
-<?xml version=\"1.0\" encoding=\"UTF-8\"?><Response><NewOrderResp><IndustryType></IndustryType><MessageType>AC</MessageType><MerchantID>1111111</MerchantID><TerminalID>001</TerminalID><CardBrand>CC</CardBrand><AccountNum>XXXXXXXXXXXX5454</AccountNum><OrderID>88132d30-f4f7-4028-949</OrderID><TxRefNum>5834EAC9C7A53FEB600A479629FB6C6427A2532C</TxRefNum><TxRefIdx>1</TxRefIdx><ProcStatus>0</ProcStatus><ApprovalStatus>1</ApprovalStatus><RespCode>00</RespCode><AVSRespCode>3 </AVSRespCode><CVV2RespCode>I</CVV2RespCode><AuthCode>tst703</AuthCode><RecurringAdviceCd></RecurringAdviceCd><CAVVRespCode></CAVVRespCode><StatusMsg>Approved</StatusMsg><RespMsg></RespMsg><HostRespCode>100</HostRespCode><HostAVSRespCode>  </HostAVSRespCode><HostCVV2RespCode>I</HostCVV2RespCode><CustomerRefNum></CustomerRefNum><CustomerName></CustomerName><ProfileProcStatus></ProfileProcStatus><CustomerProfileMessage></CustomerProfileMessage><RespTime>200306</RespTime><PartialAuthOccurred></PartialAuthOccurred><RequestedAmount></RequestedAmount><RedeemedAmount></RedeemedAmount><RemainingBalance></RemainingBalance><CountryFraudFilterStatus></CountryFraudFilterStatus><IsoCountryCode></IsoCountryCode></NewOrderResp></Response>    XML
-    XML
+    include_examples 'payment_flow_spec'
   end
 
-  def validate_cvv_indicator_field(expected_field = nil)
-    ::ActiveMerchant::Billing::OrbitalGateway.any_instance.stub(:ssl_post) do |host, request_body|
-      if expected_field.nil?
-        request_body.should_not match('<CardSecValInd>')
-      else
-        request_body.should match("<CardSecValInd>#{expected_field}</CardSecValInd>")
-      end
-      if request_body.include? '<MessageType>A</MessageType>'
-        successful_authorize_response
-      else
-        successful_purchase_response
-      end
+  context 'tokenized credit card flow discover' do
+    before(:each) do
+      cryptogram  = 'EHuWW9PiBkWvqE5juRwDzAUFBAk='
+      @properties = build_pm_properties(nil,
+                                        {
+                                            :cc_number => 6011111111111117,
+                                            :cc_type => 'discover',
+                                            :payment_cryptogram => cryptogram
+                                        })
+      @pm         = create_payment_method(::Killbill::Orbital::OrbitalPaymentMethod, nil, @call_context.tenant_id, @properties, {})
+      @amount     = BigDecimal.new('100')
+      @currency   = 'USD'
     end
-    create_payment
-    payment_response = @plugin.authorize_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[0].id, @pm.kb_payment_method_id, @amount, @currency, @properties, @call_context)
-    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
-    payment_response.amount.should == @amount
-    payment_response.transaction_type.should == :AUTHORIZE
 
-    create_payment
-    payment_response = @plugin.purchase_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[0].id, @pm.kb_payment_method_id, @amount, @currency, @properties, @call_context)
-    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
-    payment_response.amount.should == @amount
-    payment_response.transaction_type.should == :PURCHASE
+    include_examples 'payment_flow_spec'
   end
 
-  def create_payment
-    kb_payment_id = SecureRandom.uuid
-    1.upto(6) do
-      @kb_payment = @plugin.kb_apis.proxied_services[:payment_api].add_payment(kb_payment_id)
+  context 'tokenized credit card flow visa' do
+    before(:each) do
+      cryptogram  = 'EHuWW9PiBkWvqE5juRwDzAUFBAk='
+      @properties = build_pm_properties(nil,
+                                        {
+                                            :cc_number => 4112344112344113,
+                                            :cc_type => 'visa',
+                                            :payment_cryptogram => cryptogram
+                                        })
+      @pm         = create_payment_method(::Killbill::Orbital::OrbitalPaymentMethod, nil, @call_context.tenant_id, @properties, {})
+      @amount     = BigDecimal.new('100')
+      @currency   = 'USD'
     end
-    kb_payment_id
+
+    include_examples 'payment_flow_spec'
   end
 end

data/spec/orbital/remote/shared_examples_for_payment_flow.rb

@@ -0,0 +1,120 @@
+shared_examples 'payment_flow_spec' do
+
+  before(:each) do
+    create_payment
+  end
+
+  after(:each) do
+    @plugin.stop_plugin
+  end
+
+  it 'should be able to purchase' do
+    Killbill::Orbital::OrbitalResponse.all.size.should == 1
+    Killbill::Orbital::OrbitalTransaction.all.size.should == 0
+
+    payment_response = @plugin.purchase_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[0].id, @pm.kb_payment_method_id, @amount, @currency, @properties, @call_context)
+    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
+    payment_response.amount.should == @amount
+    payment_response.transaction_type.should == :PURCHASE
+    find_value_from_properties(payment_response.properties, 'processorResponse').should == '100'
+
+    responses = Killbill::Orbital::OrbitalResponse.all
+    responses.size.should == 2
+    responses[0].api_call.should == 'add_payment_method'
+    responses[0].message.should == 'Profile Request Processed'
+    responses[1].api_call.should == 'purchase'
+    responses[1].message.should == 'Approved'
+    transactions = Killbill::Orbital::OrbitalTransaction.all
+    transactions.size.should == 1
+    transactions[0].api_call.should == 'purchase'
+  end
+
+  it 'should be able to charge and refund' do
+    payment_response = @plugin.purchase_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[0].id, @pm.kb_payment_method_id, @amount, @currency, @properties, @call_context)
+    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
+    payment_response.amount.should == @amount
+    payment_response.transaction_type.should == :PURCHASE
+    find_value_from_properties(payment_response.properties, 'processorResponse').should == '100'
+
+    # Try a full refund
+    refund_response = @plugin.refund_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[1].id, @pm.kb_payment_method_id, @amount, @currency, @properties, @call_context)
+    refund_response.status.should eq(:PROCESSED), refund_response.gateway_error
+    refund_response.amount.should == @amount
+    refund_response.transaction_type.should == :REFUND
+  end
+
+  it 'should be able to auth, capture and refund' do
+    payment_response = @plugin.authorize_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[0].id, @pm.kb_payment_method_id, @amount, @currency, @properties, @call_context)
+    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
+    payment_response.amount.should == @amount
+    payment_response.transaction_type.should == :AUTHORIZE
+    find_value_from_properties(payment_response.properties, 'processorResponse').should == '100'
+
+    # Try multiple partial captures
+    partial_capture_amount = BigDecimal.new('10')
+    1.upto(3) do |i|
+      payment_response = @plugin.capture_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[i].id, @pm.kb_payment_method_id, partial_capture_amount, @currency, @properties, @call_context)
+      payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
+      payment_response.amount.should == partial_capture_amount
+      payment_response.transaction_type.should == :CAPTURE
+    end
+
+    # Try a partial refund
+    refund_response = @plugin.refund_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[4].id, @pm.kb_payment_method_id, partial_capture_amount, @currency, @properties, @call_context)
+    refund_response.status.should eq(:PROCESSED), refund_response.gateway_error
+    refund_response.amount.should == partial_capture_amount
+    refund_response.transaction_type.should == :REFUND
+
+    # Try to capture again
+    payment_response = @plugin.capture_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[5].id, @pm.kb_payment_method_id, partial_capture_amount, @currency, @properties, @call_context)
+    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
+    payment_response.amount.should == partial_capture_amount
+    payment_response.transaction_type.should == :CAPTURE
+  end
+
+  it 'should be able to auth and void' do
+    payment_response = @plugin.authorize_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[0].id, @pm.kb_payment_method_id, @amount, @currency, @properties, @call_context)
+    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
+    payment_response.amount.should == @amount
+    payment_response.transaction_type.should == :AUTHORIZE
+    find_value_from_properties(payment_response.properties, 'processorResponse').should == '100'
+
+    payment_response = @plugin.void_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[1].id, @pm.kb_payment_method_id, @properties, @call_context)
+    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
+    payment_response.transaction_type.should == :VOID
+  end
+
+  it 'should be able to auth, partial capture and void' do
+    payment_response = @plugin.authorize_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[0].id, @pm.kb_payment_method_id, @amount, @currency, @properties, @call_context)
+    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
+    payment_response.amount.should == @amount
+    payment_response.transaction_type.should == :AUTHORIZE
+    find_value_from_properties(payment_response.properties, 'processorResponse').should == '100'
+
+    partial_capture_amount = BigDecimal.new('10')
+    payment_response       = @plugin.capture_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[1].id, @pm.kb_payment_method_id, partial_capture_amount, @currency, @properties, @call_context)
+    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
+    payment_response.amount.should == partial_capture_amount
+    payment_response.transaction_type.should == :CAPTURE
+
+    payment_response = @plugin.void_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[2].id, @pm.kb_payment_method_id, @properties, @call_context)
+    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
+    payment_response.transaction_type.should == :VOID
+  end
+
+  it 'should be able to credit' do
+    payment_response = @plugin.credit_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[0].id, @pm.kb_payment_method_id, @amount, @currency, @properties, @call_context)
+    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
+    payment_response.amount.should == @amount
+    payment_response.transaction_type.should == :CREDIT
+  end
+
+  it 'should include host response code' do
+    # Sending a specific amount of 530 will trigger the Do Not Honor error.
+    payment_response = @plugin.authorize_payment(@pm.kb_account_id, @kb_payment.id, @kb_payment.transactions[0].id, @pm.kb_payment_method_id, BigDecimal.new('530'), @currency, @properties, @call_context)
+    payment_response.status.should eq(:ERROR), payment_response.gateway_error
+    payment_response.transaction_type.should == :AUTHORIZE
+    payment_response.amount.should be_nil
+    find_value_from_properties(payment_response.properties, 'processorResponse').should == '530'
+  end
+end

data/spec/orbital/tokenized_creditcard_payload_spec.rb

@@ -0,0 +1,134 @@
+require 'spec_helper'
+
+ActiveMerchant::Billing::Base.mode = :test
+
+describe 'Payment request for network tokenized card' do
+
+  include ::Killbill::Plugin::ActiveMerchant::RSpec
+
+  before(:each) do
+    # Start the plugin early to configure ActiveRecord
+    @plugin = build_plugin(::Killbill::Orbital::PaymentPlugin, 'orbital')
+    @plugin.start_plugin
+
+    ::Killbill::Orbital::OrbitalPaymentMethod.delete_all
+    ::Killbill::Orbital::OrbitalResponse.delete_all
+    ::Killbill::Orbital::OrbitalTransaction.delete_all
+
+    @call_context = build_call_context
+  end
+
+  after(:each) do
+    @plugin.stop_plugin
+  end
+
+  it 'should send correct payload for visa network tokenized card' do
+
+    cryptogram  = 'EHuWW9PiBkWvqE5juRwDzAUFBAk='
+    @properties = build_pm_properties(nil,
+                                     {
+                                         :cc_number => 4111111111111111,
+                                         :cc_type => 'visa',
+                                         :payment_cryptogram => cryptogram
+                                     })
+    @amount     = BigDecimal.new('100')
+    @currency   = 'USD'
+
+    ::ActiveMerchant::Billing::OrbitalGateway.any_instance.stub(:ssl_post) do |host, request_body|
+      request_body.should match("<CAVV>#{cryptogram}</CAVV>")
+      request_body.should match('<DPANInd>Y</DPANInd>')
+      request_body.should match("<DigitalTokenCryptogram>#{cryptogram}</DigitalTokenCryptogram>")
+
+      successful_authorize_response
+    end
+
+    validate_payment
+  end
+
+  it 'should send correct payload for amex network tokenized card' do
+
+    cryptogram  = 'EHuWW9PiBkWvqE5juRwDzAUFBAk='
+    @properties = build_pm_properties(nil,
+                                      {
+                                          :cc_number => 378282246310005,
+                                          :cc_type => 'american_express',
+                                          :payment_cryptogram => cryptogram,
+                                          :eci => '7',
+                                      })
+    @amount     = BigDecimal.new('100')
+    @currency   = 'USD'
+
+    ::ActiveMerchant::Billing::OrbitalGateway.any_instance.stub(:ssl_post) do |host, request_body|
+      request_body.should match("<AEVV>#{cryptogram}</AEVV>")
+      request_body.should match('<DPANInd>Y</DPANInd>')
+      request_body.should match("<DigitalTokenCryptogram>#{cryptogram}</DigitalTokenCryptogram>")
+      request_body.should match('<AuthenticationECIInd>7</AuthenticationECIInd>')
+
+      successful_authorize_response
+    end
+
+    validate_payment
+  end
+
+  it 'should send correct payload for master network tokenized card' do
+
+    cryptogram  = 'EHuWW9PiBkWvqE5juRwDzAUFBAk='
+    @properties = build_pm_properties(nil,
+                                      {
+                                          :cc_number => 5555555555554444,
+                                          :cc_type => 'master',
+                                          :payment_cryptogram => cryptogram,
+                                      })
+    @amount     = BigDecimal.new('100')
+    @currency   = 'USD'
+
+    ::ActiveMerchant::Billing::OrbitalGateway.any_instance.stub(:ssl_post) do |host, request_body|
+      request_body.should match("<AAV>#{cryptogram}</AAV>")
+      request_body.should match('<DPANInd>Y</DPANInd>')
+      request_body.should match("<DigitalTokenCryptogram>#{cryptogram}</DigitalTokenCryptogram>")
+
+      successful_authorize_response
+    end
+
+    validate_payment
+  end
+
+  it 'should send correct payload for discover network tokenized card' do
+
+    cryptogram  = 'EHuWW9PiBkWvqE5juRwDzAUFBAk='
+    @properties = build_pm_properties(nil,
+                                      {
+                                          :cc_number => 6011111111111117,
+                                          :cc_type => 'discover',
+                                          :payment_cryptogram => cryptogram
+                                      })
+    @amount     = BigDecimal.new('100')
+    @currency   = 'USD'
+
+    ::ActiveMerchant::Billing::OrbitalGateway.any_instance.stub(:ssl_post) do |host, request_body|
+      request_body.should match('<DPANInd>Y</DPANInd>')
+      request_body.should match("<DigitalTokenCryptogram>#{cryptogram}</DigitalTokenCryptogram>")
+
+      successful_authorize_response
+    end
+
+    validate_payment
+  end
+
+  private
+
+  def successful_authorize_response
+    <<-XML
+<?xml version=\"1.0\" encoding=\"UTF-8\"?><Response><NewOrderResp><IndustryType></IndustryType><MessageType>A</MessageType><MerchantID>1111111</MerchantID><TerminalID>001</TerminalID><CardBrand>CC</CardBrand><AccountNum>XXXXXXXXXXXX5454</AccountNum><OrderID>5b257b31-1f84-44bc-b32</OrderID><TxRefNum>5834AA75E4466AEA59512165057C37DD810053C2</TxRefNum><TxRefIdx>0</TxRefIdx><ProcStatus>0</ProcStatus><ApprovalStatus>1</ApprovalStatus><RespCode>00</RespCode><AVSRespCode>B </AVSRespCode><CVV2RespCode> </CVV2RespCode><AuthCode>tst424</AuthCode><RecurringAdviceCd></RecurringAdviceCd><CAVVRespCode></CAVVRespCode><StatusMsg>Approved</StatusMsg><RespMsg></RespMsg><HostRespCode>100</HostRespCode><HostAVSRespCode>I3</HostAVSRespCode><HostCVV2RespCode>  </HostCVV2RespCode><CustomerRefNum></CustomerRefNum><CustomerName></CustomerName><ProfileProcStatus></ProfileProcStatus><CustomerProfileMessage></CustomerProfileMessage><RespTime>152837</RespTime><PartialAuthOccurred></PartialAuthOccurred><RequestedAmount></RequestedAmount><RedeemedAmount></RedeemedAmount><RemainingBalance></RemainingBalance><CountryFraudFilterStatus></CountryFraudFilterStatus><IsoCountryCode></IsoCountryCode></NewOrderResp></Response>
+    XML
+  end
+
+  def validate_payment
+    kb_payment_id, kb_transaction_id = create_payment
+    payment_response = @plugin.authorize_payment(SecureRandom.uuid, kb_payment_id, kb_transaction_id, SecureRandom.uuid, @amount, @currency, @properties, @call_context)
+    payment_response.status.should eq(:PROCESSED), payment_response.gateway_error
+    payment_response.amount.should == @amount
+    payment_response.transaction_type.should == :AUTHORIZE
+  end
+
+end

data/spec/spec_helper.rb

@@ -22,3 +22,11 @@ ActiveRecord::Base.establish_connection(
 # Create the schema
 require File.expand_path(File.dirname(__FILE__) + '../../db/schema.rb')
 
+def create_payment
+  kb_payment_id = SecureRandom.uuid
+  1.upto(6) do
+    @kb_payment = @plugin.kb_apis.proxied_services[:payment_api].add_payment(kb_payment_id)
+  end
+  [kb_payment_id, @kb_payment.transactions[0].id]
+end
+

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: killbill-orbital
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Kill Bill core team
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2017-01-19 00:00:00.000000000 Z
+date: 2017-02-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -317,7 +317,10 @@ files:
 - pom.xml
 - release.sh
 - spec/orbital/base_plugin_spec.rb
+- spec/orbital/cvv_indicator_spec.rb
 - spec/orbital/remote/integration_spec.rb
+- spec/orbital/remote/shared_examples_for_payment_flow.rb
+- spec/orbital/tokenized_creditcard_payload_spec.rb
 - spec/spec_helper.rb
 homepage: http://killbill.io
 licenses: