Fix IDs not being serialized as strings in annual reports API (#32710)

This commit is contained in:
Eugen Rochko 2024-10-31 10:38:00 +01:00 committed by GitHub
parent 01e25af2e3
commit c2b498a2b0
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
6 changed files with 10 additions and 10 deletions

View file

@ -7,7 +7,7 @@ class AnnualReport::CommonlyInteractedWithAccounts < AnnualReport::Source
{ {
commonly_interacted_with_accounts: commonly_interacted_with_accounts.map do |(account_id, count)| commonly_interacted_with_accounts: commonly_interacted_with_accounts.map do |(account_id, count)|
{ {
account_id: account_id, account_id: account_id.to_s,
count: count, count: count,
} }
end, end,

View file

@ -7,7 +7,7 @@ class AnnualReport::MostRebloggedAccounts < AnnualReport::Source
{ {
most_reblogged_accounts: most_reblogged_accounts.map do |(account_id, count)| most_reblogged_accounts: most_reblogged_accounts.map do |(account_id, count)|
{ {
account_id: account_id, account_id: account_id.to_s,
count: count, count: count,
} }
end, end,

View file

@ -8,9 +8,9 @@ class AnnualReport::TopStatuses < AnnualReport::Source
{ {
top_statuses: { top_statuses: {
by_reblogs: top_reblogs, by_reblogs: top_reblogs&.to_s,
by_favourites: top_favourites, by_favourites: top_favourites&.to_s,
by_replies: top_replies, by_replies: top_replies&.to_s,
}, },
} }
end end

View file

@ -32,7 +32,7 @@ RSpec.describe AnnualReport::CommonlyInteractedWithAccounts do
expect(subject.generate) expect(subject.generate)
.to include( .to include(
commonly_interacted_with_accounts: contain_exactly( commonly_interacted_with_accounts: contain_exactly(
include(account_id: other_account.id, count: 2) include(account_id: other_account.id.to_s, count: 2)
) )
) )
end end

View file

@ -32,7 +32,7 @@ RSpec.describe AnnualReport::MostRebloggedAccounts do
expect(subject.generate) expect(subject.generate)
.to include( .to include(
most_reblogged_accounts: contain_exactly( most_reblogged_accounts: contain_exactly(
include(account_id: other_account.id, count: 2) include(account_id: other_account.id.to_s, count: 2)
) )
) )
end end

View file

@ -39,9 +39,9 @@ RSpec.describe AnnualReport::TopStatuses do
expect(subject.generate) expect(subject.generate)
.to include( .to include(
top_statuses: include( top_statuses: include(
by_reblogs: reblogged_status.id, by_reblogs: reblogged_status.id.to_s,
by_favourites: favourited_status.id, by_favourites: favourited_status.id.to_s,
by_replies: replied_status.id by_replies: replied_status.id.to_s
) )
) )
end end