Builds for
1 pipeline
failed
in
0 seconds
Fixed bug in oxxo specs
Showing
2 changed files
with
2 additions
and
3 deletions
| ... | @@ -5,7 +5,7 @@ module Var | ... | @@ -5,7 +5,7 @@ module Var |
| 5 | 5 | ||
| 6 | class << self | 6 | class << self |
| 7 | def analyze(event) | 7 | def analyze(event) |
| 8 | - process_payment event[:data][:object] if | 8 | + return process_payment event[:data][:object] if |
| 9 | event[:data][:object][:status] == 'paid' | 9 | event[:data][:object][:status] == 'paid' |
| 10 | process_charge event[:data][:object] | 10 | process_charge event[:data][:object] |
| 11 | end | 11 | end |
| ... | @@ -15,7 +15,6 @@ module Var | ... | @@ -15,7 +15,6 @@ module Var |
| 15 | time = Time.strptime payment_info[:paid_at].to_s, '%s' | 15 | time = Time.strptime payment_info[:paid_at].to_s, '%s' |
| 16 | amount = payment_info[:amount].to_f / 100.0 | 16 | amount = payment_info[:amount].to_f / 100.0 |
| 17 | fee = payment_info[:fee].to_f / 100.0 | 17 | fee = payment_info[:fee].to_f / 100.0 |
| 18 | - | ||
| 19 | object.update(var_fee: fee, var_paid_amount: amount, | 18 | object.update(var_fee: fee, var_paid_amount: amount, |
| 20 | var_payment_method: self::PAYMENT_METHOD_STRING, | 19 | var_payment_method: self::PAYMENT_METHOD_STRING, |
| 21 | var_payment_at: time, | 20 | var_payment_at: time, | ... | ... |
| ... | @@ -12,7 +12,6 @@ describe Var::ConektaOxxoHandler do | ... | @@ -12,7 +12,6 @@ describe Var::ConektaOxxoHandler do |
| 12 | ConektaChargeCreateObject::OXXO_CREATE_CHARGE_OBJECT, | 12 | ConektaChargeCreateObject::OXXO_CREATE_CHARGE_OBJECT, |
| 13 | object_class: OpenStruct) | 13 | object_class: OpenStruct) |
| 14 | allow(product).to receive(:conekta_charge).and_return(conekta_object_mock) | 14 | allow(product).to receive(:conekta_charge).and_return(conekta_object_mock) |
| 15 | - | ||
| 16 | Var.create_charge(:conekta, product, {"conekta_type": "oxxo"}) | 15 | Var.create_charge(:conekta, product, {"conekta_type": "oxxo"}) |
| 17 | 16 | ||
| 18 | Var::ConektaOxxoHandler.analyze(WebHookPostRequest::OXXO_CHARGE_CREATED) | 17 | Var::ConektaOxxoHandler.analyze(WebHookPostRequest::OXXO_CHARGE_CREATED) |
| ... | @@ -29,6 +28,7 @@ describe Var::ConektaOxxoHandler do | ... | @@ -29,6 +28,7 @@ describe Var::ConektaOxxoHandler do |
| 29 | 28 | ||
| 30 | Var.create_charge(:conekta, product, {"conekta_type": "oxxo"}) | 29 | Var.create_charge(:conekta, product, {"conekta_type": "oxxo"}) |
| 31 | Var::ConektaOxxoHandler.analyze(WebHookPostRequest::OXXO_CHARGE_PAID) | 30 | Var::ConektaOxxoHandler.analyze(WebHookPostRequest::OXXO_CHARGE_PAID) |
| 31 | + product.reload | ||
| 32 | expect(product.var_id).to eq conekta_object_mock.id | 32 | expect(product.var_id).to eq conekta_object_mock.id |
| 33 | expect(product.var_status).to eq 'paid' | 33 | expect(product.var_status).to eq 'paid' |
| 34 | end | 34 | end | ... | ... |
-
Please register or login to post a comment