mpi_client 0.0.1 → 0.0.2

data/lib/mpi_client/mpi_response.rb

@@ -0,0 +1,15 @@
+class MPIResponse
+  attr_reader   :data
+
+  def initialize(data = {})
+    @data = data
+  end
+
+  def success?
+    error_code.nil? && error_message.nil?
+  end
+
+  def method_missing(method)
+    data[method]
+  end
+end

data/lib/mpi_client/option_translator.rb

@@ -0,0 +1,30 @@
+class OptionTranslator
+  OPTION_MAP = {
+    :id           => :Id,
+    :site_name    => :Name,
+    :url          => :URL,
+    :cert_subject => :IP,
+    :acquirer_bin => :AcquirerBIN,
+    :country_code => :CountryCode,
+    :password     => :Password,
+    :pub_cert     => :PublicCertificate,
+    :priv_key     => :PrivateKey,
+    :directory_server_url => :DirectoryServerURL,
+    :brand        => :CardType,
+    :response_url => :ResponseURL,
+    :client_url   => :ClientURL,
+    :term_url     => :TermURL,
+    :account_id   => :AccountId,
+    :error        => :Error,
+    :card_number  => :CardNumber,
+    :status       => :status
+  }
+
+  def self.to_client(option)
+    OPTION_MAP.invert[option]
+  end
+
+  def self.to_server(option)
+    OPTION_MAP[option]
+  end
+end

data/lib/mpi_client.rb

@@ -15,18 +15,19 @@ class MPIClient
   end
 
   def method_missing(method, *args)
-    submit_request(method, args.first) if CLIENT_METHODS.include?(method.to_s)
+    submit_request(method, *args) if CLIENT_METHODS.include?(method.to_s)
   end
 
   private
-  def submit_request(request_type, options)
-    parse_response(@connection.post(prepare_request_data(request_type, options)))
+  def submit_request(request_type, *args)
+    parse_response(@connection.post(prepare_request_data(request_type, *args)))
   end
 
-  def prepare_request_data(request_type, options)
+  def prepare_request_data(request_type, options, transaction_attrs = {})
+    options = options.dup
     builder = Nokogiri::XML::Builder.new(:encoding => 'UTF-8') do |xml|
       xml.REQUEST(:type => request_type) do |xml|
-        xml.Transaction do |xml|
+        xml.Transaction(transaction_attrs) do |xml|
           options.each { |k,v|  xml.send(OptionTranslator.to_server(k), v) }
         end
       end
@@ -49,4 +50,4 @@ class MPIClient
 
     MPIResponse.new(response)
   end
-end
+end

data/remote_spec/mpi_client_spec.rb

@@ -0,0 +1,49 @@
+require File.dirname(__FILE__) + '/../lib/mpi_client.rb'
+
+MPI_SERVER_URL = 'http://192.168.65.11/xml'
+
+describe "MPIClient requests" do
+  before(:each) do
+    @mpi_client = MPIClient.new(MPI_SERVER_URL)
+  end
+
+  it "should create account, update, get_info and delete" do
+    options = {
+      :id           => 'some id',
+      :site_name    => 'site name',
+      :url          => 'http://siteurl/',
+      :cert_subject => '*',
+      :acquirer_bin => '3213',
+      :country_code => '432',
+      :password     => 'qwerty',
+      :directory_server_url => 'http://www.visa.com/ds/',
+      :brand        => '1',
+      :response_url => 'http://www.response.com/gk/',
+      :client_url   => 'http://www.client.com/',
+      :term_url     => 'http://www.term.com/',
+    }
+
+    account_id = @mpi_client.create_account(options).account_id
+
+    options.merge!({:account_id => account_id, :site_name => 'new_site_name'})
+
+    @mpi_client.update_account(options).account_id.should == account_id
+    @mpi_client.get_account_info({ :account_id => account_id }).site_name.should == options[:site_name]
+    @mpi_client.delete_account({ :account_id => account_id }).account_id.should  == account_id
+  end
+  
+  describe "card enrollment check" do
+    it "should return Y if card is enrolled" do
+      enrollment_response('4012001037141112').status.should == 'Y'
+    end
+
+    it "should return N if card isn't enrolled" do
+      enrollment_response('4012001038443335').status.should == 'N'
+    end
+
+    def enrollment_response(card_number)
+      @mpi_client.enrolled({:account_id=> '0'*32, :card_number=> card_number})
+    end
+  end
+  
+end

