ruby-saferpay 0.0.2 → 0.0.3

data/lib/ruby-saferpay.rb

@@ -9,6 +9,7 @@ require 'net/https'
 require 'uri'
 require 'net/https'
 require	'hpricot'
+require 'ostruct'
 
 class Saferpay
 	
@@ -53,13 +54,15 @@ class Saferpay
 			{:code => 301, :short_message => 'Authentication Error', :long_message => 'An error happened during the authentication request. The merchant application could choose to:\n- continue the payment without authentication or\n- ask customer for other payment method or\n- stop the payment.'}			
 		]
 		attr_reader :error
-		def initialize(saferpay_reply)
-			if saferpay_reply.is_a? String
-				result	=	Hpricot.parse(cmd_result).at('idp').attributes['result'].to_i rescue 152
-			elsif saferpay_reply.is_a? Hash
-				result	=	saferpay_reply['result'].to_i rescue 152
-			elsif saferpay_reply.is_a? Fixnum
-				result	=	saferpay_reply
+		def initialize(saferpay_response)
+			if saferpay_response.is_a? String
+				result	=	Hpricot.parse(saferpay_response).at('idp').attributes['result'].to_i rescue 152
+			elsif saferpay_response.is_a? Hash
+				result	=	saferpay_response['result'].to_i rescue 152
+			elsif saferpay_response.is_a? Fixnum
+				result	=	saferpay_response
+			elsif saferpay_response.is_a? OpenStruct
+				result	=	saferpay_response.response
 			else
 				result	=	152 # Unknown error
 			end
@@ -82,17 +85,17 @@ class Saferpay
 	CURRENCIES	=	['CHF','CZK','DKK','EUR','GBP','PLN','SEK','USD']
 	
 	attr_accessor	:account_id
-	attr_reader	:current_transaction
+	attr_reader	:current_request
 	
-	def initialize( account_id = "99867-94913159", pan = "9451123100000004", expiry_date = nil, cvc = nil, name = nil, tolerance = 0 )
+	def initialize( account_id = "99867-94913159", pan = "9451123100000004", expiry_date = '1107', cvc = '', name = '', tolerance = 0 )
 		Saferpay.check_install
-		@current_transaction	=	nil
+		@current_request	=	nil
 		
-		@account_id	=	account_id
-		@pan				=	pan
-		@expiry_date=	expiry_date
-		@cvc				=	cvc
-		@name				=	name
+		@account_id	=	account_id.gsub(/[\s]/,'').chomp
+		@pan				=	pan.gsub(/[\s]/,'').chomp
+		@expiry_date=	expiry_date.gsub(/[\s]/,'').chomp
+		@cvc				=	cvc.gsub(/[\s]/,'').chomp
+		@name				=	name.chomp
 		@tolerance	=	tolerance	# Amount tolerance in percent. The finally captured amount is AMOUNT + TOLERANCE
 	end
 	
@@ -107,61 +110,48 @@ class Saferpay
 	end
 	
 	def details( transaction_id = nil )
-		transaction_id =  @current_transaction.transaction_id if transaction_id.nil?
-		cmd	=	"#{BASEDIR}#{EXECUTABLE} -exec -p #{CONFIG} -m Inquiry -a ID #{transaction_id} -a TYPE Transaction -a TOKEN \"(not used)\""
-		#logger.debug "#{self.class}#details COMMAND: #{cmd}"
-		do_direct_cc cmd
+		transaction_id =  @current_request.transaction_id if transaction_id.nil?
+		raise ArgumentError, "Cannot retrieve transaction details without a transaction id." if transaction_id.nil?
+		req =	Request.new(:inquiry, :transaction_id => transaction_id)
+		do_direct_cc req.cmd
 	end
 	
 	def reserve( amount, currency )
 		check_params amount, currency
-		@current_transaction	=	TransactionParams.new(
-					:amount				=>	amount,
-					:currency			=>	currency,
-					:accountid		=>	@account_id,
-					:pan					=>	@pan,
-					:exp					=>	@expiry_date
-					)
-		do_direct_cc
+		@current_request 	=	Request.new(:reserve, :amount => amount, :currency => currency, :accountid => @account_id, :pan => @pan, :exp => @expiry_date)
+		do_direct_cc( @current_request.cmd )
 	end
 
 	def refund( amount, currency )
 		check_params amount, currency
-		@current_transaction	=	TransactionParams.new(
-					:amount				=>	amount,
-					:currency			=>	currency,
-					:accountid		=>	@account_id,
-					:pan					=>	@pan,
-					:exp					=>	@expiry_date,
-					:action				=>	"Credit"
-					)
-		do_direct_cc
+		@current_request	=	Request.new(:refund, :amount => amount, :currency => currency, :accountid => @account_id, :pan => @pan, :exp => @expiry_date)
+		do_direct_cc @current_request.cmd
 	end
 	
 	def refund_last
-		refund_transaction @current_transaction.transaction_id
+		raise ArgumentError, "No current request to refund." if @current_request.transaction_id.nil?
+		refund_transaction @current_request.transaction_id
 	end
 	
-	def refund_transaction( transaction_id )
-		cmd			=	"#{BASEDIR}#{EXECUTABLE} -capt -p #{CONFIG} -i #{transaction_id} -t \"(not used)\""
-		do_direct_cc cmd
+	def refund_transaction( transaction_id = nil )
+		raise ArgumentError, "Cannot refund without transaction_id" if transaction_id.nil?
+		@current_request	=	Request.new(:refund_transaction, :transaction_id => transaction_id)
+		do_direct_cc @current_request.cmd
 	end	
 
 	# Capture the amount of the transaction with id transaction_id
 	# Defaults to capturing the current transaction's amount.
 	def capture( transaction_id = nil, token = nil, params = nil )
-	  transaction_id =  @current_transaction.transaction_id if transaction_id.nil?
-	  token          =  @current_transaction.token if token.nil?
-	  cmd			=	"#{BASEDIR}#{EXECUTABLE} -capt -p #{CONFIG} -i #{transaction_id} -t \"#{token}\" #{params}"
-		do_direct_cc cmd
+	  transaction_id =  @current_request.transaction_id if transaction_id.nil?
+		@current_request	=	Request.new( :capture, :transaction_id => transaction_id )
+		do_direct_cc @current_request.cmd
 	end
 	
 	def cancel( transaction_id = nil, token = nil )
-	  transaction_id  = @current_transaction.transaction_id if transaction_id.nil?
-	  token           = @current_transaction.token if token.nil?
-	  params          = "-a ACTION Cancel"
-	  
-		capture transaction_id, token, params
+	  transaction_id  = @current_request.transaction_id if transaction_id.nil?
+	  token           = @current_request.token if token.nil?
+		@current_request	=	Request.new( :cancel, :transaction_id => transaction_id )
+		do_direct_cc @current_request.cmd
 	end
 
 	# Direct debit on aquierer's bank account.
@@ -172,13 +162,10 @@ class Saferpay
 		check_params amount, currency
 		raise AttributeError, "Account number is a ten digit number" unless account_number.length == 10
 		raise AttributeError, "BLZ is a eight digit number" unless blz.length == 8
-		@current_transaction	=	TransactionParams.new(
-					:amount				=>	amount,
-					:currency			=>	currency,
-					:accountid		=>	@account_id,
-					:track2				=>	"\";59#{blz}=#{account_number}\""
-					)
-		do_direct_cc		
+
+		track2	=	"\";59#{blz}=#{account_number}\""	# Weird, huh?
+		@current_request	=	Request.new(:debit_card_reserve, :amount => amount, :currency => currency, :accountid => @account_id, :track2 => track2 )
+		do_direct_cc @current_request.cmd
 	end
 	alias :lastschrift :debit_card_reserve
 	
@@ -187,7 +174,7 @@ class Saferpay
 	# couple of more methods to handle the paycomplete and other types of messages.
 	# Take it for what it is: a stub)
 	def payinit( amount, currency, description, backlink = 'http://localhost', faillink = 'http://localhost', successlink = 'http://localhost', notifyurl = 'http://localhost')
