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 unwrap and create transaction issue and added unit tests #242

Merged
merged 3 commits into from
Oct 29, 2024

Conversation

evelinemolnar
Copy link
Contributor

No description provided.

Copy link

github-actions bot commented Oct 29, 2024

Coverage Summary

Totals

Count Covered %
Lines 11457 6229 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.80% 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 618817c to 4b118d7

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 9195 ➡️ 9152 🟢 false without message
bridged-tokens-wrapper-dbg.wasm 26808 ➡️ 26745 🟢 false without message
multisig-dbg.wasm 72044 false without message
multisig.wasm 29365 false without message

dragos-rebegea
dragos-rebegea previously approved these changes Oct 29, 2024
@evelinemolnar evelinemolnar marked this pull request as ready for review October 29, 2024 14:10
@dragos-rebegea dragos-rebegea merged commit 2ac831c into feat/v3 Oct 29, 2024
5 checks passed
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