geti 1.2.0 → 1.3.0

data/VERSION

@@ -1 +1 @@
-1.2.0
+1.3.0

data/lib/geti/app_client.rb

@@ -174,6 +174,7 @@ class Geti::AppClient < Geti::Client
   end
 
   def data(xml, opts)
+    opts = filter_invalid_characters(opts)
     filename = "%s_%s.xml" % [opts[:id].to_s, Time.now.strftime("%d_%b_%Y_%H_%M_%S")]
     xml.Envelope do
       xml.Body :FileName => filename, :FileDate => Time.now.iso8601 do
@@ -255,6 +256,20 @@ class Geti::AppClient < Geti::Client
     end
   end
 
+  def filter_invalid_characters(opts)
+    opts = opts.dup
+    opts[:address].gsub!(/[^a-zA-Z0-9 #-:;']/, '')
+    opts[:physical_address].gsub!(/[^a-zA-Z0-9 #-:;']/, '')
+    opts[:principal_address].gsub!(/[^a-zA-Z0-9 #-:;']/, '')
+
+    opts[:city].gsub!(/[^a-zA-Z0-9 ]/, '')
+    opts[:physical_city].gsub!(/[^a-zA-Z0-9 ]/, '')
+    opts[:principal_city].gsub!(/[^a-zA-Z0-9 ]/, '')
+
+    opts[:principal_ssn].gsub!(/[^0-9]/, '')
+    opts
+  end
+
   def service_address
     "https://#{domain}/webservices/AppGateway.asmx?WSDL"
   end

data/spec/geti_app_client_spec.rb

@@ -147,6 +147,46 @@ describe Geti::AppClient do
       client.board_merchant_ach(request_payload)
     end
 
+    describe 'character filtering' do
+      let(:client) { Geti::AppClient.new(test_credentials, {}) }
+
+      it 'filters city' do
+        data = request_payload.merge(:city => 'St. Johns')
+        client.filter_invalid_characters(data)[:city].should == "St Johns"
+      end
+
+      it 'filters address' do
+        data = request_payload.merge(:address => '12 Main St, Suite B!')
+        client.filter_invalid_characters(data)[:address].should == "12 Main St, Suite B"
+      end
+
+      it 'filters physical_city' do
+        data = request_payload.merge(:physical_city => 'St. Johns')
+        client.filter_invalid_characters(data)[:physical_city].should == "St Johns"
+      end
+
+      it 'filters physical_address' do
+        data = request_payload.merge(:physical_address => '12 Main St, Suite B!')
+        client.filter_invalid_characters(data)[:physical_address].should == "12 Main St, Suite B"
+      end
+
+      it 'filters principal_city' do
+        data = request_payload.merge(:principal_city => 'St. Johns')
+        client.filter_invalid_characters(data)[:principal_city].should == "St Johns"
+      end
+
+      it 'filters principal_address' do
+        data = request_payload.merge(:principal_address => '12 Main St, Suite B!')
+        client.filter_invalid_characters(data)[:principal_address].should == "12 Main St, Suite B"
+      end
+
+      it 'filters principal_ssn' do
+        data = request_payload.merge(:principal_ssn => '111-22-3333')
+        client.filter_invalid_characters(data)[:principal_ssn].should == "111223333"
+      end
+
+    end
+
     describe 'response on success' do
       let(:client) {
         Geti::AppClient.new(test_credentials, {}).tap{|c|

data/spec/remote/geti_app_client_spec.rb

@@ -1,6 +1,6 @@
 require 'helper'
 
-describe Geti::AppClient do
+describe Geti::AppClient, :remote => true do
   before :all do
     @timestamp = Time.now.to_i
   end
@@ -18,7 +18,7 @@ describe Geti::AppClient do
       :days_in_business => 404,
 
       :contact_name     => 'George Jetson',
-      :physical_address => "123 Main St",
+      :physical_address => "123 Main St, Suite B",
       :physical_city    => "Vancouver",
       :physical_state   => "WA",
       :physical_zip     => "10120",

data/spec/remote/geti_auth_client_spec.rb

@@ -8,7 +8,7 @@ def routing_number(type)
   }[type]
 end
 
-describe Geti::AuthClient do
+describe Geti::AuthClient, :remote => true do
   let(:credentials) { test_credentials('auth') }
 
   describe '#get_terminal_settings' do

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: geti
 version: !ruby/object:Gem::Version 
-  hash: 31
+  hash: 27
   prerelease: 
   segments: 
   - 1
-  - 2
+  - 3
   - 0
-  version: 1.2.0
+  version: 1.3.0
 platform: ruby
 authors: 
 - Jamie Macey
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-11-13 00:00:00 Z
+date: 2012-11-23 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: savon