Skip to content
Snippets Groups Projects
Select Git revision
  • a07776ad84337de929479e4fc2eddf2e32dfae78
  • master default protected
  • 270-parametrage-de-la-gtest
  • network/gdev-800 protected
  • cgeek/issue-297-cpu
  • gdev-800-tests
  • update-docker-compose-rpc-squid-names
  • fix-252
  • 1000i100-test
  • hugo/tmp-0.9.1
  • network/gdev-803 protected
  • hugo/endpoint-gossip
  • network/gdev-802 protected
  • hugo/distance-precompute
  • network/gdev-900 protected
  • tuxmain/anonymous-tx
  • debug/podman
  • hugo/195-doc
  • hugo/195-graphql-schema
  • hugo-tmp-dockerfile-cache
  • release/client-800.2 protected
  • gdev-900-0.10.1 protected
  • gdev-900-0.10.0 protected
  • gdev-900-0.9.2 protected
  • gdev-800-0.8.0 protected
  • gdev-900-0.9.1 protected
  • gdev-900-0.9.0 protected
  • gdev-803 protected
  • gdev-802 protected
  • runtime-801 protected
  • gdev-800 protected
  • runtime-800-bis protected
  • runtime-800 protected
  • runtime-800-backup protected
  • runtime-701 protected
  • runtime-700 protected
  • runtime-600 protected
  • runtime-500 protected
  • v0.4.1 protected
  • runtime-401 protected
  • v0.4.0 protected
41 results

weights-benchmarking.md

