cieloz 0.0.8 → 0.0.9

data/config/locales/en.yml

@@ -5,7 +5,28 @@ en:
         cieloz/requisicao_transacao:
           attributes:
             autorizar:
+              inclusion: should be in [0,1,2,3,4]
               authentication_not_supported: Authentication not supported
               direct_auth_available_for_credit_only: Direct auth available for credit only
-            forma_pagamento:
+            capturar:
+              inclusion: should be true or false
+        cieloz/requisicao_transacao/dados_portador:
+          attributes:
+            numero:
+              invalid: should have 16 digits
+            validade:
+              invalid: should be in YYYYDD format
+            codigo_seguranca:
+              invalid: should have 3 or 4 digits
+        cieloz/requisicao_transacao/dados_pedido:
+          attributes:
+            idioma:
+              inclusion: should be PT (Portuguese), EN (English) or ES (Spanish)
+            valor:
               minimum_installment_not_satisfied: Installment should be greater than or equal to R$ 5,00
+        cieloz/requisicao_transacao/forma_pagamento:
+          attributes:
+            bandeira:
+              blank: choose credit card flag
+              inclusion: choose credit card flag
+

data/config/locales/pt-BR.yml

@@ -5,7 +5,28 @@ pt-BR:
         cieloz/requisicao_transacao:
           attributes:
             autorizar:
+              inclusion: deveria estar entre [0,1,2,3,4]
               authentication_not_supported: Bandeira não possui programa de autenticação
               direct_auth_available_for_credit_only: Autorização Direta disponível apenas em operações de crédito
-            forma_pagamento:
+            capturar:
+              inclusion: deveria ser true ou false
+        cieloz/requisicao_transacao/dados_portador:
+          attributes:
+            numero:
+              invalid: deve ter 16 dígitos
+            validade:
+              invalid: deve estar no formato YYYYDD
+            codigo_seguranca:
+              invalid: deve ter 3 ou 4 dígitos
+        cieloz/requisicao_transacao/dados_pedido:
+          attributes:
+            idioma:
+              inclusion: deveria ser PT (Português), EN (Inglês) ou ES (Espanhol)
+            valor:
               minimum_installment_not_satisfied: O valor mínimo da parcela deve ser R$ 5,00
+        cieloz/requisicao_transacao/forma_pagamento:
+          attributes:
+            bandeira:
+              blank: escolha a bandeira
+              inclusion: escolha a bandeira
+

data/lib/cieloz/bandeiras.rb

@@ -4,17 +4,17 @@ module Cieloz
     AMEX, DINERS, DISCOVER, ELO, MASTERCARD, VISA = ALL
 
     def self.operacao produto
-      case produto.to_sym
-      when :mastercard_securecode
+      produto = produto.to_s
+      case produto
+      when 'mastercard_securecode'
         [MASTERCARD,      :autorizar_somente_autenticada]
-      when :verified_by_visa
+      when 'verified_by_visa'
         [VISA,            :autorizar_somente_autenticada]
       else
-        produto = produto.to_s
         if ALL.include? produto
           [produto,  :autorizacao_direta]
         else
-          raise "product_not_supported"
+          [nil, nil]
         end
       end
     end

data/lib/cieloz/requisicao_transacao/dados_pedido.rb

@@ -8,14 +8,14 @@ class Cieloz::RequisicaoTransacao
 
     validates :numero, :valor, :moeda, :data_hora, presence: true
 
-    validates :numero, length: { in: 1..20 }
+    validates :numero, length: { maximum: 20 }
 
-    validates :valor, length: { in: 1..12 }
-    validates :valor, numericality: { only_integer: true }
+    validates :valor, length: { maximum: 12 }
+    validates :valor, numericality: { only_integer: true }, unless: "@valor.blank?"
 
-    validates :descricao, length: { in: 0..1024 }
+    validates :descricao, length: { maximum: 1024 }
     validates :idioma, inclusion: { in: IDIOMAS }
-    validates :soft_descriptor, length: { in: 0..13 }
+    validates :soft_descriptor, length: { maximum: 13 }
 
     def attributes
       {

data/lib/cieloz/requisicao_transacao/dados_portador.rb

@@ -10,19 +10,21 @@ class Cieloz::RequisicaoTransacao
     attr_accessor :numero, :nome_portador, :validade
     attr_reader :indicador, :codigo_seguranca
 
-    validates :nome_portador, length: { in: 0..50 }
+    validates :nome_portador, length: { maximum: 50 }
 
-    validates :numero, :validade, :indicador, presence: true
-
-    validates :numero, length: { is: 16 }
-    validates :numero, numericality: { only_integer: true }
+    set_callback :validate, :before do |portador|
+      [:numero, :validade, :codigo_seguranca].each {|attr|
+        val = portador.send attr
+        portador.instance_variable_set "@#{attr}", val.to_s
+      }
+      portador.numero.gsub! ' ', ''
+    end
 
-    validates :validade, length: { is: 6 }
-    validates :validade, format: { with: /2\d{3}(0[1-9]|1[012])/ }
-    validates :validade, numericality: { only_integer: true }
+    validates :numero, format: { with: /^\d{16}$/ }
+    validates :validade, format: { with: /^2\d{3}(0[1-9]|1[012])$/ }
+    validates :codigo_seguranca, format: { with: /^(\d{3}|\d{4})$/ }
 
-    validates :codigo_seguranca, length: { in: 3..4 }
-    validates :codigo_seguranca, numericality: { only_integer: true }
+    validates :indicador, presence: true
 
     def initialize attrs={}
       super

data/lib/cieloz/requisicao_transacao.rb

@@ -23,18 +23,18 @@ class Cieloz::RequisicaoTransacao < Cieloz::Requisicao
   validate :parcela_minima?,
     if: "not @dados_pedido.nil? and not @forma_pagamento.nil?"
 
-  validates :autorizar, presence: true, inclusion: {
+  validates :autorizar, inclusion: {
     in: [
       SOMENTE_AUTENTICAR, AUTORIZAR_SE_AUTENTICADA,
       AUTORIZAR_NAO_AUTENTICADA, AUTORIZACAO_DIRETA, RECORRENTE
     ]
   }
   # validates string values because false.blank? is true, failing presence validation
-  validates :capturar,  presence: true, inclusion: { in: ["true", "false"] }
+  validates :capturar, inclusion: { in: ["true", "false"] }
 
   with_options if: "@autorizar != AUTORIZACAO_DIRETA" do |txn|
     txn.validates :url_retorno, presence: true
-    txn.validates :url_retorno, length: { in: 1..1024 }
+    txn.validates :url_retorno, length: { maximum: 1024 }
   end
 
   validates :campo_livre, length: { maximum: 128 }
@@ -52,10 +52,9 @@ class Cieloz::RequisicaoTransacao < Cieloz::Requisicao
   end
 
   def parcela_minima?
-    if @dados_pedido.valid? and @forma_pagamento.valid?
-      if @dados_pedido.valor / @forma_pagamento.parcelas < 500
-        errors.add :forma_pagamento, :minimum_installment_not_satisfied
-      end
+    valor, parcelas = @dados_pedido.valor.to_i, @forma_pagamento.parcelas.to_i
+    if parcelas > 0 and valor / parcelas < 500
+      @dados_pedido.errors.add :valor, :minimum_installment_not_satisfied
     end
   end
 

data/lib/cieloz/version.rb

@@ -1,3 +1,3 @@
 module Cieloz
-  VERSION = "0.0.8"
+  VERSION = "0.0.9"
 end

data/test/unit/bandeiras_operacoes_test.rb

@@ -19,7 +19,6 @@ describe "Bandeiras e Operacoes" do
   end
 
   it "disallows unsupported products" do
-    error = lambda { _.operacao("anything else") }.must_raise RuntimeError
-    error.message.must_match /product_not_supported/
+    assert_equal [nil, nil], _.operacao("anything else")
   end
 end

data/test/unit/validations_test.rb

@@ -16,18 +16,21 @@ describe Cieloz::RequisicaoTransacao::DadosPortador do
 
   it { must ensure_length_of(:nome_portador).is_at_most(50) }
 
-  it { must validate_presence_of :numero }
-  it { must ensure_length_of(:numero).is_equal_to 16 }
-  it { must validate_numericality_of(:numero).only_integer }
+  it { must allow_value(1234567890123456).for(:numero) }
+  it { wont allow_value(123456789012345).for(:numero) }
+  it { wont allow_value(12345678901234567).for(:numero) }
+  it { wont allow_value("ABC4567890123456").for(:numero) }
 
-  it { must validate_presence_of :validade }
-  it { must ensure_length_of(:validade).is_equal_to 6 }
-  it { must validate_numericality_of(:validade).only_integer }
+  it {
+    (100..9999).step(123).each {|val|
+      must allow_value(val).for(:codigo_seguranca)
+    }
+  }
 
-  it { must ensure_length_of(:codigo_seguranca)
-                              .is_at_least(3)
-                              .is_at_most(4) }
-  it { must validate_numericality_of(:codigo_seguranca).only_integer }
+  it { wont allow_value(99).for(:codigo_seguranca) }
+  it { wont allow_value(10000).for(:codigo_seguranca) }
+  it { wont allow_value("ab1").for(:codigo_seguranca) }
+  it { wont allow_value("abc1").for(:codigo_seguranca) }
 
   def mm_values range
     yyyy = 2013
@@ -90,15 +93,21 @@ end
 
 describe Cieloz::RequisicaoTransacao::DadosPedido do
   it { must validate_presence_of :numero }
-  it { must ensure_length_of(:numero)
-                            .is_at_least(1)
-                            .is_at_most(20) }
+  it { must ensure_length_of(:numero).is_at_most(20) }
 
   it { must validate_presence_of :valor }
+  it { must ensure_length_of(:valor).is_at_most(12) }
   it { must validate_numericality_of(:valor).only_integer }
-  it { must ensure_length_of(:valor)
-                            .is_at_least(1)
-                            .is_at_most(12) }
+
+  it "dont validate valor if it's blank" do
+    subject.valor = ""
+    refute subject.valid?
+    assert_equal [I18n.t('errors.messages.blank')], subject.errors[:valor]
+
+    subject.valor = "abc"
+    refute subject.valid?
+    assert_equal [I18n.t('errors.messages.not_a_number')], subject.errors[:valor]
+  end
 
   it { must validate_presence_of :moeda }
   it { must validate_presence_of :data_hora }
@@ -238,14 +247,13 @@ describe Cieloz::RequisicaoTransacao do
     wont validate_presence_of :url_retorno
   end
 
-  it { must ensure_length_of(:url_retorno).is_at_least(1).is_at_most(1024) }
+  it { must ensure_length_of(:url_retorno).is_at_most(1024) }
 
   it "doesnt validate url_retorno length for autorizacao direta" do
     subject.autorizacao_direta
     wont ensure_length_of(:url_retorno).is_at_least(1).is_at_most(1024)
   end
 
-  it { must validate_presence_of :autorizar }
   it {
     must ensure_inclusion_of(:autorizar).in_array [
       _::SOMENTE_AUTENTICAR, _::AUTORIZAR_SE_AUTENTICADA,
@@ -309,7 +317,6 @@ describe Cieloz::RequisicaoTransacao do
     }
   end
 
-  it { must validate_presence_of :capturar }
   it { must ensure_inclusion_of(:capturar).in_array(["true", "false"]) }
 
   it { must ensure_length_of(:campo_livre).is_at_most(128) }
@@ -349,6 +356,6 @@ describe Cieloz::RequisicaoTransacao do
 
     refute subject.valid?
     msg = "Installment should be greater than or equal to R$ 5,00"
-    assert_equal msg, subject.errors[:forma_pagamento].first
+    assert_equal msg, subject.dados_pedido.errors[:valor].first
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cieloz
 version: !ruby/object:Gem::Version
-  version: 0.0.8
+  version: 0.0.9
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-03-18 00:00:00.000000000 Z
+date: 2013-03-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -246,12 +246,18 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: 3193634196579590297
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: 3193634196579590297
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23