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

tests: Added tests for each SubintentStructureError #2013

Conversation

dhedey
Copy link
Contributor

@dhedey dhedey commented Nov 22, 2024

Summary

Added tests for each SubintentStructureError

Copy link

github-actions bot commented Nov 22, 2024

Docker tags
docker.io/radixdlt/private-scrypto-builder:757913ce08

Copy link

github-actions bot commented Nov 22, 2024

Benchmark for 757913c

Click to view benchmark
Test Base PR %
costing::bench_prepare_wasm 45.1±0.22ms 45.0±0.16ms -0.22%
costing::decode_encoded_i8_array_to_manifest_raw_value 19.3±0.01ms 19.4±0.03ms +0.52%
costing::decode_encoded_i8_array_to_manifest_value 40.7±0.06ms 42.3±0.08ms +3.93%
costing::decode_encoded_tuple_array_to_manifest_raw_value 70.4±0.07ms 71.9±0.05ms +2.13%
costing::decode_encoded_tuple_array_to_manifest_value 98.7±0.13ms 97.9±0.20ms -0.81%
costing::decode_encoded_u8_array_to_manifest_raw_value 25.6±0.17µs 32.2±0.13µs +25.78%
costing::decode_encoded_u8_array_to_manifest_value 40.7±0.03ms 42.3±0.08ms +3.93%
costing::decode_rpd_to_manifest_raw_value 14.6±0.02µs 14.6±0.03µs 0.00%
costing::decode_rpd_to_manifest_value 10.7±0.01µs 10.9±0.08µs +1.87%
costing::deserialize_wasm 1223.4±6.55µs 1255.4±3.85µs +2.62%
costing::execute_transaction_creating_big_vec_substates 682.2±2.95ms 690.6±9.13ms +1.23%
costing::execute_transaction_reading_big_vec_substates 577.2±3.44ms 601.1±2.15ms +4.14%
costing::instantiate_flash_loan 981.6±1271.26µs 921.5±863.50µs -6.12%
costing::instantiate_radiswap 1089.0±1650.81µs 886.9±402.89µs -18.56%
costing::scrypto_malloc 703.0±2.85ms 673.7±2.19ms -4.17%
costing::scrypto_sbor_decode 684.1±1.15ms 664.1±1.42ms -2.92%
costing::scrypto_sha256 608.5±1.66ms 592.7±1.00ms -2.60%
costing::spin_loop_v1 522.0±2.59ms 512.6±1.13ms -1.80%
costing::spin_loop_v2 611.0±6.75ms 597.8±9.03ms -2.16%
costing::validate_sbor_payload 29.2±0.08µs 29.4±0.08µs +0.68%
costing::validate_sbor_payload_bytes 247.5±0.62ns 259.5±0.55ns +4.85%
costing::validate_secp256k1 76.4±0.10µs 76.5±0.07µs +0.13%
costing::validate_wasm 34.0±0.06ms 34.0±0.06ms 0.00%
decimal::add/0 8.4±0.00ns 8.4±0.01ns 0.00%
decimal::add/rust-native 9.8±0.00ns 9.8±0.00ns 0.00%
decimal::add/wasmi 316.5±1.29ns 322.9±0.61ns +2.02%
decimal::add/wasmi-call-native 2.9±0.00µs 2.9±0.00µs 0.00%
decimal::div/0 168.5±0.13ns 168.6±0.21ns +0.06%
decimal::from_string/0 157.0±0.08ns 158.0±0.11ns +0.64%
decimal::mul/0 128.3±0.14ns 128.2±0.09ns -0.08%
decimal::mul/rust-native 128.4±0.05ns 124.9±0.13ns -2.73%
decimal::mul/wasmi 19.4±0.04µs 19.3±0.17µs -0.52%
decimal::mul/wasmi-call-native 3.0±0.00µs 3.0±0.00µs 0.00%
decimal::pow/0 607.4±0.44ns 590.6±0.08ns -2.77%
decimal::pow/rust-native 590.3±0.25ns 590.8±1.30ns +0.08%
decimal::pow/wasmi 91.4±0.11µs 91.0±0.21µs -0.44%
decimal::pow/wasmi-call-native 4.8±0.00µs 4.7±0.00µs -2.08%
decimal::root/0 8.2±0.01µs 8.2±0.01µs 0.00%
decimal::sub/0 8.2±0.00ns 8.2±0.00ns 0.00%
decimal::to_string/0 442.4±0.24ns 441.8±0.20ns -0.14%
large_transaction_processing::prepare 2.5±0.01ms 2.5±0.00ms 0.00%
large_transaction_processing::prepare_and_decompile 6.4±0.02ms 6.3±0.03ms -1.56%
large_transaction_processing::prepare_and_decompile_and_recompile 31.8±0.29ms 24.8±0.06ms -22.01%
metadata_validation::validate_urls 4.8±0.20µs 4.8±0.03µs 0.00%
precise_decimal::add/0 8.9±0.01ns 8.9±0.03ns 0.00%
precise_decimal::add/rust-native 10.6±0.01ns 10.6±0.01ns 0.00%
precise_decimal::add/wasmi 414.1±2.32ns 412.2±1.26ns -0.46%
precise_decimal::add/wasmi-call-native 3.9±0.01µs 3.8±0.01µs -2.56%
precise_decimal::div/0 310.9±1.65ns 312.5±2.97ns +0.51%
precise_decimal::from_string/0 201.8±0.20ns 201.9±0.28ns +0.05%
precise_decimal::mul/0 339.0±0.93ns 341.3±1.53ns +0.68%
precise_decimal::mul/rust-native 284.2±0.26ns 283.0±0.25ns -0.42%
precise_decimal::mul/wasmi 47.7±0.31µs 47.8±0.09µs +0.21%
precise_decimal::mul/wasmi-call-native 4.3±0.00µs 4.2±0.00µs -2.33%
precise_decimal::pow/0 1720.6±0.64ns 1714.5±0.79ns -0.35%
precise_decimal::pow/rust-native 1356.6±2.07ns 1355.4±1.82ns -0.09%
precise_decimal::pow/wasmi 229.6±1.02µs 230.3±0.37µs +0.30%
precise_decimal::pow/wasmi-call-native 7.7±0.05µs 7.6±0.01µs -1.30%
precise_decimal::root/0 58.4±0.04µs 57.5±0.08µs -1.54%
precise_decimal::sub/0 9.1±0.05ns 9.1±0.05ns 0.00%
precise_decimal::to_string/0 692.9±0.30ns 693.0±0.46ns +0.01%
schema::validate_payload 394.5±1.27µs 387.1±0.97µs -1.88%
transaction::radiswap 5.0±0.02ms 5.0±0.02ms 0.00%
transaction::transfer 1793.2±2.48µs 1811.3±3.18µs +1.01%
transaction_validation::validate_manifest 43.2±0.05µs 43.2±0.04µs 0.00%
transaction_validation::verify_bls_2KB 967.2±16.24µs 985.6±18.19µs +1.90%
transaction_validation::verify_bls_32B 965.9±17.44µs 971.2±14.50µs +0.55%
transaction_validation::verify_ecdsa 74.5±0.04µs 74.4±0.06µs -0.13%
transaction_validation::verify_ed25519 42.2±0.07µs 42.2±0.07µs 0.00%

…dation-tests

Add further transaction validation tests
@iamyulong iamyulong merged commit 603b3c1 into tests/transaction-validation-refactor Nov 26, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants