cieloz 0.0.16 → 0.0.17

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: bc58d2aeb4dcfa87510aebee7ec575afd4fd9a71
-  data.tar.gz: 7694e9c7a5290050fa512af92f197879354cebc2
+  metadata.gz: bafeb0b8ee2b0cc29837ae97e0a5b1c4a423fe22
+  data.tar.gz: 228cd850f742e759568869938329499f22e5213f
 SHA512:
-  metadata.gz: c60d28b5fcab2951fc0d3008154f460ba16060ddc42a83e82bb15a4979b963ca0d4b716efe1d3f953f3dcaa05e73b13d324a44fd819ee1420ded6a7bccbddc45
-  data.tar.gz: b3b664d72fcb6792f0607b26ac2e56ea1423d487a2b60a4ec022c9f06697198d7781d50bae9711f7cd5f80ff014354c3b46aa5ae7b5f370853941db7517344d4
+  metadata.gz: 369d7b7699e9c31cc23c664f7eb985fc35ef9cfcf6465204e2e50357db8c5fc5d04410560c718c1e95b1b7e57b717ce4e8ca8508d067e137b12d96cf4b1861b2
+  data.tar.gz: 5c9f92497d3396a7a81ea34302fb022d477dedd32ab92d47d380a32b70355b04907111130c0a17b40f0c6d224c56d5196ce0c992e6b6d264640afb1921307b21

data/config/locales/en.yml

@@ -13,6 +13,8 @@ en:
         cieloz/requisicao_transacao/dados_portador:
           attributes:
             numero:
+              invalid_diners: should have 14 digits
+              invalid_amex: should have 15 digits
               invalid: should have 16 digits
             codigo_seguranca:
               invalid: should have 3 or 4 digits

data/config/locales/pt-BR.yml

@@ -13,6 +13,8 @@ pt-BR:
         cieloz/requisicao_transacao/dados_portador:
           attributes:
             numero:
+              invalid_diners: deve ter 14 dígitos
+              invalid_amex: deve ter 15 dígitos
               invalid: deve ter 16 dígitos
             codigo_seguranca:
               invalid: deve ter 3 ou 4 dígitos

data/lib/cieloz/requisicao_transacao/dados_portador.rb

@@ -20,12 +20,9 @@ class Cieloz::RequisicaoTransacao
       portador.numero.gsub! ' ', ''
     end
 
-    validates :numero, format: { with: /\A\d{16}\z/ }
     validates :codigo_seguranca, format: { with: /\A(\d{3}|\d{4})\z/ }
-
-    validate :valida_validade
-
     validates :indicador, presence: true
+    validate :valida_validade
 
     def self.map(source, opts={})
       num, val, cod, nome = attrs_from source, opts,

data/lib/cieloz/requisicao_transacao.rb

@@ -12,7 +12,11 @@ class Cieloz::RequisicaoTransacao < Cieloz::Requisicao
 
   validate :nested_validations
 
-  validates :dados_portador, presence: true, if: "Cieloz::Configuracao.store_mode?"
+  with_options if: "Cieloz::Configuracao.store_mode?" do |c|
+    c.validates :dados_portador, presence: true
+    c.validate :valida_digitos_numero_cartao
+  end
+
   validates :dados_pedido, :forma_pagamento, presence: true
 
   with_options unless: "@forma_pagamento.nil?" do |txn|
@@ -146,4 +150,19 @@ class Cieloz::RequisicaoTransacao < Cieloz::Requisicao
       bin:              (@dados_portador.numero.to_s[0..5] unless @dados_portador.nil?)
     }
   end
+
+  private
+  def valida_digitos_numero_cartao
+    if dados_portador and forma_pagamento and bandeira = forma_pagamento.bandeira
+      numero = dados_portador.numero.to_s
+      case bandeira.to_s
+      when Cieloz::Bandeiras::DINERS
+        dados_portador.errors.add :numero, :invalid_diners  unless numero =~ /\A\d{14}\z/
+      when Cieloz::Bandeiras::AMEX
+        dados_portador.errors.add :numero, :invalid_amex    unless numero =~ /\A\d{15}\z/
+      else
+        dados_portador.errors.add :numero, :invalid         unless numero =~ /\A\d{16}\z/
+      end
+    end
+  end
 end

data/lib/cieloz/version.rb

@@ -1,3 +1,3 @@
 module Cieloz
-  VERSION = "0.0.16"
+  VERSION = "0.0.17"
 end

data/test/unit/validations_test.rb

@@ -17,10 +17,6 @@ describe Cieloz::RequisicaoTransacao::DadosPortador do
   let(:invalid_number) {
     I18n.t 'activemodel.errors.models.cieloz/requisicao_transacao/dados_portador.attributes.numero.invalid'
   }
-  it { must_allow_value :numero, 1234567890123456   }
-  it { wont_allow_value :numero, 123456789012345, message: invalid_number }
-  it { wont_allow_value :numero, 12345678901234567, message: invalid_number }
-  it { wont_allow_value :numero, "ABC4567890123456", message: invalid_number }
 
   it {
     (100..9999).step(123).each {|val|
@@ -394,4 +390,42 @@ describe Cieloz::RequisicaoTransacao do
     msg = "Installment should be greater than or equal to R$ 5,00"
     assert_equal msg, subject.dados_pedido.errors[:valor].first
   end
+
+  describe "validates credit card number format" do
+    after do
+      Cieloz::Configuracao.reset!
+    end
+
+    def self.validate_number_of_credit_card_digits_for(flag, digits)
+      describe flag do
+        let(:pg) { subject.class::FormaPagamento.new.credito flag }
+
+        before do
+          Cieloz::Configuracao.store_mode!
+          subject.forma_pagamento = pg
+        end
+
+        it "accepts credit card numbers with #{digits} digits" do
+          subject.dados_portador = subject.class::DadosPortador.new numero: '1' * digits
+          subject.valid?
+          subject.dados_portador.errors[:numero].must_be_empty
+        end
+
+        it "rejects credit card numbers with other formats" do
+          ['1' * (digits - 1), '1' * (digits + 1), 'ABC4567890123456'].each do |number|
+            subject.dados_portador = subject.class::DadosPortador.new numero: number
+            subject.valid?
+            subject.dados_portador.errors[:numero].wont_be_empty
+          end
+        end
+      end
+    end
+
+    validate_number_of_credit_card_digits_for Cieloz::Bandeiras::DINERS,      14
+    validate_number_of_credit_card_digits_for Cieloz::Bandeiras::AMEX,        15
+    validate_number_of_credit_card_digits_for Cieloz::Bandeiras::VISA,        16
+    validate_number_of_credit_card_digits_for Cieloz::Bandeiras::MASTERCARD,  16
+    validate_number_of_credit_card_digits_for Cieloz::Bandeiras::ELO,         16
+    validate_number_of_credit_card_digits_for Cieloz::Bandeiras::DISCOVER,    16
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cieloz
 version: !ruby/object:Gem::Version
-  version: 0.0.16
+  version: 0.0.17
 platform: ruby
 authors:
 - Fábio Luiz Nery de Miranda