-		@current_transaction	=	TransactionParams.new(
+		@current_request	=	TransactionParams.new(
 					:amount				=>	amount,
 					:currency			=>	currency,
 					:description	=>	description,
@@ -197,7 +184,7 @@ class Saferpay
 					:successlink	=>	successlink,
 					:notifyurl		=>	notifyurl
 					)
-		cmd	=	"#{BASEDIR}#{EXECUTABLE} -payinit -p #{CONFIG} -a #{@current_transaction.to_s}"
+		cmd	=	"#{BASEDIR}#{EXECUTABLE} -payinit -p #{CONFIG} -a #{@current_request.to_s}"
 		logger.debug "#{self.class}#payinit Will execute command:\n#{cmd}"
 		payinit	=	%x{#{cmd} 2>&1}
 
@@ -244,33 +231,45 @@ class Saferpay
 	
 private
 	
-	# Execute the saferpay ommand. Defaults to the most common type of command.
-	# TODO: should return a Transaction object
-	def do_direct_cc( cmd = nil )
-		cmd	=	"#{BASEDIR}#{EXECUTABLE} -exec -p #{CONFIG} -m Authorization -a #{@current_transaction.to_s}" if cmd.nil?
+	# Execute the saferpay command and return a response object (an OpenStruct with the attributes in the <IDP/> element as methods)
+	# TODO: clean this mess
+	# TODO: should raise or return response with success == false?
+	# TODO: get rid of OpenStruct and populate a Request object instead (all calls to this method initializes the @current_request ivar, _except_ details() that need to work out-of-context)
+	# TODO: move some stuff to class methods?
+	def do_direct_cc( cmd )
+		#	cmd	=	"#{BASEDIR}#{EXECUTABLE} -exec -p #{CONFIG} -m Authorization -a #{@current_request.to_s}" if cmd.nil?
 		cmd_result	=	%x{#{cmd} 2>&1}
+		#logger.debug "#{self.class}#do_direct_cc Got a result:\n\t#{cmd_result}\n\tCommand: #{cmd}\n-------------------\n"
 
 		unless $?.success?
 			logger.error "#{self.class}#do_direct_cc FAILED \nError status: #{$?.inspect}\nCommand:#{cmd}\nCommand result: #{cmd_result}"
 			raise TransactionError.new(cmd_result)
 		else
 		  unless cmd_result.empty?
-				#logger.debug "#{self.class}#do_direct_cc Got a result:\n\t#{cmd_result}\n\tCommand: #{cmd}\n-------------------\n"
-  			result	=	Hpricot.parse(cmd_result).at('idp').attributes # A hash
-  			if result['result'] === "0"
-					@current_transaction.transaction_id =	result['id']
-					@current_transaction.token =	result['token']
-				elsif result['msgtype'] == 'InquiryResponse'
-					# TODO: check for valid InquiryResponse message
+				response_hsh	=	Hpricot.parse(cmd_result).at('idp').attributes # A hash
+  			response			=	OpenStruct.new( response_hsh.merge('success' => false, :transaction_id => response_hsh['id']) )
+				#logger.debug "#{self.class}#do_direct_cc Have response object?\n\tresponse: #{response.inspect}\n\tOriginal hash: #{response_hsh.inspect}\n\tSTDOUT: #{cmd_result}"
+  			if response.result === "0"
+					response.success	=	true
+					@current_request.transaction_id =	response.transaction_id
+					@current_request.token =	response.token
+				elsif response.msgtype == 'InquiryResponse'
+					if response.transaction_id
+						response.success	=	true
+						response.result		=	"0"
+					else
+	  				logger.error "#{self.class}#do_direct_cc FAILED INQUIRY \nError status: #{$?.inspect}\nCommand:#{cmd}\nCommand result: #{cmd_result}"
+						raise TransactionError.new(response)
+					end
 				else
   				logger.error "#{self.class}#do_direct_cc FAILED \nError status: #{$?.inspect}\nCommand:#{cmd}\nCommand result: #{cmd_result}"
-					raise TransactionError.new(result)
+					raise TransactionError.new(response)
   			end
   		else
-				result =	true
+				response =	OpenStruct.new(:response => '0', 'success?' => true)
 		  end
 		end
-		result
+		response
 	end
 	
 	def check_params(amount, currency) #:nodoc:
@@ -340,18 +339,94 @@ private
 		true	
 	end
 
-	class TransactionParams
-		DEFAULTS	=	{}
+	class Request
+		attr_reader :cmd, :type, :accountid, :account_id, :amount, :currency, :pan, :exp, :expiry_date, :cvc, :track2
+		attr_accessor	:transaction_id, :token
+		def initialize(type = :authorization, args = {})
+			logger	=	Saferpay.logger
+			# logger.debug "#{self.class}#initialize ARGS: #{args.inspect}"
+			# logger.debug "#{self.class}#initialize #{type.to_s.capitalize}. Required attributes are:...."
+			opts	= nil
+			attrs	=	nil
+			mess	=	nil
+			case type
+			when :authorization, :auth, :reserve
+				op			=	'exec'
+				mess		=	'Authorization'
+				attrs		=	{:amount => args[:amount], :currency => args[:currency], :accountid => args[:accountid], :pan => args[:pan], :exp => args[:exp]}
+			when :debit_card_reserve, :lastschrift
+				op			=	'exec'
+				mess		=	'Authorization'
+				attrs		=	{:amount => args[:amount], :currency => args[:currency], :accountid => args[:accountid], :track2 => args[:track2]}
+			when :capture
+				raise ArgumentError, "#{type.to_s.capitalize}: missing required parameter \"transaction_id\"" unless args[:transaction_id]
+				op			=	'capt'
+				opts		=	"-i #{args[:transaction_id]} -t \"(not used)\""	
+			when :refund
+				op			=	'exec'
+				mess		=	'Authorization'
+				attrs		=	{:action => 'Debit', :amount => args[:amount], :currency => args[:currency], :accountid => args[:accountid], :pan => args[:pan], :exp => args[:exp]}
+			when :refund_transaction
+				raise ArgumentError, "#{type.to_s.capitalize}: missing required parameter \"transaction_id\"" unless args[:transaction_id]
+				op			=	'capt'
+				opts		=	"-i #{args[:transaction_id]} -t \"(not used)\""
+			when :cancel
+				raise ArgumentError, "#{type.to_s.capitalize}: missing required parameter \"transaction_id\"" unless args[:transaction_id]
+				op			=	'capt'
+				opts		=	"-i #{args[:transaction_id]} -t \"(not used)\""	
+				attrs		=	{:action => 'Cancel'}
+			when :inquiry, :details, :info
+				op		=	'exec'
+				mess	=	'Inquiry'
+				attrs		=	{:type => 'Transaction', :id => args[:transaction_id]}
+			end
+			message			=	mess.nil? ? '' : "-m #{mess}"
 
-		attr_accessor :params, :transaction_id, :token
-		def initialize( params )
-			@params					=	DEFAULTS.merge( params )
-			@transaction_id	=	nil
-			@token				  = nil
+			# Check for missing values and complain; Compile attributes string
+			if attrs
+				attrs.each{|attr, value|
+					raise ArgumentError, "#{type.to_s.capitalize}: missing required parameter \"#{attr}\"" if(value.nil? || value.to_s.empty? || (value.is_a?(Fixnum) && value == 0))
+					}
+				attrs	=	'-a ' + attrs.map{|k,v| "#{k.to_s.upcase} #{v}"	}.join(" -a ")
+			end
+			
+			# TODO: fill all ivars and class_eval them to be attr_reader-able
+			@amount					=	args[:amount]
+			@pan						=	args[:pan]
+			@exp						=	args[:exp]; @expiry_date =	@exp
+			@currency				=	args[:currency]
+			@cvc						=	args[:cvc]
+			@track2					=	args[:track2]
+			@accountid			=	args[:accountid]; @account_id =	@accountid
+			@transaction_id	=	args[:transaction_id]
+			@type	=	type
+			@cmd	=	"#{Saferpay::BASEDIR}#{Saferpay::EXECUTABLE} -#{op} -p #{Saferpay::CONFIG} #{message} #{opts} #{attrs}"
 		end
 		
-		def to_s
-			@params.map{|k,v| "#{k.to_s.upcase} #{v}"}.join " -a "
+		def to_hash
+			self.instance_variables.inject({}){|hsh,ivar| hsh.merge(ivar.gsub(/@/,'').to_sym => self.instance_variable_get(ivar))}
 		end
+		
 	end
-end
+end
+# Example response messages
+# Sucessful auth:
+# <IDP MSGTYPE="AuthorizationResponse" AUTHMESSAGE="" ACCOUNTID="99867-94913159" PROVIDERID="90" PROVIDERNAME="Saferpay Test Card" CONTRACTNUMBER="123456789" CCCOUNTRY="XX" ID="3vQ3nvbp4bM8SAlEtd1lbAM0SSnA" TOKEN="(unused)" RESULT="0" AUTHCODE="074869"/>
+# Failed auths
+# <IDP MSGTYPE="AuthorizationResponse" MESSAGE="request was not processed" AUTHMESSAGE="Access Denied" ACCOUNTID="99867-9491" RESULT="5"/>
+# <IDP MSGTYPE="AuthorizationResponse" MESSAGE="request was not processed" AUTHMESSAGE="currency &apos;EURR&apos; is unknown" ACCOUNTID="99867-94913159" RECEIPTDATA="" RESULT="83"/>
+# <IDP MSGTYPE="AuthorizationResponse" MESSAGE="request was not processed" AUTHMESSAGE="no contract available" ACCOUNTID="99867-94913159" RECEIPTDATA="" RESULT="67"/>
+# Sucessful inquiry
+# <IDP MSGTYPE="InquiryResponse" ID="3vQ3nvbp4bM8SAlEtd1lbAM0SSnA" PAN="9451123100000004" TRACK2=";9451123100000004=0711?0" EXP="0711" CCCOUNTRY="XX" 
+#		CVC="no" ACCOUNTID="99867-94913159" CONTRACTNUMBER="123456789" AMOUNT="300" CURRENCY="EUR" PROVIDERID="90" PROVIDERNAME="Saferpay Test Card" 
+#		AUTHCODE="074869" AUTHMESSAGE="" AUTHDATE="20070815 17:37:12" APPLICATION="Saferpay Card Authorization Interface" ECI="0" SETTLED="no" ACTION="Debit" 
+#		CANCELLED="no" CLOSED="no"/>
+# Failed inquiry
+# "" (empty if ID is bad)
+# <IDP MSGTYPE="InquiryResponse" MESSAGE="request was not processed"/>
+# Sucessful capture:
+# ""
+# Successful cancel
+# ""
+# Failed cancel
+# ERROR: 0x80040213 (the request failed)

data/lib/ruby-saferpay/version.rb

@@ -2,7 +2,7 @@ module RubySaferpay #:nodoc:
   module VERSION #:nodoc:
     MAJOR = 0
     MINOR = 0
-    TINY  = 2
+    TINY  = 3
 
     STRING = [MAJOR, MINOR, TINY].join('.')
   end

data/test/test_ruby-saferpay.rb

@@ -57,6 +57,16 @@ class SaferpayTest < Test::Unit::TestCase
 		assert_raise( NoMethodError	)		{ @sfp2.refund_last	}
 	end
 	
+	def test_refund_transcation
+		assert_nothing_raised{
+			res	=	@sfp.reserve( 6666, 'EUR' )
+			@sfp.refund_transaction( res.transaction_id )
+		}
+		
+		assert_raise( ArgumentError )		{ @sfp.refund_transaction	}
+	end
+	
+	
 	def test_capture
 		@sfp.reserve( 600, 'EUR' )
 		assert @sfp.capture
@@ -69,48 +79,48 @@ class SaferpayTest < Test::Unit::TestCase
 		@sfp.capture
 		assert @sfp.cancel
 	end
-	
-	def test_debit_card_reserve
-		assert_raise( ArgumentError ) { @sfp.debit_card_reserve	}
-		assert_raise( Saferpay::TransactionError )	{ 
-			@sfp.debit_card_reserve(100200, "EUR", "1234567890", "12345678")
-			assert_is_authorization_response res
-		}		
-	end
-	
-	def test_details
-		assert_raise ( NoMethodError ) { @sfp.details }
-		assert_nothing_raised{
-			@sfp.reserve( 3000, "USD" )
-			@sfp.capture
-			@sfp.details
-		}
-		
-		assert_nothing_raised{
-			res	=	@sfp.reserve( 1000, "USD" )
-			@sfp.capture
-			
-			res2 = @sfp.reserve( 1100, "USD" )
-			@sfp.capture
-			
-			assert_not_equal @sfp.details( res['id'] ), @sfp.details( res2['id'] )
-			assert_not_equal @sfp.details( res['id'] ), @sfp.details
-		}		
-	end
-	
-	def test_pay
-		res	=	@sfp.pay( 4000 )
-		assert_is_inquiry_response res
-	end
+	# 
+	# def test_debit_card_reserve
+	# 	assert_raise( ArgumentError ) { @sfp.debit_card_reserve	}
+	# 	assert_raise( Saferpay::TransactionError )	{ 
+	# 		@sfp.debit_card_reserve(100200, "EUR", "1234567890", "12345678")
+	# 		assert_is_authorization_response res
+	# 	}		
+	# end
+	# 
+	# def test_details
+	# 	assert_raise ( NoMethodError ) { @sfp.details }
+	# 	assert_nothing_raised{
+	# 		@sfp.reserve( 3000, "USD" )
+	# 		@sfp.capture
+	# 		@sfp.details
+	# 	}
+	# 	
+	# 	assert_nothing_raised{
+	# 		res	=	@sfp.reserve( 1000, "USD" )
+	# 		@sfp.capture
+	# 		
+	# 		res2 = @sfp.reserve( 1100, "USD" )
+	# 		@sfp.capture
+	# 		
+	# 		assert_not_equal @sfp.details( res.transaction_id ), @sfp.details( res2.transaction_id )
+	# 		assert_not_equal @sfp.details( res.transaction_id ), @sfp.details
+	# 	}		
+	# end
+	# 
+	# def test_pay
+	# 	res	=	@sfp.pay( 4000 )
+	# 	assert_is_inquiry_response res
+	# end
 	
 	
 private
 	def assert_is_inquiry_response(res)
-		assert_instance_of Hash, res
+		assert_instance_of OpenStruct, res
 		assert_is_saferpay_response res
 		require_these	=	[:pan, :track2, :exp, :cccountry, :cvc, :accountid, :contractnumber, :amount, :currency, :providerid, :providername, :authcode,:authmessage, :authdate, :application,  :eci, :settled, :settledate, :action, :cancelled, :closed]
-		assert	require_these.all?{ |param| res.has_key? param.to_s },	"Required parameter is missing in response: #{res.inspect}"
-		assert res['msgtype']	== "InquiryResponse",	"Msgtype is not \"InquiryResponse\""
+		assert	require_these.all?{ |param| res.respond_to? param.to_s },	"Required parameter is missing in response: #{res.inspect}"
+		assert res.msgtype	== "InquiryResponse",	"Msgtype is not \"InquiryResponse\""
 	end
 
 	#<IDP MSGTYPE="InquiryResponse" 
@@ -138,25 +148,25 @@ private
 		# CLOSED="no"
 	#/>
 	def assert_is_authorization_response(res)
-		assert_instance_of Hash, res
+		assert_instance_of OpenStruct, res
 		assert_is_saferpay_response res
-		assert res['msgtype']	== "AuthorizationResponse",	"Msgtype is not \"AuthorizationResponse\""
-		assert res.has_key?( 'result' ),							"No result code received in response"
-		assert res['result'] == "0", 									"Result code is not Zero (\"0\")"
-		assert res.has_key?( 'token' ),								"No token in response"
-		assert res['token']	== "(unused)",						"Token is not \"(unused)\""
+		assert res.msgtype	== "AuthorizationResponse",	"Msgtype is not \"AuthorizationResponse\""
+		assert res.respond_to?( 'result' ),							"No result code received in response"
+		assert res.result == "0", 											"Result code is not Zero (\"0\")"
+		assert res.respond_to?( 'token' ),							"No token in response"
+		assert res.token	== "(unused)",								"Token is not \"(unused)\""
 	end
 	
 	def assert_is_saferpay_response(res)
-		assert res.has_key?( 'accountid' ),						"No accountid received in response"
-		assert res['accountid'] == @accountid,				"Accountid is not the test accountid"
-		assert res.has_key?( 'msgtype' ),							"No msgtype in response"
-		assert res.has_key?( 'authcode' ),						"No authcode in response"
-		assert res.has_key?( 'id' ),									"No id in response"
-		assert res.has_key?( 'providerid' ),					"No providerid in response"
-		assert res.has_key?( 'providername' ),				"No providername in response"
-		assert res.has_key?( 'cccountry' ),						"No cccountry in response"
-		assert res.has_key?( 'contractnumber' ),			"No contractnumber in response"		
+		assert res.respond_to?( 'accountid' ),					"No accountid received in response"
+		assert res.accountid == @accountid,							"Accountid is not the test accountid"
+		assert res.respond_to?( 'msgtype' ),						"No msgtype in response"
+		assert res.respond_to?( 'authcode' ),						"No authcode in response"
+		assert res.respond_to?( 'id' ),									"No id in response"
+		assert res.respond_to?( 'providerid' ),					"No providerid in response"
+		assert res.respond_to?( 'providername' ),				"No providername in response"
+		assert res.respond_to?( 'cccountry' ),					"No cccountry in response"
+		assert res.respond_to?( 'contractnumber' ),			"No contractnumber in response"		
 	end
 	
 end

metadata

@@ -3,7 +3,7 @@ rubygems_version: 0.9.4
 specification_version: 1
 name: ruby-saferpay
 version: !ruby/object:Gem::Version 
-  version: 0.0.2
+  version: 0.0.3
 date: 2007-08-15 00:00:00 +02:00
 summary: Ruby interface to the saferpay e-commerce payment provider
 require_paths: