Skip to content

fix(indexer): touch ups #340

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

fix(indexer): touch ups #340

wants to merge 7 commits into from

Conversation

@KodeyThomas
Copy link
Contributor

@KodeyThomas KodeyThomas commented Nov 18, 2025

No description provided.

carte7000
carte7000 reviewed Nov 18, 2025
View reviewed changes
carte7000
carte7000 reviewed Nov 18, 2025
View reviewed changes
carte7000
carte7000 previously approved these changes Nov 18, 2025
View reviewed changes
carte7000
carte7000 previously approved these changes Nov 18, 2025
View reviewed changes
@KodeyThomas KodeyThomas marked this pull request as ready for review November 18, 2025 16:13
@KodeyThomas KodeyThomas requested review from a team and skudasov as code owners November 18, 2025 16:13
@KodeyThomas KodeyThomas enabled auto-merge (squash) November 18, 2025 16:13
@github-actions
Copy link

github-actions bot commented Nov 19, 2025

Code coverage report:

Package main kodey/indexer-nits
aggregator 37.36% 46.17%
cmd 0.00% 0.00%
committee 100.00% 100.00%
common 0.00% 0.00%
executor 34.24% 58.21%
indexer 29.98% 25.95%
integration 14.77% 15.34%
protocol 33.53% 48.13%
verifier 59.92% 59.92%

@github-actions
Copy link

github-actions bot commented Nov 19, 2025

E2E Smoke Test Results

Test Case Status Duration
TestE2ESmoke/extra_args_v2/src->dst_msg_execution_eoa_receiver pass 5.02s
TestE2ESmoke/extra_args_v2/dst->src_msg_execution_eoa_receiver pass 6.02s
TestE2ESmoke/extra_args_v2/1337->3337_msg_execution_mock_receiver pass 5.02s
TestE2ESmoke/extra_args_v2 pass 16.07s
TestE2ESmoke/extra_args_v3_messaging/EOA_receiver_and_default_committee_verifier pass 5.02s
TestE2ESmoke/extra_args_v3_messaging/EOA_receiver_and_secondary_committee_verifier fail 41.02s
TestE2ESmoke/extra_args_v3_messaging/receiver_w/_secondary_verifier_required fail 41.02s
TestE2ESmoke/extra_args_v3_messaging/receiver_w/_secondary_required_and_tertiary_optional_threshold=1 fail 41.02s
TestE2ESmoke/extra_args_v3_messaging/receiver_w/_default_required,_secondary_and_tertiary_optional,_threshold=1,_message_specifies_all_three fail 41.02s
TestE2ESmoke/extra_args_v3_messaging/receiver_w/_default_required,_secondary_and_tertiary_optional,_threshold=1,_message_specifies_default_and_secondary fail 41.02s
TestE2ESmoke/extra_args_v3_messaging/receiver_w/_default_required,_secondary_and_tertiary_optional,_threshold=1,_message_specifies_default_and_tertiary fail 41.02s
TestE2ESmoke/extra_args_v3_messaging/EOA_receiver_and_default_committee_verifier#01 pass 5.02s
TestE2ESmoke/extra_args_v3_messaging/max_data_size pass 5.02s
TestE2ESmoke/extra_args_v3_messaging fail 261.2s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_EOA_receiver_and_token_transfer_(TEST_(BurnMintTokenPool_1.6.1_[]_to_BurnMintTokenPool_1.6.1_[])) pass 7.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_EOA_receiver_and_token_transfer_(TEST_(LockReleaseTokenPool_1.7.0_[default]_to_LockReleaseTokenPool_1.7.0_[default])) pass 6.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_EOA_receiver_and_token_transfer_(TEST_(BurnMintTokenPool_1.7.0_[default]_to_BurnMintTokenPool_1.7.0_[default])) pass 6.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_EOA_receiver_and_token_transfer_(TEST_(BurnMintTokenPool_1.6.1_[]_to_BurnMintTokenPool_1.7.0_[default])) pass 7.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_EOA_receiver_and_token_transfer_(TEST_(BurnMintTokenPool_1.7.0_[default]_to_BurnMintTokenPool_1.6.1_[])) pass 6.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_EOA_receiver_and_token_transfer_(TEST_(BurnMintTokenPool_1.7.0_[default_secondary]_to_BurnMintTokenPool_1.7.0_[default_secondary])) fail 47.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_EOA_receiver_and_token_transfer_(TEST_(BurnMintTokenPool_1.7.0_[]_to_BurnMintTokenPool_1.7.0_[])) pass 6.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_EOA_receiver_and_token_transfer_(TEST_(BurnMintTokenPool_1.7.0_[secondary]_to_BurnMintTokenPool_1.7.0_[secondary])) fail 47.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_EOA_receiver_and_token_transfer_1.7.0_(TEST_(LockReleaseTokenPool_1.7.0_[default]_to_LockReleaseTokenPool_1.7.0_[default]))_default_finality pass 7.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_mock_receiver_and_token_transfer_1.7.0_(TEST_(LockReleaseTokenPool_1.7.0_[default]_to_LockReleaseTokenPool_1.7.0_[default]))_default_finality pass 7.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_EOA_receiver_and_token_transfer_1.7.0_(TEST_(BurnMintTokenPool_1.7.0_[default]_to_BurnMintTokenPool_1.7.0_[default]))_default_finality pass 7.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_mock_receiver_and_token_transfer_1.7.0_(TEST_(BurnMintTokenPool_1.7.0_[default]_to_BurnMintTokenPool_1.7.0_[default]))_default_finality pass 7.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_EOA_receiver_and_token_transfer_1.7.0_(TEST_(BurnMintTokenPool_1.7.0_[default_secondary]_to_BurnMintTokenPool_1.7.0_[default_secondary]))_default_finality fail 47.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_mock_receiver_and_token_transfer_1.7.0_(TEST_(BurnMintTokenPool_1.7.0_[default_secondary]_to_BurnMintTokenPool_1.7.0_[default_secondary]))_default_finality fail 47.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_EOA_receiver_and_token_transfer_1.7.0_(TEST_(BurnMintTokenPool_1.7.0_[]_to_BurnMintTokenPool_1.7.0_[]))_default_finality pass 7.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_mock_receiver_and_token_transfer_1.7.0_(TEST_(BurnMintTokenPool_1.7.0_[]_to_BurnMintTokenPool_1.7.0_[]))_default_finality pass 7.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_EOA_receiver_and_token_transfer_1.7.0_(TEST_(BurnMintTokenPool_1.7.0_[secondary]_to_BurnMintTokenPool_1.7.0_[secondary]))_default_finality fail 47.03s
TestE2ESmoke/extra_args_v3_token_transfer/src_dst_msg_execution_with_mock_receiver_and_token_transfer_1.7.0_(TEST_(BurnMintTokenPool_1.7.0_[secondary]_to_BurnMintTokenPool_1.7.0_[secondary]))_default_finality fail 47.03s
TestE2ESmoke/extra_args_v3_token_transfer fail 362.55s
TestE2ESmoke fail 639.97s

Full logs are available in the workflow artifacts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@carte7000 carte7000 carte7000 left review comments

@skudasov skudasov Awaiting requested review from skudasov skudasov is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@KodeyThomas @carte7000