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

Feat add benchmarks payments #325

Merged
merged 5 commits into from
Nov 24, 2023
Merged

Feat add benchmarks payments #325

merged 5 commits into from
Nov 24, 2023

Conversation

darkforest0202
Copy link
Contributor

This PR adds the benchmarking for all the extrinsics defined in the pallet.

Here the successful test executions:

➜  virto-node git:(feat-add-benchmarks-payments)  cargo test --package pallet-payments --features runtime-benchmarks
    Blocking waiting for file lock on package cache
    Blocking waiting for file lock on build directory
   Compiling pallet-payments v0.1.0 (/Users/hectorb/side/dark/virto-node/pallets/payments)
    Finished test [unoptimized] target(s) in 7.70s
     Running unittests src/lib.rs (target/debug/deps/pallet_payments-f80d3ba3c5e8ade5)

running 8 tests
test tests::payment_refunded_request ... ok
test tests::request_payment ... ok
test mock::__construct_runtime_integrity_test::runtime_integrity_tests ... ok
test tests::test_pay_and_cancel_works ... ok
test tests::test_pay_and_release_works ... ok
test tests::payment_disputed_sender_wins ... ok
test tests::payment_disputed_beneficiary_wins ... ok
test benchmarking::benchmarks::benchmark_tests::test_benchmarks ... ok

test result: ok. 8 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.03s

   Doc-tests pallet-payments

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

Please note that benchmarks cannot be executed until the task #309 is not completed.

@olanod olanod merged commit 3b2c09a into master Nov 24, 2023
2 checks passed
@olanod olanod deleted the feat-add-benchmarks-payments branch November 24, 2023 07:07
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.

2 participants