Docker/Maybe
Docker/Maybe
1
0
Fork 0
mirror of https://github.com/maybe-finance/maybe.git synced 2025-07-19 13:19:39 +02:00
Code Issues Releases Activity Actions 1

Split family and Plaid item syncs into multiple jobs (#1799)

Browse source 
* Split family and Plaid item syncs into multiple jobs

* fix test expectations
This commit is contained in:
Zach Gollwitzer 2025-02-04 14:22:44 -05:00 • committed by GitHub
parent 75c8627577
commit c9c5eb315a
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 5 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

8
app/models/family.rb
View file

@ -37,16 +37,12 @@ class Family < ApplicationRecord
update!(last_synced_at: Time.current)
accounts.manual.each do |account|
account.sync_data(start_date: start_date)
account.sync_later(start_date: start_date)
end
plaid_data = []
plaid_items.each do |plaid_item|
plaid_data << plaid_item.sync_data(start_date: start_date)
plaid_item.sync_later(start_date: start_date)
end
plaid_data
end
def post_sync

2
app/models/plaid_item.rb
View file

@ -41,7 +41,7 @@ class PlaidItem < ApplicationRecord
plaid_data = fetch_and_load_plaid_data
accounts.each do |account|
account.sync_data(start_date: start_date)
account.sync_later(start_date: start_date)
end
plaid_data

4
test/models/family_test.rb
View file

@ -16,11 +16,11 @@ class FamilyTest < ActiveSupport::TestCase
manual_accounts_count = @syncable.accounts.manual.count
items_count = @syncable.plaid_items.count
Account.any_instance.expects(:sync_data)
Account.any_instance.expects(:sync_later)
.with(start_date: nil)
.times(manual_accounts_count)
PlaidItem.any_instance.expects(:sync_data)
PlaidItem.any_instance.expects(:sync_later)
.with(start_date: nil)
.times(items_count)