data/spec/mpi_client_spec.rb

@@ -18,14 +18,20 @@ describe "MPIClient" do
     options = { :id => 'one', :account_id => 'account id' }
     request_type = :some_type
     result = @client.send(:prepare_request_data, request_type, options)
-    result.should match %r{<REQUEST\ type="#{request_type}">}
+    result.should match %r{<REQUEST type="#{request_type}">}
 
     options.each do |k,v| 
       key = OptionTranslator.to_server(k)
       result.should match %r{<#{key}>#{v}</#{key}>}
     end
   end
-
+  
+  it "should prepare transaction attributes" do
+    id = 10
+    result = @client.send(:prepare_request_data, 'any', {}, {:id => id} )
+    result.should match %r{<Transaction id="#{id}".?>}
+  end
+  
   it "should parse input XML to MPIResponse and convert keys to client format" do
     response = <<-RESPONSE
     <REQUEST type=\"update_account\">

data/spec/mpi_response_spec.rb

@@ -0,0 +1,31 @@
+require File.dirname(__FILE__) + '/../lib/mpi_client.rb'
+
+describe "MPIResponse" do
+  it "should contain data" do
+    response_data = {:a => 1, :b => 2}
+    MPIResponse.new(response_data).data.should == response_data
+  end
+
+  describe "automatic accessors" do
+    it "should return 'data' hash value" do
+      response = MPIResponse.new({:status => 'Y'})
+      response.status.should == response.data[:status]
+    end
+
+    it "should return nil if there is no value in 'data' hash" do
+      response = MPIResponse.new()
+      response.status.should be_nil
+    end
+  end
+
+  describe "success?" do
+    it "should be true if error code and error message is nil" do
+      MPIResponse.new.should be_success
+    end
+
+    it "should be false if error code or error message not nil" do
+      response = MPIResponse.new(:error_code => "C3", :error_message => 'Somthing went wrong.')
+      response.should_not be_success
+    end
+  end
+end

data/spec/option_translator_spec.rb

@@ -0,0 +1,25 @@
+require File.dirname(__FILE__) + '/../lib/mpi_client.rb'
+
+describe "OptionTranslator" do
+  it "should translate options" do
+    { :id           => :Id,
+      :site_name    => :Name,
+      :url          => :URL,
+      :cert_subject => :IP,
+      :acquirer_bin => :AcquirerBIN,
+      :country_code => :CountryCode,
+      :password     => :Password,
+      :pub_cert     => :PublicCertificate,
+      :priv_key     => :PrivateKey,
+      :directory_server_url => :DirectoryServerURL,
+      :brand        => :CardType,
+      :response_url => :ResponseURL,
+      :client_url   => :ClientURL,
+      :term_url     => :TermURL,
+      :account_id   => :AccountId
+    }.each do |client_option, server_option|
+      OptionTranslator.to_client(server_option).should == client_option
+      OptionTranslator.to_server(client_option).should == server_option
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: mpi_client
 version: !ruby/object:Gem::Version 
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors: 
 - Dmitry Plashchynski
@@ -42,7 +42,13 @@ extensions: []
 extra_rdoc_files: []
 
 files: 
+- lib/mpi_client/mpi_response.rb
+- lib/mpi_client/option_translator.rb
 - lib/mpi_client.rb
+- remote_spec/mpi_client_spec.rb
+- spec/mpi_client_spec.rb
+- spec/mpi_response_spec.rb
+- spec/option_translator_spec.rb
 has_rdoc: true
 homepage: http://github.com/plashchynski/mpi_client/
 licenses: []
@@ -67,9 +73,9 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.3.5
+rubygems_version: 1.3.4
 signing_key: 
 specification_version: 3
 summary: MPI client library
-test_files: 
-- spec/mpi_client_spec.rb
+test_files: []
+