Blame
    • Benjamin Gallois's avatar
      cca16dbb
      Eliminate all Instances of `Weight::zero()` (!197) · cca16dbb
      Benjamin Gallois authored and Hugo Trentesaux's avatar Hugo Trentesaux committed
      * refactor functions naming for pallet-membership
      
      * refactor functions naming for pallet-certification
      
      * fix tests with runtime-benchmark feature
      
      * fix compilation with runtime-benchmarks feature
      
      * update documentation
      
      * remove handlers weights for pallet_membership
      
      * remove handlers weights for pallet_identity
      
      * remove handlers weights for pallet_certification
      
      * add best practices for benchmarking
      
      * add hooks benchmark pallet_universal_dividend
      
      * add missing benchmarks pallet_universal_dividend
      
      * update weights
      
      * fix pallet_provide_randomness on_initialize weight
      
      * fix pallet_identity weight::zero
      
      * fix pallet_membership weight::zero
      
      * fix pallet_certification weight::zero
      
      * fix pallet_authority_members weight::zero
      
      * fix pallet_identity weight::zero
      
      * fix pallet_membership weight::zero
      cca16dbb
      History
      Eliminate all Instances of `Weight::zero()` (!197)
      Benjamin Gallois authored and Hugo Trentesaux's avatar Hugo Trentesaux committed
      * refactor functions naming for pallet-membership
      
      * refactor functions naming for pallet-certification
      
      * fix tests with runtime-benchmark feature
      
      * fix compilation with runtime-benchmarks feature
      
      * update documentation
      
      * remove handlers weights for pallet_membership
      
      * remove handlers weights for pallet_identity
      
      * remove handlers weights for pallet_certification
      
      * add best practices for benchmarking
      
      * add hooks benchmark pallet_universal_dividend
      
      * add missing benchmarks pallet_universal_dividend
      
      * update weights
      
      * fix pallet_provide_randomness on_initialize weight
      
      * fix pallet_identity weight::zero
      
      * fix pallet_membership weight::zero
      
      * fix pallet_certification weight::zero
      
      * fix pallet_authority_members weight::zero
      
      * fix pallet_identity weight::zero
      
      * fix pallet_membership weight::zero
    integration_tests.rs 14.42 KiB
    // Copyright 2021 Axiom-Team
    //
    // This file is part of Substrate-Libre-Currency.
    //
    // Substrate-Libre-Currency is free software: you can redistribute it and/or modify
    // it under the terms of the GNU Affero General Public License as published by
    // the Free Software Foundation, version 3 of the License.
    //
    // Substrate-Libre-Currency is distributed in the hope that it will be useful,
    // but WITHOUT ANY WARRANTY; without even the implied warranty of
    // MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
    // GNU Affero General Public License for more details.
    //
    // You should have received a copy of the GNU Affero General Public License
    // along with Substrate-Libre-Currency. If not, see <https://www.gnu.org/licenses/>.
    
    mod common;
    
    use common::*;
    use frame_support::traits::{PalletInfo, StorageInfo, StorageInfoTrait};
    use frame_support::{assert_err, assert_ok};
    use frame_support::{StorageHasher, Twox128};
    use gdev_runtime::*;
    use sp_keyring::AccountKeyring;
    use sp_runtime::MultiAddress;
    
    #[test]
    fn verify_treasury_account() {
        println!("{}", Treasury::account_id());
    }
    
    #[test]
    fn verify_pallet_prefixes() {
        let prefix = |pallet_name, storage_name| {
            let mut res = [0u8; 32];
            res[0..16].copy_from_slice(&Twox128::hash(pallet_name));
            res[16..32].copy_from_slice(&Twox128::hash(storage_name));
            res.to_vec()
        };
        assert_eq!(
            <Timestamp as StorageInfoTrait>::storage_info(),
            vec![
                StorageInfo {
                    pallet_name: b"Timestamp".to_vec(),
                    storage_name: b"Now".to_vec(),
                    prefix: prefix(b"Timestamp", b"Now"),
                    max_values: Some(1),
                    max_size: Some(8),
                },
                StorageInfo {
                    pallet_name: b"Timestamp".to_vec(),
                    storage_name: b"DidUpdate".to_vec(),
                    prefix: prefix(b"Timestamp", b"DidUpdate"),
                    max_values: Some(1),
                    max_size: Some(1),
                }
            ]
        );
    }
    
    #[test]
    fn verify_pallet_indices() {
        fn is_pallet_index<P: 'static>(index: usize) {
            assert_eq!(
                <Runtime as frame_system::Config>::PalletInfo::index::<P>(),
                Some(index)
            );
        }
        is_pallet_index::<System>(0);
    }
    
    #[test]
    fn verify_proxy_type_indices() {
        assert_eq!(ProxyType::Any as u8, 0);
    }
    
    #[test]
    fn test_genesis_build() {
        ExtBuilder::new(1, 3, 4).build().execute_with(|| {
            run_to_block(2);
        });
    }
    
    #[test]
    fn test_remove_identity() {
        ExtBuilder::new(1, 3, 4).build().execute_with(|| {
            run_to_block(2);
    
            assert_ok!(Identity::remove_identity(
                frame_system::RawOrigin::Root.into(),
                4,
                None
            ));
            let events = System::events();
            assert_eq!(events.len(), 3);
            assert_eq!(
                System::events()[0].event,
                Event::Membership(pallet_membership::Event::MembershipRevoked(4))
            );
            /*println!(
                "{}",
                get_account_id_from_seed::<sp_core::sr25519::Public>("Charlie")
            );*/
            assert_eq!(
                System::events()[1].event,
                Event::System(frame_system::Event::KilledAccount {
                    account: AccountKeyring::Dave.to_account_id()
                })
            );
            assert_eq!(
                System::events()[2].event,
                Event::Identity(pallet_identity::Event::IdtyRemoved { idty_index: 4 })
            );
            //println!("{:#?}", events);
        });
    }
    
    #[test]
    fn test_remove_smith_identity() {
        ExtBuilder::new(1, 3, 4).build().execute_with(|| {
            run_to_block(2);
    
            assert_ok!(Identity::remove_identity(
                frame_system::RawOrigin::Root.into(),
                3,
                None
            ));
            let events = System::events();
            assert_eq!(events.len(), 4);
            assert_eq!(
                System::events()[0].event,
                Event::SmithsMembership(pallet_membership::Event::MembershipRevoked(3))
            );
            assert_eq!(
                System::events()[1].event,
                Event::AuthorityMembers(pallet_authority_members::Event::MemberRemoved(3))
            );
            assert_eq!(
                System::events()[2].event,
                Event::Membership(pallet_membership::Event::MembershipRevoked(3))
            );
            assert_eq!(
                System::events()[3].event,
                Event::Identity(pallet_identity::Event::IdtyRemoved { idty_index: 3 })
            );
            //println!("{:#?}", events);
        });
    }
    
    #[test]
    fn test_create_new_account_with_insufficient_balance() {
        ExtBuilder::new(1, 3, 4)
            .with_initial_balances(vec![(AccountKeyring::Alice.to_account_id(), 1_000)])
            .build()
            .execute_with(|| {
                run_to_block(2);
    
                // Should be able to transfer 2 units to a new account
                assert_ok!(Balances::transfer(
                    frame_system::RawOrigin::Signed(AccountKeyring::Alice.to_account_id()).into(),
                    MultiAddress::Id(AccountKeyring::Eve.to_account_id()),
                    200
                ));
                let events = System::events();
                //println!("{:#?}", events);
                assert_eq!(events.len(), 2);
                assert_eq!(
                    System::events()[0].event,
                    Event::Balances(pallet_balances::Event::Endowed {
                        account: AccountKeyring::Eve.to_account_id(),
                        free_balance: 200,
                    })
                );
                assert_eq!(
                    System::events()[1].event,
                    Event::Balances(pallet_balances::Event::Transfer {
                        from: AccountKeyring::Alice.to_account_id(),
                        to: AccountKeyring::Eve.to_account_id(),
                        amount: 200,
                    })
                );
    
                // At next bloc, the new account must be reaped because it's balance is not sufficient
                // to pay the "new account tax"
                run_to_block(3);
                let events = System::events();
                //println!("{:#?}", events);
                assert_eq!(events.len(), 3);
                assert_eq!(
                    System::events()[0].event,
                    Event::Account(pallet_duniter_account::Event::ForceDestroy {
                        who: AccountKeyring::Eve.to_account_id(),
                        balance: 200,
                    })
                );
                assert_eq!(
                    System::events()[1].event,
                    Event::Balances(pallet_balances::Event::Deposit {
                        who: Treasury::account_id(),
                        amount: 200,
                    })
                );
                assert_eq!(
                    System::events()[2].event,
                    Event::Treasury(pallet_treasury::Event::Deposit { value: 200 })
                );
                assert_eq!(
                    Balances::free_balance(AccountKeyring::Eve.to_account_id()),
                    0
                );
                assert_eq!(Balances::free_balance(Treasury::account_id()), 400);
            });
    }
    
    #[test]
    fn test_create_new_account() {
        ExtBuilder::new(1, 3, 4)
            .with_initial_balances(vec![(AccountKeyring::Alice.to_account_id(), 1_000)])
            .build()
            .execute_with(|| {
                run_to_block(2);
    
                // Should be able to transfer 5 units to a new account
                assert_ok!(Balances::transfer(
                    frame_system::RawOrigin::Signed(AccountKeyring::Alice.to_account_id()).into(),
                    MultiAddress::Id(AccountKeyring::Eve.to_account_id()),
                    500
                ));
                let events = System::events();
                //println!("{:#?}", events);
                assert_eq!(events.len(), 2);
                assert_eq!(
                    System::events()[0].event,
                    Event::Balances(pallet_balances::Event::Endowed {
                        account: AccountKeyring::Eve.to_account_id(),
                        free_balance: 500,
                    })
                );
                assert_eq!(
                    System::events()[1].event,
                    Event::Balances(pallet_balances::Event::Transfer {
                        from: AccountKeyring::Alice.to_account_id(),
                        to: AccountKeyring::Eve.to_account_id(),
                        amount: 500,
                    })
                );
    
                // At next bloc, the new account must be created,
                // and new account tax should be collected and deposited in the terasury
                run_to_block(3);
                let events = System::events();
                println!("{:#?}", events);
                assert_eq!(events.len(), 4);
                assert_eq!(
                    System::events()[0].event,
                    Event::System(frame_system::Event::NewAccount {
                        account: AccountKeyring::Eve.to_account_id(),
                    })
                );
                assert_eq!(
                    System::events()[1].event,
                    Event::Balances(pallet_balances::Event::Withdraw {
                        who: AccountKeyring::Eve.to_account_id(),
                        amount: 300,
                    })
                );
                assert_eq!(
                    System::events()[2].event,
                    Event::Balances(pallet_balances::Event::Deposit {
                        who: Treasury::account_id(),
                        amount: 300,
                    })
                );
                assert_eq!(
                    System::events()[3].event,
                    Event::Treasury(pallet_treasury::Event::Deposit { value: 300 })
                );
                assert_eq!(
                    Balances::free_balance(AccountKeyring::Eve.to_account_id()),
                    200
                );
                assert_eq!(Balances::free_balance(Treasury::account_id()), 500);
    
                // A random id request should be registered
                assert_eq!(
                    Account::pending_random_id_assignments(0),
                    Some(AccountKeyring::Eve.to_account_id())
                );
    
                // We can't remove the account until the random id is assigned
                run_to_block(4);
                assert_err!(
                    Balances::transfer(
                        frame_system::RawOrigin::Signed(AccountKeyring::Eve.to_account_id()).into(),
                        MultiAddress::Id(AccountKeyring::Alice.to_account_id()),
                        200
                    ),
                    pallet_balances::Error::<Runtime>::KeepAlive,
                );
                assert_eq!(
                    Balances::free_balance(AccountKeyring::Eve.to_account_id()),
                    200
                );
                assert_ok!(Balances::transfer_all(
                    frame_system::RawOrigin::Signed(AccountKeyring::Eve.to_account_id()).into(),
                    MultiAddress::Id(AccountKeyring::Alice.to_account_id()),
                    false
                ),);
                assert_eq!(
                    Balances::free_balance(AccountKeyring::Eve.to_account_id()),
                    200
                );
            });
    }
    
    #[test]
    fn test_create_new_idty() {
        ExtBuilder::new(1, 3, 4)
            .with_initial_balances(vec![(AccountKeyring::Alice.to_account_id(), 1_000)])
            .build()
            .execute_with(|| {
                run_to_block(2);
    
                // Should be able to create an identity
                assert_ok!(Balances::transfer(
                    frame_system::RawOrigin::Signed(AccountKeyring::Alice.to_account_id()).into(),
                    MultiAddress::Id(AccountKeyring::Eve.to_account_id()),
                    200
                ));
                assert_ok!(Identity::create_identity(
                    frame_system::RawOrigin::Signed(AccountKeyring::Alice.to_account_id()).into(),
                    AccountKeyring::Eve.to_account_id(),
                ));
    
                // At next bloc, nothing should be preleved
                run_to_block(3);
                let events = System::events();
                assert_eq!(events.len(), 0);
    
                // A random id request should be registered
                assert_eq!(
                    Account::pending_random_id_assignments(0),
                    Some(AccountKeyring::Eve.to_account_id())
                );
            });
    }
    
    #[test]
    fn test_oneshot_accounts() {
        ExtBuilder::new(1, 3, 4)
            .with_initial_balances(vec![(AccountKeyring::Alice.to_account_id(), 1_000)])
            .build()
            .execute_with(|| {
                run_to_block(6);
    
                assert_ok!(Account::create_oneshot_account(
                    frame_system::RawOrigin::Signed(AccountKeyring::Alice.to_account_id()).into(),
                    MultiAddress::Id(AccountKeyring::Eve.to_account_id()),
                    400
                ));
                assert_eq!(
                    Balances::free_balance(AccountKeyring::Alice.to_account_id()),
                    600
                );
                run_to_block(7);
    
                assert_ok!(Account::consume_oneshot_account_two_dests(
                    frame_system::RawOrigin::Signed(AccountKeyring::Eve.to_account_id()).into(),
                    0,
                    (
                        true,
                        MultiAddress::Id(AccountKeyring::Ferdie.to_account_id())
                    ),
                    (
                        false,
                        MultiAddress::Id(AccountKeyring::Alice.to_account_id())
                    ),
                    300
                ));
                assert_eq!(
                    Balances::free_balance(AccountKeyring::Alice.to_account_id()),
                    700
                );
                assert_err!(
                    Account::consume_oneshot_account(
                        frame_system::RawOrigin::Signed(AccountKeyring::Eve.to_account_id()).into(),
                        0,
                        (
                            true,
                            MultiAddress::Id(AccountKeyring::Ferdie.to_account_id())
                        ),
                    ),
                    pallet_duniter_account::Error::<Runtime>::OneshotAccountNotExist
                );
                run_to_block(8);
    
                assert_ok!(Account::consume_oneshot_account(
                    frame_system::RawOrigin::Signed(AccountKeyring::Ferdie.to_account_id()).into(),
                    0,
                    (
                        false,
                        MultiAddress::Id(AccountKeyring::Alice.to_account_id())
                    ),
                ));
                assert_eq!(
                    Balances::free_balance(AccountKeyring::Alice.to_account_id()),
                    1000
                );
                assert_err!(
                    Account::consume_oneshot_account(
                        frame_system::RawOrigin::Signed(AccountKeyring::Eve.to_account_id()).into(),
                        0,
                        (
                            false,
                            MultiAddress::Id(AccountKeyring::Alice.to_account_id())
                        ),
                    ),
                    pallet_duniter_account::Error::<Runtime>::OneshotAccountNotExist
                );
            });
    }