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

framework upgrade to 0.50.0 #39

Merged
merged 23 commits into from
May 20, 2024
Merged

framework upgrade to 0.50.0 #39

merged 23 commits into from
May 20, 2024

Conversation

alyn509
Copy link
Contributor

@alyn509 alyn509 commented May 9, 2024

No description provided.

@alyn509 alyn509 changed the title framework upgrade to 0.49.0 framework upgrade to 0.50.0 May 13, 2024
Copy link

github-actions bot commented May 14, 2024

Coverage Summary

Totals

Count Covered %
Lines 2952 2338 79.20
Regions 1057 566 53.55
Functions 301 193 64.12
Instantiations 1739 314 18.06

Files

Expand
File Lines Regions Functions Instantiations
/esdt-system-sc-mock/meta/src/main.rs 0.00% 0.00% 0.00% 0.00%
/esdt-system-sc-mock/src/esdt_system_sc_mock.rs 88.37% 72.73% 70.00% 17.98%
/esdt-system-sc-mock/tests/esdt_system_sc_mock_go_test.rs 100.00% 100.00% 100.00% 100.00%
/esdt-system-sc-mock/tests/esdt_system_sc_mock_rs_test.rs 100.00% 100.00% 100.00% 100.00%
/nft-minter-deployer/meta/src/main.rs 0.00% 0.00% 0.00% 0.00%
/nft-minter-deployer/src/events.rs 51.72% 26.67% 60.00% 13.64%
/nft-minter-deployer/src/factory.rs 100.00% 57.14% 100.00% 15.79%
/nft-minter-deployer/src/lib.rs 85.29% 67.86% 80.00% 14.67%
/nft-minter-deployer/tests/nft_minter_deployer_test.rs 100.00% 100.00% 100.00% 100.00%
/nft-minter/meta/src/main.rs 0.00% 0.00% 0.00% 0.00%
/nft-minter/src/admin_whitelist.rs 60.00% 66.67% 75.00% 14.75%
/nft-minter/src/brand_creation.rs 88.28% 59.30% 76.92% 20.30%
/nft-minter/src/common_storage.rs 63.64% 34.83% 63.64% 18.47%
/nft-minter/src/events.rs 100.00% 75.00% 100.00% 15.79%
/nft-minter/src/lib.rs 95.00% 60.71% 100.00% 24.72%
/nft-minter/src/nft_attributes_builder.rs 96.83% 82.61% 100.00% 17.24%
/nft-minter/src/nft_marketplace_interactor.rs 0.00% 0.00% 0.00% 0.00%
/nft-minter/src/nft_minter_proxy.rs 18.39% 4.76% 11.94% 6.45%
/nft-minter/src/nft_minting.rs 96.24% 74.47% 100.00% 9.84%
/nft-minter/src/nft_tier.rs 100.00% 85.71% 100.00% 16.90%
/nft-minter/src/royalties.rs 69.23% 66.67% 66.67% 12.29%
/nft-minter/src/views.rs 58.14% 12.31% 15.38% 2.25%
/nft-minter/tests/nft_minter_interactor/mod.rs 92.67% 87.80% 91.67% 91.67%
/nft-minter/tests/rust_test.rs 100.00% 100.00% 100.00% 100.00%
/royalties-handler/meta/src/main.rs 0.00% 0.00% 0.00% 0.00%
/royalties-handler/src/common_storage.rs 100.00% 25.00% 100.00% 9.09%
/royalties-handler/src/lib.rs 100.00% 25.00% 100.00% 7.41%
/royalties-handler/src/nft_minter_interactor.rs 100.00% 50.00% 100.00% 15.38%
/royalties-handler/src/reward_entries.rs 95.89% 64.44% 81.82% 22.22%
/royalties-handler/src/shareholders.rs 43.48% 33.33% 33.33% 5.00%
/royalties-handler/src/token_balance.rs 72.41% 50.00% 80.00% 19.23%
/royalties-handler/tests/nft_minter_setup/mod.rs 99.41% 92.00% 100.00% 100.00%
/royalties-handler/tests/royalties_handler_setup/mod.rs 100.00% 100.00% 100.00% 100.00%
/royalties-handler/tests/rust_test.rs 100.00% 100.00% 100.00% 100.00%

Copy link

github-actions bot commented May 14, 2024

Contract comparison - from c05fa37 to 23c8212

Path                                                                                             size                  has-allocator                     has-format
mx-nft-collection-minter-sc
- esdt-system-sc-mock/esdt-system-sc-mock.wasm 4454 ➡️ 3783 🟢 No No
- nft-minter/nft-minter.wasm 26389 ➡️ 25192 🟢 No No
- nft-minter-deployer/nft-minter-deployer.wasm 5408 ➡️ 5220 🟢 No No
- royalties-handler/royalties-handler.wasm 10641 ➡️ 10163 🟢 No No

nft-minter-deployer/src/lib.rs Outdated Show resolved Hide resolved
royalties-handler/src/nft_minter_interactor.rs Outdated Show resolved Hide resolved
@alyn509 alyn509 merged commit c198141 into main May 20, 2024
6 checks passed
@alyn509 alyn509 deleted the sc-upgrade branch May 20, 2024 11:09
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.

4 participants