stp_client 0.4.0 → 0.4.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5daa60f12ce66c0757768630a8851208f4e16a6f57b769c75da532820c5daff8
-  data.tar.gz: 8af7bade911342be87701c0eefd401858663631f1c1a90287e6ec0f6f22c310b
+  metadata.gz: e6b8ee11ef13c3ad87564a457471e65eec86c5f139f734416bbb5923dbfe0a50
+  data.tar.gz: cd9d980754eb8d4b820e63f88bf03278d3ee4b670242d14b11a813a6b5716068
 SHA512:
-  metadata.gz: 2b65070385c57e1258d2b8df82704d12e9f9b616e3db201f81410431eb51c7a27ca436c4856138de2e64bcea22827811cadae64e4d5f24bb8d4d7eddc4db7e64
-  data.tar.gz: d4edc29c1ddb34da6db3d22ee91183753547e84aef5df64e2077aa5ee465ea4031b76d2f828e6e8bf49415207cb152ec39d93b5899f6df8c61b4ed9568401f74
+  metadata.gz: 85e591aaee0f159c472cd7f5cccba6463c16756215137730d2ed3656364715ef6d049681ee32ed428f7e498f5f014f7a8b9e78cb0d3cb6ca6fdb4b196a4fcab0
+  data.tar.gz: 5d18948844c674333ac6cc896d15bac7fef9e81ff24917a23c2d0a9d9f39e661bba263eb7613369a063bf7ed6c03da023eedbf9bc0cfc0f2c67b28a23903725c

data/Gemfile.lock

@@ -1,16 +1,16 @@
 PATH
   remote: .
   specs:
-    stp_client (0.3.0)
+    stp_client (0.4.3)
       crypto_yellowme (~> 0.4)
       faraday (~> 1.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    activemodel (5.2.4.1)
-      activesupport (= 5.2.4.1)
-    activesupport (5.2.4.1)
+    activemodel (5.2.4.3)
+      activesupport (= 5.2.4.3)
+    activesupport (5.2.4.3)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 0.7, < 2)
       minitest (~> 5.1)
@@ -18,7 +18,7 @@ GEM
     addressable (2.7.0)
       public_suffix (>= 2.0.2, < 5.0)
     bcrypt (3.1.13)
-    concurrent-ruby (1.1.5)
+    concurrent-ruby (1.1.6)
     crack (0.4.3)
       safe_yaml (~> 1.0.0)
     crypto_yellowme (0.4.0)
@@ -29,13 +29,13 @@ GEM
     docile (1.3.2)
     faker (2.10.0)
       i18n (>= 1.6, < 1.8)
-    faraday (1.0.0)
+    faraday (1.0.1)
       multipart-post (>= 1.2, < 3)
     hashdiff (1.0.1)
-    i18n (1.7.0)
+    i18n (1.7.1)
       concurrent-ruby (~> 1.0)
     json (2.3.0)
-    minitest (5.14.0)
+    minitest (5.14.1)
     multipart-post (2.1.1)
     public_suffix (4.0.3)
     rspec (3.9.0)
@@ -58,7 +58,7 @@ GEM
       simplecov-html (~> 0.10.0)
     simplecov-html (0.10.2)
     thread_safe (0.3.6)
-    tzinfo (1.2.6)
+    tzinfo (1.2.7)
       thread_safe (~> 0.1)
     webmock (3.4.2)
       addressable (>= 2.3.6)
@@ -76,4 +76,4 @@ DEPENDENCIES
   webmock (~> 3.4.2)
 
 BUNDLED WITH
-   2.0.2
+   2.1.2

data/lib/stp/account.rb

@@ -10,8 +10,8 @@ module STP
       end
       puts response.body if STP.verbose
       hash = JSON.parse(response.body)
-      raise STP::Errors::AccountAlreadyExists.new(hash['id'], hash['message']) if hash['id'] == 1 && hash['message'] == "Cuenta Duplicada"
-      raise STP::Errors::STPError.new(hash['id'], hash['message']) unless hash['id'] == 0
+      raise STP::Errors::AccountAlreadyExists.new(hash['id'], hash['descripcion']) if hash['id'] == 1 && hash['descripcion'] == 'Cuenta Duplicada'
+      raise STP::Errors::STPError.new(hash['id'], hash['descripcion']) unless hash['id'] == 0
     end
 
     def self.delete(account)
@@ -24,7 +24,7 @@ module STP
       end
       puts response.body if STP.verbose
       hash = JSON.parse(response.body)
-      raise STP::Errors::STPError.new(hash['id'], hash['message']) unless hash['id'] == 0
+      raise STP::Errors::STPError.new(hash['id'], hash['descripcion']) unless hash['id'] == 0
     end
   end
 end

data/lib/stp/structs/account.rb

@@ -13,7 +13,7 @@ module STP
                     :economic_activity,                  # actividadEconomica
                     :street,                             # calle
                     :exterior_number,                    # numeroExterior
-                    :interior_nunmber,                   # numeroInterior
+                    :interior_number,                    # numeroInterior
                     :neighborhood,                       # colonia
                     :municipality,                       # alcaldiaMunicipio
                     :zip_code,                           # cp
@@ -34,12 +34,15 @@ module STP
       end
 
       def as_json(_options = {})
+        name = @name.present? ? I18n.transliterate(@name) : nil
+        paternal_surname = @paternal_surname.present? ? I18n.transliterate(@paternal_surname) : nil
+        maternal_surname = @maternal_surname.present? ? I18n.transliterate(@maternal_surname) : nil
         {
           cuenta: @account,
           empresa: @company,
-          nombre: @name,
-          apellidoPaterno: @paternal_surname,
-          apellidoMaterno: @maternal_surname,
+          nombre: name,
+          apellidoPaterno: paternal_surname,
+          apellidoMaterno: maternal_surname,
           rfcCurp: @rfc,
           fechaNacimiento: @birthdate,
           genero: @sex,
@@ -47,7 +50,7 @@ module STP
           actividadEconomica: @economic_activity,
           calle: @street,
           numeroExterior: @exterior_number,
-          numeroInterior: @interior_nunmber,
+          numeroInterior: @interior_number,
           colonia: @neighborhood,
           alcaldiaMunicipio: @municipality,
           cp: @zip_code,

data/lib/stp/version.rb

@@ -1,3 +1,3 @@
 module STP
-  VERSION = '0.4.0'
+  VERSION = '0.4.5'
 end

data/spec/stp/account_create_spec.rb

@@ -21,7 +21,7 @@ RSpec.describe STP::Account do
           account.economic_activity = 'programador'
           account.street = Faker::Address.street_name
           account.exterior_number = Faker::Address.building_number
-          account.interior_nunmber = Faker::Address.building_number
+          account.interior_number = Faker::Address.building_number
           account.neighborhood = Faker::Address.community
           account.municipality = Faker::Address.community
           account.zip_code = Faker::Number.number(digits: 5)
@@ -61,7 +61,7 @@ RSpec.describe STP::Account do
     end
 
     context 'fails when' do
-      context 'repeating account and rfcCurp' do
+      context 'random error' do
         let(:account_obj) do
           account = STP::Structs::Account.new
           account.account = '01010101'
@@ -75,7 +75,7 @@ RSpec.describe STP::Account do
           account.economic_activity = 'programador'
           account.street = Faker::Address.street_name
           account.exterior_number = Faker::Address.building_number
-          account.interior_nunmber = Faker::Address.building_number
+          account.interior_number = Faker::Address.building_number
           account.neighborhood = Faker::Address.community
           account.municipality = Faker::Address.community
           account.zip_code = Faker::Number.number(digits: 5)
@@ -94,6 +94,40 @@ RSpec.describe STP::Account do
 
         it { expect {subject}.to raise_error(STP::Errors::STPError) }
       end
+
+      context 'repeating account and rfcCurp' do
+        let(:account_obj) do
+          account = STP::Structs::Account.new
+          account.account = '01010101'
+          account.name = Faker::Name.first_name
+          account.paternal_surname = Faker::Name.last_name
+          account.maternal_surname = Faker::Name.last_name
+          account.rfc = 'RFC'
+          account.birthdate = Faker::Date.birthday(min_age: 6, max_age: 65)
+          account.sex = 'male'
+          account.state = 'Mérida'
+          account.economic_activity = 'programador'
+          account.street = Faker::Address.street_name
+          account.exterior_number = Faker::Address.building_number
+          account.interior_number = Faker::Address.building_number
+          account.neighborhood = Faker::Address.community
+          account.municipality = Faker::Address.community
+          account.zip_code = Faker::Number.number(digits: 5)
+          account.country = 'México'
+          account.email = Faker::Internet.email
+          account.identification_id = 'id'
+          account.phone = Faker::PhoneNumber.phone_number
+          account
+        end
+        
+        before do
+          stub_request(:put, "#{STP.api_uri}/cuentaModule/fisica").to_return(
+            body: JSON.dump(STPTestHelpers::ACCOUNT_CREATE_MOCK_ERROR(1, 'Cuenta Duplicada')), status: 200
+          )
+        end
+
+        it { expect {subject}.to raise_error(STP::Errors::AccountAlreadyExists) }
+      end
     end
   end
 end

data/spec/stp/account_delete_spec.rb

@@ -21,7 +21,7 @@ RSpec.describe STP::Account do
           account.economic_activity = 'programador'
           account.street = Faker::Address.street_name
           account.exterior_number = Faker::Address.building_number
-          account.interior_nunmber = Faker::Address.building_number
+          account.interior_number = Faker::Address.building_number
           account.neighborhood = Faker::Address.community
           account.municipality = Faker::Address.community
           account.zip_code = Faker::Number.number(digits: 5)
@@ -41,7 +41,7 @@ RSpec.describe STP::Account do
           allow(STP::Account).to receive(:create).
             and_return(STPTestHelpers::ACCOUNT_DELETE_MOCK_SUCCESS())
         end
-        
+
         before do
           stub_request(:delete, "#{STP.api_uri}/cuentaModule/fisica").to_return(
             body: JSON.dump(STPTestHelpers::ACCOUNT_DELETE_MOCK_SUCCESS()), status: 200
@@ -59,7 +59,7 @@ RSpec.describe STP::Account do
           account.rfc = 'RFCURP'
           account
         end
-        
+
         before do
           stub_request(:delete, "#{STP.api_uri}/cuentaModule/fisica").to_return(
             body: JSON.dump(STPTestHelpers::ACCOUNT_DELETE_MOCK_SUCCESS()), status: 200
@@ -85,7 +85,7 @@ RSpec.describe STP::Account do
           account.economic_activity = 'programador'
           account.street = Faker::Address.street_name
           account.exterior_number = Faker::Address.building_number
-          account.interior_nunmber = Faker::Address.building_number
+          account.interior_number = Faker::Address.building_number
           account.neighborhood = Faker::Address.community
           account.municipality = Faker::Address.community
           account.zip_code = Faker::Number.number(digits: 5)
@@ -95,7 +95,7 @@ RSpec.describe STP::Account do
           account.phone = Faker::PhoneNumber.phone_number
           account
         end
-        
+
         before do
           stub_request(:delete, "#{STP.api_uri}/cuentaModule/fisica").to_return(
             body: JSON.dump(STPTestHelpers::ACCOUNT_DELETE_MOCK_ERROR(-2)), status: 200

data/spec/stp/payment_order_spec.rb

@@ -96,7 +96,7 @@ RSpec.describe STP::PaymentOrder do
         end
 
         it { expect(subject['id']).to eq -2 }
-        it { expect(subject['descripcionError']).not_to be_nil }
+        it { expect(subject['descripcion']).not_to be_nil }
       end
     end
   end

data/spec/support/stp_test_helpers.rb

@@ -1,7 +1,7 @@
 module STPTestHelpers
   def self.PAYMENT_ORDER_MOCK_ERROR(error_code)
     {
-      'descripcionError' => 'regular error msg',
+      'descripcion' => 'regular error msg',
       'id' => error_code,
     }
   end
@@ -16,9 +16,9 @@ module STPTestHelpers
     }
   end
 
-  def self.ACCOUNT_CREATE_MOCK_ERROR(error_code)
+  def self.ACCOUNT_CREATE_MOCK_ERROR(error_code, message='regular error msg')
     {
-      'descripcionError' => 'regular error msg',
+      'descripcion' => message,
       'id' => error_code,
     }
   end
@@ -32,7 +32,7 @@ module STPTestHelpers
 
   def self.ACCOUNT_DELETE_MOCK_ERROR(error_code)
     {
-      'descripcionError' => 'regular error msg',
+      'descripcion' => 'regular error msg',
       'id' => error_code,
     }
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stp_client
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.4.5
 platform: ruby
 authors:
 - Yellowme
@@ -142,8 +142,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: STP Client is a ruby integration to STP services