Skip to content
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.

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

Fix “update_lowest_tx_id” fails #243

Merged
merged 2 commits into from
Oct 29, 2024
Merged

Conversation

evelinemolnar
Copy link
Contributor

No description provided.

Copy link

github-actions bot commented Oct 29, 2024

Coverage Summary

Totals

Count Covered %
Lines 11463 6232 54.37
Regions 1877 1025 54.61
Functions 1054 505 47.91
Instantiations 5212 1279 24.54

Files

Expand
File Lines Regions Functions Instantiations
/home/runner/.cargo/git/checkouts/mx-contracts-rs-13011bd47afef959/d91bbff/contracts/crowdfunding-esdt/src/crowdfunding_esdt.rs 51.56% 57.14% 77.78% 64.00%
/home/runner/.cargo/git/checkouts/mx-contracts-rs-13011bd47afef959/d91bbff/contracts/crowdfunding-esdt/src/crowdfunding_esdt_proxy.rs 100.00% 100.00% 100.00% 100.00%
/bridge-proxy/meta/src/main.rs 0.00% 0.00% 0.00% 0.00%
/bridge-proxy/src/bridge-proxy.rs 48.84% 62.71% 62.50% 19.42%
/bridge-proxy/src/bridge_proxy_contract_proxy.rs 47.93% 44.44% 44.44% 51.35%
/bridge-proxy/src/bridged_tokens_wrapper_proxy.rs 27.04% 27.27% 27.27% 15.79%
/bridge-proxy/src/config.rs 60.71% 53.33% 75.00% 22.92%
/bridge-proxy/src/esdt_safe_proxy.rs 0.00% 0.00% 0.00% 0.00%
/bridge-proxy/tests/bridge_proxy_blackbox_test.rs 97.20% 96.00% 90.00% 90.00%
/bridged-tokens-wrapper/meta/src/main.rs 0.00% 0.00% 0.00% 0.00%
/bridged-tokens-wrapper/src/dfp_big_uint.rs 100.00% 100.00% 100.00% 43.48%
/bridged-tokens-wrapper/src/esdt_safe_proxy.rs 2.74% 5.71% 4.92% 3.57%
/bridged-tokens-wrapper/src/events.rs 100.00% 33.33% 100.00% 8.82%
/bridged-tokens-wrapper/src/lib.rs 100.00% 100.00% 100.00% 25.37%
/bridged-tokens-wrapper/tests/bridged_tokens_wrapper_whitebox_test.rs 90.47% 91.28% 90.62% 90.62%
/bridged-tokens-wrapper/tests/dfp_big_uint_test.rs 100.00% 100.00% 100.00% 100.00%
/bridged-tokens-wrapper/tests/scenario_go_test.rs 100.00% 100.00% 100.00% 100.00%
/common/eth-address/src/lib.rs 93.33% 66.67% 88.89% 46.74%
/common/fee-estimator-module/src/lib.rs 100.00% 93.75% 100.00% 20.26%
/common/fee-estimator-module/src/price_aggregator_proxy.rs 6.14% 4.76% 6.67% 5.26%
/common/max-bridged-amount-module/src/lib.rs 100.00% 87.50% 100.00% 20.39%
/common/token-module/src/lib.rs 90.56% 79.27% 90.48% 24.12%
/common/transaction/src/lib.rs 73.58% 50.98% 66.67% 29.35%
/common/transaction/src/transaction_status.rs 83.33% 66.67% 83.33% 29.79%
/common/tx-batch-module/src/batch_status.rs 33.33% 25.00% 33.33% 7.14%
/common/tx-batch-module/src/lib.rs 88.64% 82.19% 87.50% 20.34%
/common/tx-batch-module/src/tx_batch_mapper.rs 92.54% 89.66% 100.00% 40.00%
/esdt-safe/meta/src/main.rs 0.00% 0.00% 0.00% 0.00%
/esdt-safe/src/lib.rs 70.30% 54.78% 62.16% 17.48%
/esdt-safe/tests/esdt_safe_scenario_rs_test.rs 100.00% 100.00% 100.00% 100.00%
/esdt-safe/tests/scenario_go_test.rs 100.00% 100.00% 100.00% 100.00%
/multi-transfer-esdt/meta/src/main.rs 0.00% 0.00% 0.00% 0.00%
/multi-transfer-esdt/src/bridge_proxy_contract_proxy.rs 10.06% 11.11% 11.11% 15.38%
/multi-transfer-esdt/src/bridged_tokens_wrapper_proxy.rs 44.64% 45.45% 45.45% 44.68%
/multi-transfer-esdt/src/esdt_safe_proxy.rs 31.00% 21.43% 24.59% 20.00%
/multi-transfer-esdt/src/lib.rs 83.41% 78.69% 81.25% 26.64%
/multi-transfer-esdt/src/multi_transfer_proxy.rs 44.50% 42.86% 42.86% 41.67%
/multi-transfer-esdt/tests/multi_transfer_blackbox_test.rs 99.68% 89.29% 100.00% 100.00%
/multi-transfer-esdt/tests/multi_transfer_esdt_scenario_rs_test.rs 57.14% 30.00% 30.00% 30.00%
/multi-transfer-esdt/tests/scenario_go_test.rs 30.00% 30.00% 30.00% 30.00%
/multisig/meta/src/main.rs 0.00% 0.00% 0.00% 0.00%
/multisig/src/action.rs 25.00% 22.22% 33.33% 30.77%
/multisig/src/bridge_proxy_contract_proxy.rs 0.00% 0.00% 0.00% 0.00%
/multisig/src/esdt_safe_proxy.rs 23.25% 15.71% 18.03% 10.34%
/multisig/src/events.rs 100.00% 66.67% 100.00% 12.90%
/multisig/src/lib.rs 69.77% 71.05% 70.00% 25.66%
/multisig/src/multi_transfer_esdt_proxy.rs 19.00% 22.73% 22.73% 13.64%
/multisig/src/multisig_general.rs 80.00% 65.38% 75.00% 30.77%
/multisig/src/multisig_proxy.rs 15.38% 12.24% 14.12% 13.86%
/multisig/src/queries.rs 64.52% 61.76% 57.89% 15.79%
/multisig/src/setup.rs 25.97% 32.61% 25.81% 7.00%
/multisig/src/storage.rs 100.00% 66.67% 100.00% 27.41%
/multisig/src/user_role.rs 83.33% 72.73% 75.00% 41.67%
/multisig/src/util.rs 94.44% 84.38% 100.00% 44.12%
/multisig/tests/multisig_blackbox_test.rs 100.00% 100.00% 100.00% 100.00%
/multisig/tests/multisig_scenario_rs_test.rs 82.69% 72.73% 72.73% 72.73%
/multisig/tests/scenario_go_test.rs 70.00% 70.00% 70.00% 70.00%
/test-caller/meta/src/main.rs 0.00% 0.00% 0.00% 0.00%
/test-caller/src/test-caller.rs 0.00% 0.00% 0.00% 0.00%

Copy link

github-actions bot commented Oct 29, 2024

Contract comparison - from 2ac831c to ce7f921

Path                                                                                             size                  has-allocator                     has-format
test-caller.wasm 3274 false without message
test-caller-dbg.wasm 7110 false without message
esdt-safe.wasm 26679 false without message
esdt-safe-dbg.wasm 64252 false without message
multi-transfer-esdt.wasm 17273 false without message
multi-transfer-esdt-dbg.wasm 40924 false without message
bridge-proxy-dbg.wasm 34841 false without message
bridge-proxy.wasm 13364 false without message
bridged-tokens-wrapper.wasm 9152 false without message
bridged-tokens-wrapper-dbg.wasm 26745 false without message
multisig-dbg.wasm 72044 false without message
multisig.wasm 29365 false without message

@evelinemolnar evelinemolnar marked this pull request as ready for review October 29, 2024 15:06
@dragos-rebegea dragos-rebegea merged commit 9c6c0c8 into feat/v3 Oct 29, 2024
5 checks passed
@dragos-rebegea dragos-rebegea deleted the bridge-proxy-audit-fix branch October 29, 2024 16:29
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.

3 participants