mirror of
https://github.com/we-promise/sure.git
synced 2026-08-03 23:52:13 +00:00
5b9fd1e1a0
* fix(enable-banking): handle overdraft balance sync * fix(enable-banking): skip nil provider balances * test(enable-banking): cover nil provider balance * fix(enable-banking): avoid stale and unsafe balance logs * fix(enable-banking): guard unavailable balance writes * test(enable-banking): stub unsaved account api id * style(enable-banking): remove extra test blank line --------- Signed-off-by: Juan José Mata <juanjo.mata@gmail.com> Co-authored-by: Juan José Mata <juanjo.mata@gmail.com>
102 lines
3.5 KiB
Ruby
102 lines
3.5 KiB
Ruby
require "test_helper"
|
|
|
|
class EnableBankingAccount::ProcessorTest < ActiveSupport::TestCase
|
|
setup do
|
|
@family = families(:dylan_family)
|
|
@account = accounts(:depository)
|
|
@enable_banking_item = EnableBankingItem.create!(
|
|
family: @family,
|
|
name: "Test EB",
|
|
country_code: "FR",
|
|
application_id: "app_id",
|
|
client_certificate: "cert"
|
|
)
|
|
@enable_banking_account = EnableBankingAccount.create!(
|
|
enable_banking_item: @enable_banking_item,
|
|
name: "Compte courant",
|
|
uid: "hash_abc",
|
|
currency: "EUR",
|
|
current_balance: 1500.00
|
|
)
|
|
AccountProvider.create!(account: @account, provider: @enable_banking_account)
|
|
end
|
|
|
|
test "calls set_current_balance instead of direct account update" do
|
|
EnableBankingAccount::Processor.new(@enable_banking_account).process
|
|
|
|
assert_equal 1500.0, @account.reload.cash_balance
|
|
end
|
|
|
|
test "updates account currency" do
|
|
@enable_banking_account.update!(currency: "USD")
|
|
|
|
EnableBankingAccount::Processor.new(@enable_banking_account).process
|
|
|
|
assert_equal "USD", @account.reload.currency
|
|
end
|
|
|
|
test "does nothing when no linked account" do
|
|
@account.account_providers.destroy_all
|
|
|
|
result = EnableBankingAccount::Processor.new(@enable_banking_account).process
|
|
assert_nil result
|
|
end
|
|
|
|
test "skips balance update when provider current_balance is nil" do
|
|
@account.update!(cash_balance: 987.65)
|
|
@enable_banking_account.update_columns(current_balance: nil)
|
|
|
|
EnableBankingAccount::Transactions::Processor.any_instance.expects(:process).once
|
|
|
|
assert_no_changes -> { @account.reload.cash_balance } do
|
|
EnableBankingAccount::Processor.new(@enable_banking_account).process
|
|
end
|
|
end
|
|
|
|
test "sets CC balance as absolute debt and tracks available_credit when limit is present" do
|
|
cc_account = accounts(:credit_card)
|
|
@enable_banking_account.update!(
|
|
current_balance: 450.00,
|
|
credit_limit: 1000.00
|
|
)
|
|
AccountProvider.find_by(provider: @enable_banking_account)&.destroy
|
|
AccountProvider.create!(account: cc_account, provider: @enable_banking_account)
|
|
|
|
EnableBankingAccount::Processor.new(@enable_banking_account).process
|
|
|
|
assert_equal 450.0, cc_account.reload.cash_balance
|
|
if cc_account.accountable.respond_to?(:available_credit)
|
|
assert_equal 550.0, cc_account.accountable.reload.available_credit
|
|
end
|
|
end
|
|
|
|
test "sets CC balance as absolute debt and keeps stored available_credit when limit absent" do
|
|
cc_account = accounts(:credit_card)
|
|
cc_account.accountable.update!(available_credit: 1000.0)
|
|
|
|
@enable_banking_account.update!(current_balance: 300.00, credit_limit: nil)
|
|
|
|
AccountProvider.find_by(provider: @enable_banking_account)&.destroy
|
|
AccountProvider.create!(account: cc_account, provider: @enable_banking_account)
|
|
|
|
EnableBankingAccount::Processor.new(@enable_banking_account).process
|
|
|
|
assert_equal 300.0, cc_account.reload.cash_balance
|
|
assert_equal 1000.0, cc_account.accountable.reload.available_credit
|
|
end
|
|
|
|
test "sets CC balance to absolute debt when both limit and stored available_credit are absent" do
|
|
cc_account = accounts(:credit_card)
|
|
cc_account.accountable.update!(available_credit: nil)
|
|
|
|
@enable_banking_account.update!(current_balance: 300.00, credit_limit: nil)
|
|
|
|
AccountProvider.find_by(provider: @enable_banking_account)&.destroy
|
|
AccountProvider.create!(account: cc_account, provider: @enable_banking_account)
|
|
|
|
EnableBankingAccount::Processor.new(@enable_banking_account).process
|
|
|
|
assert_equal 300.0, cc_account.reload.cash_balance
|
|
end
|
|
end
|