Showing
2 changed files
with
6 additions
and
6 deletions
... | @@ -30,10 +30,10 @@ module Var | ... | @@ -30,10 +30,10 @@ module Var |
30 | 30 | ||
31 | def conekta_webhook(event) | 31 | def conekta_webhook(event) |
32 | raise "Hash expected, received #{event.class}" unless event.class == Hash | 32 | raise "Hash expected, received #{event.class}" unless event.class == Hash |
33 | - raise 'Malformed hash received' unless (event[:data] && | 33 | + raise 'Malformed hash received' unless event[:data] && |
34 | - event[:data][:object]) || | 34 | + event[:data][:object] |
35 | - event[:object] | 35 | + payment_type = event[:data][:object][:object] |
36 | - if event.keys[0] == :object | 36 | + if payment_type == "transfer" |
37 | ConektaPayoutHandler.analyze(event) | 37 | ConektaPayoutHandler.analyze(event) |
38 | else | 38 | else |
39 | payment_type = event[:data][:object][:payment_method][:object] | 39 | payment_type = event[:data][:object][:payment_method][:object] | ... | ... |
... | @@ -120,7 +120,7 @@ describe Var do | ... | @@ -120,7 +120,7 @@ describe Var do |
120 | 120 | ||
121 | it 'handles payout scheduled' do | 121 | it 'handles payout scheduled' do |
122 | null = nil | 122 | null = nil |
123 | - params = {"object":{"livemode":true,"currency":"MXN","method":{"id":"pytmtd_MymQ7favB2hQJC","created_at":1467832748,"payee_id":null,"_type":"BankTransferPayoutMethod","object":"bank_transfer_payout_method","account_number":"072580002374915018","account_holder":"UKKO SOLUTIONS SA DE CV","bank":"Banco Mercantil del Norte, S.A."},"object":"transfer","amount":303534,"id":"577d59ac243c7fee8100146d","created_at":1467832748,"status":"scheduled","statement_reference":"4632539","statement_description":"Conekta 4632539"},"previous_attributes":{"status":"pending","broker_transaction_reference":null}} | 123 | + params = {"data":{"object":{"livemode":true, "currency":"MXN", "method":{"id":"pytmtd_hqsxocPm8CZoNu", "created_at":1467992925, "payee_id":nil, "_type":"BankTransferPayoutMethod", "object":"bank_transfer_payout_method", "account_number":"072580002374915018", "account_holder":"UKKO SOLUTIONS SA DE CV", "bank":"Banco Mercantil del Norte, S.A."}, "object":"transfer", "amount":453320, "id":"577fcb5d7b823b818e001791", "created_at":1467992925, "status":"scheduled", "statement_reference":"3121951", "statement_description":"Conekta 3121951"}, "previous_attributes":{"status":"pending", "broker_transaction_reference":nil}}, "livemode":true, "webhook_status":"failing", "webhook_logs":[{"id":"webhl_QPq8SQ9uaMGABaE", "url":"http://kipping.mx/conekta_webhook", "failed_attempts":71, "last_http_response_status":500, "object":"webhook_log", "last_attempted_at":1468245529}], "id":"577fd618243c7fd79d0009c5", "object":"event", "type":"payout.scheduled", "created_at":1467995672, "webhook":{"data":{"object":{"livemode":true, "currency":"MXN", "method":{"id":"pytmtd_hqsxocPm8CZoNu", "created_at":1467992925, "payee_id":nil, "_type":"BankTransferPayoutMethod", "object":"bank_transfer_payout_method", "account_number":"072580002374915018", "account_holder":"UKKO SOLUTIONS SA DE CV", "bank":"Banco Mercantil del Norte, S.A."}, "object":"transfer", "amount":453320, "id":"577fcb5d7b823b818e001791", "created_at":1467992925, "status":"scheduled", "statement_reference":"3121951", "statement_description":"Conekta 3121951"}, "previous_attributes":{"status":"pending", "broker_transaction_reference":nil}}, "livemode":true, "webhook_status":"failing", "webhook_logs":[{"id":"webhl_QPq8SQ9uaMGABaE", "url":"http://kipping.mx/conekta_webhook", "failed_attempts":71, "last_http_response_status":500, "object":"webhook_log", "last_attempted_at":1468245529}], "id":"577fd618243c7fd79d0009c5", "object":"event", "type":"payout.scheduled", "created_at":1467995672}} |
124 | payoutHandler = class_double("Var::ConektaPayoutHandler"). | 124 | payoutHandler = class_double("Var::ConektaPayoutHandler"). |
125 | as_stubbed_const(transfer_nested_constants: true) | 125 | as_stubbed_const(transfer_nested_constants: true) |
126 | expect(payoutHandler).to receive(:analyze).with(params) | 126 | expect(payoutHandler).to receive(:analyze).with(params) |
... | @@ -129,7 +129,7 @@ describe Var do | ... | @@ -129,7 +129,7 @@ describe Var do |
129 | 129 | ||
130 | it 'handles payout paid' do | 130 | it 'handles payout paid' do |
131 | null = nil | 131 | null = nil |
132 | - params = {"object":{"livemode":true,"currency":"MXN","method":{"id":"pytmtd_MymQ7favB2hQJC","created_at":1467832748,"payee_id":null,"_type":"BankTransferPayoutMethod","object":"bank_transfer_payout_method","account_number":"072580002374915018","account_holder":"UKKO SOLUTIONS SA DE CV","bank":"Banco Mercantil del Norte, S.A."},"object":"transfer","amount":303534,"id":"577d59ac243c7fee8100146d","created_at":1467832748,"status":"scheduled","statement_reference":"4632539","statement_description":"Conekta 4632539"},"previous_attributes":{}} | 132 | + params = {"data":{"object":{"livemode":true, "currency":"MXN", "method":{"id":"pytmtd_hqsxocPm8CZoNu", "created_at":1467992925, "payee_id":nil, "_type":"BankTransferPayoutMethod", "object":"bank_transfer_payout_method", "account_number":"072580002374915018", "account_holder":"UKKO SOLUTIONS SA DE CV", "bank":"Banco Mercantil del Norte, S.A."}, "object":"transfer", "amount":453320, "id":"577fcb5d7b823b818e001791", "created_at":1467992925, "status":"scheduled", "statement_reference":"3121951", "statement_description":"Conekta 3121951"}, "previous_attributes":{}}, "livemode":true, "webhook_status":"pending", "webhook_logs":[{"id":"webhl_nKYa1YWSLNhrWLU", "url":"http://kipping.mx/conekta_webhook", "failed_attempts":0, "last_http_response_status":-1, "object":"webhook_log", "last_attempted_at":1468265161}], "id":"5783f2c67b823b46ee0021af", "object":"event", "type":"payout.paid_out", "created_at":1468265158, "webhook":{"data":{"object":{"livemode":true, "currency":"MXN", "method":{"id":"pytmtd_hqsxocPm8CZoNu", "created_at":1467992925, "payee_id":nil, "_type":"BankTransferPayoutMethod", "object":"bank_transfer_payout_method", "account_number":"072580002374915018", "account_holder":"UKKO SOLUTIONS SA DE CV", "bank":"Banco Mercantil del Norte, S.A."}, "object":"transfer", "amount":453320, "id":"577fcb5d7b823b818e001791", "created_at":1467992925, "status":"scheduled", "statement_reference":"3121951", "statement_description":"Conekta 3121951"}, "previous_attributes":{}}, "livemode":true, "webhook_status":"pending", "webhook_logs":[{"id":"webhl_nKYa1YWSLNhrWLU", "url":"http://kipping.mx/conekta_webhook", "failed_attempts":0, "last_http_response_status":-1, "object":"webhook_log", "last_attempted_at":1468265161}], "id":"5783f2c67b823b46ee0021af", "object":"event", "type":"payout.paid_out", "created_at":1468265158}} |
133 | payoutHandler = class_double("Var::ConektaPayoutHandler"). | 133 | payoutHandler = class_double("Var::ConektaPayoutHandler"). |
134 | as_stubbed_const(transfer_nested_constants: true) | 134 | as_stubbed_const(transfer_nested_constants: true) |
135 | expect(payoutHandler).to receive(:analyze).with(params) | 135 | expect(payoutHandler).to receive(:analyze).with(params) | ... | ... |
-
Please register or login to post a comment