difftreelog
test fix build
in: master
9 files changed
pallets/collator-selection/src/mock.rsdiffbeforeafterboth--- a/pallets/collator-selection/src/mock.rs
+++ b/pallets/collator-selection/src/mock.rs
@@ -32,23 +32,22 @@
use frame_support::{
ord_parameter_types, parameter_types,
- traits::{FindAuthor, GenesisBuild, ValidatorRegistration},
+ traits::{ConstU32, FindAuthor, ValidatorRegistration},
PalletId,
};
use frame_system as system;
use frame_system::EnsureSignedBy;
-use sp_core::H256;
+use sp_core::{ConstBool, H256};
use sp_runtime::{
- testing::{Header, UintAuthorityId},
+ testing::UintAuthorityId,
traits::{BlakeTwo256, IdentityLookup, OpaqueKeys},
- Perbill, RuntimeAppPublic,
+ BuildStorage, Perbill, RuntimeAppPublic,
};
use super::*;
use crate as collator_selection;
-type UncheckedExtrinsic = frame_system::mocking::MockUncheckedExtrinsic<Test>;
-type Block = frame_system::mocking::MockBlock<Test>;
+type Block = frame_system::mocking::MockBlockU32<Test>;
// Configure a mock runtime to test the pallet.
frame_support::construct_runtime!(
@@ -64,12 +63,13 @@
);
parameter_types! {
- pub const BlockHashCount: u64 = 250;
+ pub const BlockHashCount: u32 = 250;
pub const SS58Prefix: u8 = 42;
}
impl system::Config for Test {
type BaseCallFilter = frame_support::traits::Everything;
+ type Block = Block;
type BlockWeights = ();
type BlockLength = ();
type DbWeight = ();
@@ -90,7 +90,7 @@
type SystemWeightInfo = ();
type SS58Prefix = SS58Prefix;
type OnSetCode = ();
- type MaxConsumers = frame_support::traits::ConstU32<16>;
+ type MaxConsumers = ConstU32<16>;
}
parameter_types! {
@@ -113,6 +113,7 @@
type FreezeIdentifier = [u8; 16];
type MaxHolds = MaxHolds;
type MaxFreezes = MaxFreezes;
+ type RuntimeHoldReason = RuntimeHoldReason;
}
pub struct Author4;
@@ -145,6 +146,7 @@
type AuthorityId = sp_consensus_aura::sr25519::AuthorityId;
type MaxAuthorities = MaxAuthorities;
type DisabledValidators = ();
+ type AllowMultipleBlocksPerSlot = ConstBool<true>;
}
sp_runtime::impl_opaque_keys! {
@@ -162,27 +164,27 @@
parameter_types! {
pub static SessionHandlerCollators: Vec<u64> = Vec::new();
- pub static SessionChangeBlock: u64 = 0;
+ pub static SessionChangeBlock: u32 = 0;
}
pub struct TestSessionHandler;
impl pallet_session::SessionHandler<u64> for TestSessionHandler {
const KEY_TYPE_IDS: &'static [sp_runtime::KeyTypeId] = &[UintAuthorityId::ID];
fn on_genesis_session<Ks: OpaqueKeys>(keys: &[(u64, Ks)]) {
- SessionHandlerCollators::set(keys.into_iter().map(|(a, _)| *a).collect::<Vec<_>>())
+ SessionHandlerCollators::set(keys.iter().map(|(a, _)| *a).collect::<Vec<_>>())
}
fn on_new_session<Ks: OpaqueKeys>(_: bool, keys: &[(u64, Ks)], _: &[(u64, Ks)]) {
SessionChangeBlock::set(System::block_number());
dbg!(keys.len());
- SessionHandlerCollators::set(keys.into_iter().map(|(a, _)| *a).collect::<Vec<_>>())
+ SessionHandlerCollators::set(keys.iter().map(|(a, _)| *a).collect::<Vec<_>>())
}
fn on_before_session_ending() {}
fn on_disabled(_: u32) {}
}
parameter_types! {
- pub const Offset: u64 = 0;
- pub const Period: u64 = 10;
+ pub const Offset: u32 = 0;
+ pub const Period: u32 = 10;
}
impl pallet_session::Config for Test {
@@ -201,7 +203,7 @@
parameter_types! {
pub const MaxCollators: u32 = 5;
pub const LicenseBond: u64 = 10;
- pub const KickThreshold: u64 = 10;
+ pub const KickThreshold: u32 = 10;
// the following values do not matter and are meaningless, etc.
pub const DefaultWeightToFeeCoefficient: u64 = 100_000;
pub const DefaultMinGasPrice: u64 = 100_000;
@@ -230,6 +232,7 @@
impl Config for Test {
type RuntimeEvent = RuntimeEvent;
+ type RuntimeHoldReason = RuntimeHoldReason;
type UpdateOrigin = EnsureSignedBy<RootAccount, u64>;
type PotId = PotId;
type MaxCollators = MaxCollators;
@@ -238,7 +241,6 @@
type ValidatorId = <Self as frame_system::Config>::AccountId;
type ValidatorIdOf = IdentityCollator;
type ValidatorRegistration = IsRegistered;
- type LicenceBondIdentifier = LicenceBondIdentifier;
type Currency = Balances;
type DesiredCollators = MaxCollators;
type LicenseBond = LicenseBond;
@@ -248,8 +250,8 @@
pub fn new_test_ext() -> sp_io::TestExternalities {
sp_tracing::try_init_simple();
- let mut t = frame_system::GenesisConfig::default()
- .build_storage::<Test>()
+ let mut t = <frame_system::GenesisConfig<Test>>::default()
+ .build_storage()
.unwrap();
let invulnerables = vec![1, 2];
@@ -284,9 +286,9 @@
t.into()
}
-pub fn initialize_to_block(n: u64) {
+pub fn initialize_to_block(n: u32) {
for i in System::block_number() + 1..=n {
System::set_block_number(i);
- <AllPalletsWithSystem as frame_support::traits::OnInitialize<u64>>::on_initialize(i);
+ <AllPalletsWithSystem as frame_support::traits::OnInitialize<u32>>::on_initialize(i);
}
}
pallets/collator-selection/src/tests.rsdiffbeforeafterboth1// Copyright 2019-2022 Unique Network (Gibraltar) Ltd.2// This file is part of Unique Network.34// Unique Network is free software: you can redistribute it and/or modify5// it under the terms of the GNU General Public License as published by6// the Free Software Foundation, either version 3 of the License, or7// (at your option) any later version.89// Unique Network is distributed in the hope that it will be useful,10// but WITHOUT ANY WARRANTY; without even the implied warranty of11// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the12// GNU General Public License for more details.1314// You should have received a copy of the GNU General Public License15// along with Unique Network. If not, see <http://www.gnu.org/licenses/>.1617// Original license:18// Copyright (C) 2021 Parity Technologies (UK) Ltd.19// SPDX-License-Identifier: Apache-2.02021// Licensed under the Apache License, Version 2.0 (the "License");22// you may not use this file except in compliance with the License.23// You may obtain a copy of the License at24//25// http://www.apache.org/licenses/LICENSE-2.026//27// Unless required by applicable law or agreed to in writing, software28// distributed under the License is distributed on an "AS IS" BASIS,29// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.30// See the License for the specific language governing permissions and31// limitations under the License.3233use frame_support::{34 assert_noop, assert_ok,35 traits::{fungible, GenesisBuild, OnInitialize},36};37use scale_info::prelude::*;38use sp_runtime::{traits::BadOrigin, TokenError};3940use crate::{self as collator_selection, mock::*, Config, Error};4142fn get_license_and_onboard(account_id: <Test as frame_system::Config>::AccountId) {43 assert_ok!(CollatorSelection::get_license(RuntimeOrigin::signed(44 account_id45 )));46 assert_ok!(CollatorSelection::onboard(RuntimeOrigin::signed(47 account_id48 )));49}5051#[test]52fn basic_setup_works() {53 new_test_ext().execute_with(|| {54 assert!(CollatorSelection::candidates().is_empty());55 assert_eq!(CollatorSelection::invulnerables(), vec![1, 2]);56 });57}5859#[test]60fn it_should_add_invulnerables() {61 new_test_ext().execute_with(|| {62 assert_ok!(CollatorSelection::add_invulnerable(63 RuntimeOrigin::signed(RootAccount::get()),64 165 ));66 assert_ok!(CollatorSelection::add_invulnerable(67 RuntimeOrigin::signed(RootAccount::get()),68 269 ));70 assert_eq!(CollatorSelection::invulnerables(), vec![1, 2]);7172 // cannot set with non-root.73 assert_noop!(74 CollatorSelection::add_invulnerable(RuntimeOrigin::signed(1), 3),75 BadOrigin76 );7778 // cannot set invulnerables without associated validator keys79 assert_noop!(80 CollatorSelection::add_invulnerable(RuntimeOrigin::signed(RootAccount::get()), 7),81 Error::<Test>::ValidatorNotRegistered82 );83 });84}8586#[test]87fn it_should_remove_invulnerables() {88 new_test_ext().execute_with(|| {89 assert_ok!(CollatorSelection::add_invulnerable(90 RuntimeOrigin::signed(RootAccount::get()),91 192 ));93 assert_ok!(CollatorSelection::add_invulnerable(94 RuntimeOrigin::signed(RootAccount::get()),95 296 ));9798 // cannot remove with non-root.99 assert_noop!(100 CollatorSelection::remove_invulnerable(RuntimeOrigin::signed(1), 3),101 BadOrigin102 );103104 assert_ok!(CollatorSelection::remove_invulnerable(105 RuntimeOrigin::signed(RootAccount::get()),106 2107 ));108 assert_eq!(CollatorSelection::invulnerables(), vec![1]);109110 // cannot remove an invulnerable if there would be 0 invulnerables.111 assert_noop!(112 CollatorSelection::remove_invulnerable(RuntimeOrigin::signed(RootAccount::get()), 1),113 Error::<Test>::TooFewInvulnerables114 );115 });116}117118#[test]119fn cannot_onboard_candidate_with_no_license() {120 new_test_ext().execute_with(|| {121 // can't onboard a candidate who did not get a license.122 assert_noop!(123 CollatorSelection::onboard(RuntimeOrigin::signed(3)),124 Error::<Test>::NoLicense,125 );126127 // but give it a license and welcome aboard.128 assert_ok!(CollatorSelection::get_license(RuntimeOrigin::signed(3)));129 assert_ok!(CollatorSelection::onboard(RuntimeOrigin::signed(3)));130 })131}132133#[test]134fn cannot_onboard_candidate_if_too_many() {135 new_test_ext().execute_with(|| {136 // can accept desired value of collators.137 for c in 3u64..=(<Test as Config>::DesiredCollators::get()).into() {138 assert_ok!(CollatorSelection::get_license(RuntimeOrigin::signed(c)));139 assert_ok!(CollatorSelection::onboard(RuntimeOrigin::signed(c)));140 }141142 // but no more.143 let undesired_collator = (<Test as Config>::DesiredCollators::get() + 1) as u64;144 assert_ok!(CollatorSelection::get_license(RuntimeOrigin::signed(145 undesired_collator146 )));147 assert_noop!(148 CollatorSelection::onboard(RuntimeOrigin::signed(undesired_collator)),149 Error::<Test>::TooManyCandidates,150 );151 })152}153154#[test]155fn cannot_obtain_license_if_keys_not_registered() {156 new_test_ext().execute_with(|| {157 // can't 7 because keys not registered.158 assert_noop!(159 CollatorSelection::get_license(RuntimeOrigin::signed(7)),160 Error::<Test>::ValidatorNotRegistered161 );162 })163}164165#[test]166fn cannot_obtain_license_if_poor() {167 new_test_ext().execute_with(|| {168 let ed = <Test as pallet_balances::Config>::ExistentialDeposit::get();169 assert_eq!(Balances::free_balance(3), 100);170 assert_eq!(Balances::free_balance(33), ed);171172 // works173 assert_ok!(CollatorSelection::get_license(RuntimeOrigin::signed(3)));174175 // poor176 assert_noop!(177 CollatorSelection::get_license(RuntimeOrigin::signed(33)),178 TokenError::FundsUnavailable,179 );180 });181}182183#[test]184fn cannot_onboard_dupe_candidate() {185 new_test_ext().execute_with(|| {186 // can add 3 as candidate187 get_license_and_onboard(3);188 assert_eq!(CollatorSelection::license_deposit_of(3), 10);189 assert_eq!(CollatorSelection::candidates(), vec![3]);190 assert_eq!(CollatorSelection::last_authored_block(3), 10);191 assert_eq!(Balances::free_balance(3), 90);192193 // but no more194 assert_noop!(195 CollatorSelection::get_license(RuntimeOrigin::signed(3)),196 Error::<Test>::AlreadyHoldingLicense,197 );198 assert_noop!(199 CollatorSelection::onboard(RuntimeOrigin::signed(3)),200 Error::<Test>::AlreadyCandidate,201 );202 })203}204205#[test]206fn becoming_candidate_works() {207 new_test_ext().execute_with(|| {208 assert_eq!(CollatorSelection::candidates(), Vec::new());209 assert_eq!(CollatorSelection::invulnerables(), vec![1, 2]);210211 // take two endowed, non-invulnerables accounts.212 assert_eq!(Balances::free_balance(3), 100);213 assert_eq!(Balances::free_balance(4), 100);214215 get_license_and_onboard(3);216 get_license_and_onboard(4);217218 assert_eq!(Balances::free_balance(3), 90);219 assert_eq!(Balances::free_balance(4), 90);220221 assert_eq!(CollatorSelection::candidates().len(), 2);222 });223}224225#[test]226fn cannot_become_candidate_if_invulnerable() {227 new_test_ext().execute_with(|| {228 assert_eq!(CollatorSelection::invulnerables(), vec![1, 2]);229230 // can obtain a license even if is invulnerable.231 assert_ok!(CollatorSelection::get_license(RuntimeOrigin::signed(1)));232 // but cannot onboard233 assert_noop!(234 CollatorSelection::onboard(RuntimeOrigin::signed(1)),235 Error::<Test>::AlreadyInvulnerable,236 );237238 // get a license and then become invulnerable.239 assert_ok!(CollatorSelection::get_license(RuntimeOrigin::signed(3)));240 assert_ok!(CollatorSelection::add_invulnerable(241 RuntimeOrigin::signed(RootAccount::get()),242 3243 ));244 assert_noop!(245 CollatorSelection::onboard(RuntimeOrigin::signed(3)),246 Error::<Test>::AlreadyInvulnerable,247 );248 })249}250251#[test]252fn can_become_invulnerable_if_candidate() {253 new_test_ext().execute_with(|| {254 // become a candidate and then become invulnerable.255 get_license_and_onboard(3);256 assert_eq!(CollatorSelection::candidates(), vec![3]);257258 assert_ok!(CollatorSelection::add_invulnerable(259 RuntimeOrigin::signed(RootAccount::get()),260 3261 ));262 // should exclude from candidates, but not revoke the license263 assert_eq!(CollatorSelection::candidates(), vec![]);264 assert_eq!(CollatorSelection::license_deposit_of(3), 10);265 assert_eq!(Balances::free_balance(3), 90);266 });267}268269#[test]270fn offboard() {271 new_test_ext().execute_with(|| {272 // register a candidate.273 get_license_and_onboard(3);274 assert_eq!(Balances::free_balance(3), 90);275276 // cannot leave if holds license but not yet candidate.277 assert_ok!(CollatorSelection::get_license(RuntimeOrigin::signed(4)));278 assert_noop!(279 CollatorSelection::offboard(RuntimeOrigin::signed(4)),280 Error::<Test>::NotCandidate281 );282 // cannot leave if does not hold license.283 assert_noop!(284 CollatorSelection::offboard(RuntimeOrigin::signed(5)),285 Error::<Test>::NotCandidate286 );287288 // bond is returned - only after releasing the license289 assert_ok!(CollatorSelection::offboard(RuntimeOrigin::signed(3)));290 assert_eq!(Balances::free_balance(3), 90);291 assert_eq!(CollatorSelection::last_authored_block(3), 0);292 assert_ok!(CollatorSelection::release_license(RuntimeOrigin::signed(3)));293 assert_eq!(Balances::free_balance(3), 100);294 });295}296297#[test]298fn release_license() {299 new_test_ext().execute_with(|| {300 // obtain a license to collate and reserve the bond.301 assert_ok!(CollatorSelection::get_license(RuntimeOrigin::signed(3)));302 assert_eq!(Balances::free_balance(3), 90);303304 // release the license and get the bond back.305 assert_ok!(CollatorSelection::release_license(RuntimeOrigin::signed(3)));306 assert_eq!(Balances::free_balance(3), 100);307308 // register a candidate.309 get_license_and_onboard(3);310 assert_eq!(Balances::free_balance(3), 90);311312 // can release license even if onboarded.313 assert_ok!(CollatorSelection::release_license(RuntimeOrigin::signed(3)));314 assert_eq!(Balances::free_balance(3), 100);315 assert_eq!(CollatorSelection::candidates(), vec![]);316 });317}318319#[test]320fn force_release_license() {321 new_test_ext().execute_with(|| {322 // obtain a license to collate and reserve the bond.323 assert_ok!(CollatorSelection::get_license(RuntimeOrigin::signed(3)));324 assert_eq!(Balances::free_balance(3), 90);325326 // cannot execute the operation as non-root327 assert_noop!(328 CollatorSelection::force_release_license(RuntimeOrigin::signed(3), 3),329 BadOrigin330 );331332 // release the license and get the bond back.333 assert_ok!(CollatorSelection::force_release_license(334 RuntimeOrigin::signed(RootAccount::get()),335 3336 ));337 assert_eq!(Balances::free_balance(3), 100);338339 // register a candidate.340 get_license_and_onboard(3);341 assert_eq!(Balances::free_balance(3), 90);342343 // can release license even if onboarded.344 assert_ok!(CollatorSelection::force_release_license(345 RuntimeOrigin::signed(RootAccount::get()),346 3347 ));348 assert_eq!(Balances::free_balance(3), 100);349 assert_eq!(CollatorSelection::candidates(), vec![]);350 });351}352353#[test]354fn authorship_event_handler() {355 new_test_ext().execute_with(|| {356 // put 100 in the pot + 5 for ED357 <Balances as fungible::Mutate<_>>::set_balance(&CollatorSelection::account_id(), 105);358359 // 4 is the default author.360 assert_eq!(Balances::free_balance(4), 100);361 get_license_and_onboard(4);362 // triggers `note_author`363 Authorship::on_initialize(1);364365 assert_eq!(CollatorSelection::candidates(), vec![4]);366 assert_eq!(CollatorSelection::last_authored_block(4), 0);367368 // half of the pot goes to the collator who's the author (4 in tests).369 assert_eq!(Balances::free_balance(4), 140);370 // half + ED stays.371 assert_eq!(Balances::free_balance(CollatorSelection::account_id()), 55);372 });373}374375#[test]376fn fees_edgecases() {377 new_test_ext().execute_with(|| {378 // Nothing panics, no reward when no ED in balance379 Authorship::on_initialize(1);380 // put some money into the pot at ED381 <Balances as fungible::Mutate<_>>::set_balance(&CollatorSelection::account_id(), 5);382 // 4 is the default author.383 assert_eq!(Balances::free_balance(4), 100);384 get_license_and_onboard(4);385 // triggers `note_author`386 Authorship::on_initialize(1);387388 assert_eq!(CollatorSelection::candidates(), vec![4]);389 assert_eq!(CollatorSelection::last_authored_block(4), 0);390 // Nothing received391 assert_eq!(Balances::free_balance(4), 90);392 // all fee stays393 assert_eq!(Balances::free_balance(CollatorSelection::account_id()), 5);394 });395}396397#[test]398fn session_management_works() {399 new_test_ext().execute_with(|| {400 initialize_to_block(1);401402 assert_eq!(SessionChangeBlock::get(), 0);403 assert_eq!(SessionHandlerCollators::get(), vec![1, 2]);404405 initialize_to_block(4);406407 assert_eq!(SessionChangeBlock::get(), 0);408 assert_eq!(SessionHandlerCollators::get(), vec![1, 2]);409410 // add a new collator411 get_license_and_onboard(5);412413 // session won't see this.414 assert_eq!(SessionHandlerCollators::get(), vec![1, 2]);415 // but we have a new candidate.416 assert_eq!(CollatorSelection::candidates().len(), 1);417418 initialize_to_block(10);419 assert_eq!(SessionChangeBlock::get(), 10);420 // pallet-session has 1 session delay; current validators are the same.421 assert_eq!(Session::validators(), vec![1, 2]);422 // queued ones are changed, and now we have 3.423 assert_eq!(Session::queued_keys().len(), 3);424 // session handlers (aura, et. al.) cannot see this yet.425 assert_eq!(SessionHandlerCollators::get(), vec![1, 2]);426427 initialize_to_block(20);428 assert_eq!(SessionChangeBlock::get(), 20);429 // changed are now reflected to session handlers.430 assert_eq!(SessionHandlerCollators::get(), vec![1, 2, 5]);431 });432}433434#[test]435fn kick_mechanism() {436 new_test_ext().execute_with(|| {437 // add a new collator438 get_license_and_onboard(3);439 get_license_and_onboard(4);440441 initialize_to_block(10);442 assert_eq!(CollatorSelection::candidates().len(), 2);443444 initialize_to_block(20);445 assert_eq!(SessionChangeBlock::get(), 20);446 // 4 authored this block, gets to stay 3 was kicked447 assert_eq!(CollatorSelection::candidates().len(), 1);448 // 3 will be kicked after 1 session delay449 assert_eq!(SessionHandlerCollators::get(), vec![1, 2, 3, 4]);450451 assert_eq!(CollatorSelection::candidates(), vec![4]);452 // assert_eq!(<KickThreshold<Test>>::get(), 10);453 assert_eq!(CollatorSelection::last_authored_block(4), 20);454455 initialize_to_block(30);456 // 3 gets kicked after 1 session delay457 assert_eq!(SessionHandlerCollators::get(), vec![1, 2, 4]);458 // kicked collator gets their funds slashed, the deposit going to treasury459 assert_eq!(Balances::free_balance(3), 90);460 });461}462463#[test]464#[should_panic = "duplicate invulnerables in genesis."]465fn cannot_set_genesis_value_twice() {466 sp_tracing::try_init_simple();467 let mut t = frame_system::GenesisConfig::default()468 .build_storage::<Test>()469 .unwrap();470 let invulnerables = vec![1, 1];471472 let collator_selection = collator_selection::GenesisConfig::<Test> { invulnerables };473 // collator selection must be initialized before session.474 collator_selection.assimilate_storage(&mut t).unwrap();475}pallets/identity/src/tests.rsdiffbeforeafterboth--- a/pallets/identity/src/tests.rs
+++ b/pallets/identity/src/tests.rs
@@ -43,7 +43,6 @@
use parity_scale_codec::{Decode, Encode};
use sp_core::H256;
use sp_runtime::{
- testing::Header,
traits::{BadOrigin, BlakeTwo256, IdentityLookup},
BuildStorage,
};
@@ -51,8 +50,7 @@
use super::*;
use crate as pallet_identity;
-type UncheckedExtrinsic = frame_system::mocking::MockUncheckedExtrinsic<Test>;
-type Block = frame_system::mocking::MockBlock<Test>;
+type Block = frame_system::mocking::MockBlockU32<Test>;
frame_support::construct_runtime!(
pub enum Test {
@@ -79,7 +77,7 @@
type AccountId = u64;
type Lookup = IdentityLookup<Self::AccountId>;
type RuntimeEvent = RuntimeEvent;
- type BlockHashCount = ConstU64<250>;
+ type BlockHashCount = ConstU32<250>;
type DbWeight = ();
type Version = ();
type PalletInfo = PalletInfo;
pallets/inflation/src/tests.rsdiffbeforeafterboth--- a/pallets/inflation/src/tests.rs
+++ b/pallets/inflation/src/tests.rs
@@ -34,12 +34,12 @@
use crate as pallet_inflation;
-type UncheckedExtrinsic = frame_system::mocking::MockUncheckedExtrinsic<Test>;
-type Block = frame_system::mocking::MockBlock<Test>;
+type Block = frame_system::mocking::MockBlockU32<Test>;
-const YEAR: u64 = 5_259_600; // 6-second blocks
- // const YEAR: u64 = 2_629_800; // 12-second blocks
- // Expected 100-block inflation for year 1 is 100 * 100_000_000 / YEAR = FIRST_YEAR_BLOCK_INFLATION
+// 6-second blocks
+// const YEAR: u32 = 2_629_800; // 12-second blocks
+// Expected 100-block inflation for year 1 is 100 * 100_000_000 / YEAR = FIRST_YEAR_BLOCK_INFLATION
+const YEAR: u32 = 5_259_600;
const FIRST_YEAR_BLOCK_INFLATION: u64 = 1901;
parameter_types! {
@@ -60,6 +60,7 @@
type FreezeIdentifier = ();
type MaxHolds = ();
type MaxFreezes = ();
+ type RuntimeHoldReason = RuntimeHoldReason;
}
frame_support::construct_runtime!(
@@ -71,7 +72,7 @@
);
parameter_types! {
- pub const BlockHashCount: u64 = 250;
+ pub const BlockHashCount: u32 = 250;
pub BlockWeights: frame_system::limits::BlockWeights =
frame_system::limits::BlockWeights::simple_max(Weight::from_parts(1024, 0));
pub const SS58Prefix: u8 = 42;
@@ -79,6 +80,7 @@
impl frame_system::Config for Test {
type BaseCallFilter = Everything;
+ type Block = Block;
type BlockWeights = ();
type BlockLength = ();
type DbWeight = ();
@@ -187,7 +189,7 @@
fn inflation_second_deposit() {
new_test_ext().execute_with(|| {
// Total issuance = 1_000_000_000
- let initial_issuance: u64 = 1_000_000_000;
+ let initial_issuance = 1_000_000_000;
let _ = <Balances as Balanced<_>>::deposit(&1234, initial_issuance, Precision::Exact);
assert_eq!(Balances::free_balance(1234), initial_issuance);
MockBlockNumberProvider::set(1);
@@ -196,20 +198,20 @@
assert_ok!(Inflation::start_inflation(RawOrigin::Root.into(), 1));
// Next inflation deposit happens when block is greater then or equal to NextInflationBlock
- let mut block: u64 = 2;
- let balance_before: u64 = Balances::free_balance(1234);
+ let mut block = 2;
+ let balance_before = Balances::free_balance(1234);
while block < <pallet_inflation::NextInflationBlock<Test>>::get() {
- MockBlockNumberProvider::set(block as u64);
+ MockBlockNumberProvider::set(block);
Inflation::on_initialize(0);
block += 1;
}
- let balance_just_before: u64 = Balances::free_balance(1234);
+ let balance_just_before = Balances::free_balance(1234);
assert_eq!(balance_before, balance_just_before);
// The block with inflation
- MockBlockNumberProvider::set(block as u64);
+ MockBlockNumberProvider::set(block);
Inflation::on_initialize(0);
- let balance_after: u64 = Balances::free_balance(1234);
+ let balance_after = Balances::free_balance(1234);
assert_eq!(balance_after - balance_just_before, block_inflation!());
});
}
@@ -234,7 +236,7 @@
Inflation::on_initialize(0);
}
assert_eq!(
- initial_issuance + (FIRST_YEAR_BLOCK_INFLATION * (YEAR / 100)),
+ initial_issuance + (FIRST_YEAR_BLOCK_INFLATION * ((YEAR as u64) / 100)),
<Balances as Inspect<_>>::total_issuance()
);
@@ -243,8 +245,8 @@
let block_inflation_year_2 = block_inflation!();
// Expected 100-block inflation for year 2: 100 * 9.33% * initial issuance * 110% / YEAR == 1951
let expecter_year_2_inflation: u64 = (initial_issuance
- + FIRST_YEAR_BLOCK_INFLATION * YEAR / 100)
- * 933 * 100 / (10000 * YEAR);
+ + FIRST_YEAR_BLOCK_INFLATION * (YEAR as u64) / 100)
+ * 933 * 100 / (10000 * (YEAR as u64));
assert_eq!(block_inflation_year_2 / 10, expecter_year_2_inflation / 10); // divide by 10 for approx. equality
});
}
@@ -253,8 +255,8 @@
fn inflation_start_large_kusama_block() {
new_test_ext().execute_with(|| {
// Total issuance = 1_000_000_000
- let initial_issuance: u64 = 1_000_000_000;
- let start_block: u64 = 10457457;
+ let initial_issuance = 1_000_000_000;
+ let start_block = 10457457;
let _ = <Balances as Balanced<_>>::deposit(&1234, initial_issuance, Precision::Exact);
assert_eq!(Balances::free_balance(1234), initial_issuance);
MockBlockNumberProvider::set(start_block);
@@ -273,7 +275,7 @@
Inflation::on_initialize(0);
}
assert_eq!(
- initial_issuance + (FIRST_YEAR_BLOCK_INFLATION * (YEAR / 100)),
+ initial_issuance + (FIRST_YEAR_BLOCK_INFLATION * ((YEAR as u64) / 100)),
<Balances as Inspect<_>>::total_issuance()
);
@@ -282,8 +284,8 @@
let block_inflation_year_2 = block_inflation!();
// Expected 100-block inflation for year 2: 100 * 9.33% * initial issuance * 110% / YEAR == 1951
let expecter_year_2_inflation: u64 = (initial_issuance
- + FIRST_YEAR_BLOCK_INFLATION * YEAR / 100)
- * 933 * 100 / (10000 * YEAR);
+ + FIRST_YEAR_BLOCK_INFLATION * (YEAR as u64) / 100)
+ * 933 * 100 / (10000 * (YEAR as u64));
assert_eq!(block_inflation_year_2 / 10, expecter_year_2_inflation / 10); // divide by 10 for approx. equality
});
}
@@ -320,14 +322,14 @@
#[test]
fn inflation_rate_by_year() {
new_test_ext().execute_with(|| {
- let payouts: u64 = YEAR / InflationBlockInterval::get() as u64;
+ let payouts = (YEAR / InflationBlockInterval::get()) as u64;
// Inflation starts at 10% and does down by 2/3% every year until year 9 (included),
// then it is flat.
let payout_by_year: [u64; 11] = [1000, 933, 867, 800, 733, 667, 600, 533, 467, 400, 400];
// For accuracy total issuance = payout0 * payouts * 10;
- let initial_issuance: u64 = payout_by_year[0] * payouts * 10;
+ let initial_issuance = payout_by_year[0] * payouts * 10;
let _ = <Balances as Balanced<_>>::deposit(&1234, initial_issuance, Precision::Exact);
assert_eq!(Balances::free_balance(1234), initial_issuance);
runtime/common/tests/mod.rsdiffbeforeafterboth--- a/runtime/common/tests/mod.rs
+++ b/runtime/common/tests/mod.rs
@@ -19,7 +19,7 @@
use sp_runtime::{BuildStorage, Storage};
use up_common::types::AuraId;
-use crate::{BuildGenesisConfig, ParachainInfoConfig, Runtime, RuntimeEvent, System};
+use crate::{ParachainInfoConfig, Runtime, RuntimeEvent, RuntimeGenesisConfig, System};
pub type Balance = u128;
pub mod xcm;
@@ -51,8 +51,8 @@
fn new_test_ext(balances: Vec<(AccountId, Balance)>) -> sp_io::TestExternalities {
let mut storage = make_basic_storage();
- pallet_balances::BuildGenesisConfig::<Runtime> { balances }
- .build_storage(&mut storage)
+ pallet_balances::GenesisConfig::<Runtime> { balances }
+ .assimilate_storage(&mut storage)
.unwrap();
let mut ext = sp_io::TestExternalities::new(storage);
@@ -95,7 +95,7 @@
.map(|acc| get_account_id_from_seed::<sr25519::Public>(acc))
.collect::<Vec<_>>();
- let cfg = BuildGenesisConfig {
+ let cfg = RuntimeGenesisConfig {
collator_selection: CollatorSelectionConfig { invulnerables },
session: SessionConfig { keys },
parachain_info: ParachainInfoConfig {
@@ -112,7 +112,7 @@
fn make_basic_storage() -> Storage {
use crate::AuraConfig;
- let cfg = BuildGenesisConfig {
+ let cfg = RuntimeGenesisConfig {
aura: AuraConfig {
authorities: vec![
get_from_seed::<AuraId>("Alice"),
runtime/common/tests/xcm.rsdiffbeforeafterboth--- a/runtime/common/tests/xcm.rs
+++ b/runtime/common/tests/xcm.rs
@@ -52,9 +52,9 @@
let xcm_event = &last_events(1)[0];
match xcm_event {
- RuntimeEvent::PolkadotXcm(pallet_xcm::Event::<Runtime>::Attempted(
- Outcome::Incomplete(_weight, Error::NoPermission),
- )) => { /* Pass */ }
+ RuntimeEvent::PolkadotXcm(pallet_xcm::Event::<Runtime>::Attempted {
+ outcome: Outcome::Incomplete(_weight, Error::NoPermission),
+ }) => { /* Pass */ }
_ => panic!(
"Expected PolkadotXcm.Attempted(Incomplete(_weight, NoPermission)),\
found: {xcm_event:#?}"
runtime/tests/src/dispatch.rsdiffbeforeafterboth--- /dev/null
+++ b/runtime/tests/src/dispatch.rs
@@ -0,0 +1 @@
+../../common/dispatch.rs
\ No newline at end of file
runtime/tests/src/lib.rsdiffbeforeafterboth--- a/runtime/tests/src/lib.rs
+++ b/runtime/tests/src/lib.rs
@@ -33,23 +33,20 @@
use scale_info::TypeInfo;
use sp_core::{H160, H256, U256};
use sp_runtime::{
- testing::Header,
traits::{BlakeTwo256, IdentityLookup},
+ BuildStorage,
};
use up_data_structs::mapping::{CrossTokenAddressMapping, EvmTokenAddressMapping};
-#[path = "../../common/dispatch.rs"]
mod dispatch;
use dispatch::CollectionDispatchT;
-#[path = "../../common/weights/mod.rs"]
mod weights;
use weights::CommonWeights;
-type UncheckedExtrinsic = frame_system::mocking::MockUncheckedExtrinsic<Test>;
-type Block = frame_system::mocking::MockBlock<Test>;
+type Block = frame_system::mocking::MockBlockU32<Test>;
#[cfg(test)]
mod tests;
@@ -73,13 +70,14 @@
);
parameter_types! {
- pub const BlockHashCount: u64 = 250;
+ pub const BlockHashCount: u32 = 250;
pub const SS58Prefix: u8 = 42;
}
impl system::Config for Test {
type RuntimeEvent = RuntimeEvent;
type BaseCallFilter = Everything;
+ type Block = Block;
type BlockWeights = ();
type BlockLength = ();
type DbWeight = ();
@@ -120,6 +118,7 @@
type MaxFreezes = MaxLocks;
type FreezeIdentifier = [u8; 8];
type MaxHolds = MaxLocks;
+ type RuntimeHoldReason = RuntimeHoldReason;
}
parameter_types! {
@@ -232,6 +231,7 @@
type OnMethodCall = ();
type OnCreate = ();
type OnChargeTransaction = ();
+ type OnCheckEvmTransaction = ();
type FindAuthor = ();
type BlockHashMapping = SubstrateBlockHashMapping<Self>;
type Timestamp = Timestamp;
@@ -296,8 +296,8 @@
// Build genesis storage according to the mock runtime.
pub fn new_test_ext() -> sp_io::TestExternalities {
- system::GenesisConfig::default()
- .build_storage::<Test>()
+ <system::GenesisConfig<Test>>::default()
+ .build_storage()
.unwrap()
.into()
}
runtime/tests/src/weightsdiffbeforeafterboth--- /dev/null
+++ b/runtime/tests/src/weights
@@ -0,0 +1 @@
+../../common/weights
\ No newline at end of file