Skip to content

feat: add welcome pack correction jobs and integration tests#1151

Merged
bryzettler merged 2 commits intodevelopfrom
bugfix/atomic-data-publisher-correction-jobs
Mar 17, 2026
Merged

feat: add welcome pack correction jobs and integration tests#1151
bryzettler merged 2 commits intodevelopfrom
bugfix/atomic-data-publisher-correction-jobs

Conversation

@bryzettler
Copy link
Contributor

Add welcome_pack_ownership and welcome_pack_reward_destination polling jobs that re-publish correct ownership/reward data when welcome_packs is indexed after asset_owners. INNER JOIN guards prevent publishing when asset_owners hasn't been indexed yet.

Includes 4 integration tests validating the INNER JOIN guard behavior and happy-path resolution for both queries.

Add welcome_pack_ownership and welcome_pack_reward_destination polling
jobs that re-publish correct ownership/reward data when welcome_packs
is indexed after asset_owners. INNER JOIN guards prevent publishing
when asset_owners hasn't been indexed yet.

Includes 4 integration tests validating the INNER JOIN guard behavior
and happy-path resolution for both queries.
@bryzettler bryzettler requested a review from ChewingGlass March 17, 2026 03:33
@bryzettler bryzettler force-pushed the bugfix/atomic-data-publisher-correction-jobs branch from d817769 to 21e576c Compare March 17, 2026 15:11
Extract shared helpers block_boundary_for_table and next_block_for_table
to consolidate duplicated SQL queries across correction job functions.
Simplify test helpers with run_block_range_query.
@bryzettler bryzettler force-pushed the bugfix/atomic-data-publisher-correction-jobs branch from 21e576c to 0e4878f Compare March 17, 2026 15:11
@bryzettler bryzettler merged commit b9c97de into develop Mar 17, 2026
84 of 85 checks passed
@bryzettler bryzettler deleted the bugfix/atomic-data-publisher-correction-jobs branch March 17, 2026 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants