update: sync tests and runtime primitives

node/cli/src/chain_spec.rs

@@ -25,9 +25,9 @@ use node_runtime::constants::currency::*;
 use node_runtime::Block;
 pub use node_runtime::GenesisConfig;
 use node_runtime::{
-	AuthorityDiscoveryConfig, BabeConfig, BalancesConfig, ContractsConfig, GrandpaConfig, ImOnlineConfig,
-	IndicesConfig, KtonConfig, SessionConfig, SessionKeys, StakerStatus, StakingConfig, SudoConfig, SystemConfig, COIN,
-	WASM_BINARY,
+	constants::currency::MILLICENTS, AuthorityDiscoveryConfig, BabeConfig, BalancesConfig, ContractsConfig,
+	GrandpaConfig, ImOnlineConfig, IndicesConfig, KtonConfig, SessionConfig, SessionKeys, StakerStatus, StakingConfig,
+	SudoConfig, SystemConfig, WASM_BINARY,
 };
 use primitives::{crypto::UncheckedInto, Pair, Public};
 use serde::{Deserialize, Serialize};
@@ -249,7 +249,7 @@ pub fn testnet_genesis(
 		}),
 		staking: Some(StakingConfig {
 			current_era: 0,
-			//			current_era_total_reward: 80_000_000 * COIN / 63720,
+			//			current_era_total_reward: 80_000_000 * MILLICENTS / 63720,
 			//			offline_slash: Perbill::from_parts(1_000_000),
 			session_reward: Perbill::from_percent(90),
 			validator_count: 7,
@@ -350,8 +350,8 @@ pub fn darwinia_genesis_verbose(
 		]
 	});
 
-	const ENDOWMENT: Balance = 100_000_000 * COIN;
-	const STASH: Balance = 100 * COIN;
+	const ENDOWMENT: Balance = 100_000_000 * MILLICENTS;
+	const STASH: Balance = 100 * MILLICENTS;
 
 	GenesisConfig {
 		system: Some(SystemConfig {
@@ -404,7 +404,7 @@ pub fn darwinia_genesis_verbose(
 		}),
 		staking: Some(StakingConfig {
 			current_era: 0,
-			//			current_era_total_reward: 80_000_000 * COIN / 63720,
+			//			current_era_total_reward: 80_000_000 * MILLICENTS / 63720,
 			//			offline_slash: Perbill::from_parts(1_000_000),
 			session_reward: Perbill::from_percent(90),
 			validator_count: 7,

node/runtime/src/constants.rs

@@ -20,7 +20,10 @@
 pub mod currency {
 	use node_primitives::Balance;
 
-	pub const MILLICENTS: Balance = 1_000_000_000;
+	pub const NANO: Balance = 1;
+	pub const MICRO: Balance = 1_000 * NANO;
+	pub const MILLI: Balance = 1_000 * MICRO;
+	pub const MILLICENTS: Balance = 1_000 * MILLI;
 	pub const CENTS: Balance = 1_000 * MILLICENTS; // assume this is worth about a cent.
 	pub const DOLLARS: Balance = 100 * CENTS;
 }

node/runtime/src/lib.rs

@@ -19,60 +19,52 @@
 #![cfg_attr(not(feature = "std"), no_std)]
 #![recursion_limit = "256"]
 
-use authority_discovery_primitives::{AuthorityId as EncodedAuthorityId, Signature as EncodedSignature};
-use babe_primitives::{AuthorityId as BabeId, AuthoritySignature as BabeSignature};
-pub use balances::Call as BalancesCall;
-use codec::{Decode, Encode};
 pub use contracts::Gas;
-use sr_api::impl_runtime_apis;
+pub use timestamp::Call as TimestampCall;
 
-//use grandpa::fg_primitives;
-//use grandpa::{AuthorityId as GrandpaId, AuthorityWeight as GrandpaWeight};
-//use im_online::sr25519::AuthorityId as ImOnlineId;
+pub use balances::Call as BalancesCall;
+pub use staking::StakerStatus;
 
+use authority_discovery_primitives::{AuthorityId as EncodedAuthorityId, Signature as EncodedSignature};
+use babe_primitives::{AuthorityId as BabeId, AuthoritySignature as BabeSignature};
+use codec::{Decode, Encode};
+use grandpa::{fg_primitives, AuthorityList as GrandpaAuthorityList};
+use im_online::sr25519::AuthorityId as ImOnlineId;
 use node_primitives::{AccountId, AccountIndex, Balance, BlockNumber, Hash, Index, Moment, Signature};
 use rstd::prelude::*;
-use sr_primitives::traits::{
-	self, BlakeTwo256, Block as BlockT, NumberFor, OpaqueKeys, SaturatedConversion, StaticLookup,
+use sr_api::impl_runtime_apis;
+use sr_primitives::{
+	create_runtime_str, generic, impl_opaque_keys,
+	traits::{self, BlakeTwo256, Block as BlockT, NumberFor, OpaqueKeys, SaturatedConversion, StaticLookup},
+	transaction_validity::TransactionValidity,
+	weights::Weight,
+	ApplyResult, Perbill,
 };
-use sr_primitives::transaction_validity::TransactionValidity;
-use sr_primitives::weights::Weight;
-#[cfg(any(feature = "std", test))]
-pub use sr_primitives::BuildStorage;
-
-use sr_primitives::{create_runtime_str, generic, impl_opaque_keys, ApplyResult, Perbill};
 use substrate_primitives::u32_trait::{_1, _4};
-
-use support::traits::OnUnbalanced;
-pub use support::StorageValue;
+use substrate_primitives::OpaqueMetadata;
 use support::{
 	construct_runtime, parameter_types,
-	traits::{Currency, Randomness, SplitTwoWays},
+	traits::{Currency, OnUnbalanced, Randomness, SplitTwoWays},
 };
-
-pub use timestamp::Call as TimestampCall;
+use system::offchain::TransactionSubmitter;
+use transaction_payment_rpc_runtime_api::RuntimeDispatchInfo;
 #[cfg(any(feature = "std", test))]
 use version::NativeVersion;
 use version::RuntimeVersion;
-
-use grandpa::fg_primitives;
-use grandpa::AuthorityList as GrandpaAuthorityList;
-use im_online::sr25519::AuthorityId as ImOnlineId;
-use substrate_primitives::OpaqueMetadata;
-use system::offchain::TransactionSubmitter;
-use transaction_payment_rpc_runtime_api::RuntimeDispatchInfo;
+//use grandpa::fg_primitives;
+//use grandpa::{AuthorityId as GrandpaId, AuthorityWeight as GrandpaWeight};
+//use im_online::sr25519::AuthorityId as ImOnlineId;
 
 use darwinia_support::TimeStamp;
 use staking::EraIndex;
-pub use staking::StakerStatus;
 
+/// Constant values used within the runtime.
+pub mod constants;
 /// Implementations of some helper traits passed into runtime modules as associated types.
 pub mod impls;
-use impls::{Author, CurrencyToVoteHandler, LinearWeightToFee, TargetedFeeAdjustment};
 
-/// Constant values used within the runtime.
-pub mod constants;
-use constants::time::*;
+use constants::{currency::*, time::*};
+use impls::{Author, CurrencyToVoteHandler, LinearWeightToFee, TargetedFeeAdjustment};
 
 // Make the WASM binary available.
 #[cfg(feature = "std")]
@@ -96,12 +88,15 @@ pub fn native_version() -> NativeVersion {
 	}
 }
 
-pub const NANO: Balance = 1;
-pub const MICRO: Balance = 1_000 * NANO;
-pub const MILLI: Balance = 1_000 * MICRO;
-pub const COIN: Balance = 1_000 * MILLI;
-
 type NegativeImbalance = <Balances as Currency<AccountId>>::NegativeImbalance;
+type DealWithFees = SplitTwoWays<
+	Balance,
+	NegativeImbalance,
+	_4,
+	MockTreasury, // 4 parts (80%) goes to the treasury.
+	_1,
+	Author, // 1 part (20%) goes to the block author.
+>;
 
 //pub struct Author;
 //
@@ -118,20 +113,6 @@ impl OnUnbalanced<NegativeImbalance> for MockTreasury {
 	}
 }
 
-pub type DealWithFees = SplitTwoWays<
-	Balance,
-	NegativeImbalance,
-	_4,
-	MockTreasury, // 4 parts (80%) goes to the treasury.
-	_1,
-	Author, // 1 part (20%) goes to the block author.
->;
-
-pub const SECS_PER_BLOCK: BlockNumber = 6;
-pub const MINUTES: BlockNumber = 60 / SECS_PER_BLOCK;
-pub const HOURS: BlockNumber = MINUTES * 60;
-pub const DAYS: BlockNumber = HOURS * 24;
-
 parameter_types! {
 	pub const BlockHashCount: BlockNumber = 250;
 	pub const MaximumBlockWeight: Weight = 1_000_000_000;
@@ -170,7 +151,7 @@ impl indices::Trait for Runtime {
 }
 
 parameter_types! {
-	pub const ExistentialDeposit: Balance = 1 * COIN;
+	pub const ExistentialDeposit: Balance = 1 * MILLICENTS;
 	pub const TransferFee: Balance = 1 * MILLI;
 	pub const CreationFee: Balance = 1 * MILLI;
 }
@@ -314,10 +295,10 @@ parameter_types! {
 	pub const ContractTransactionBaseFee: Balance = 1 * MILLI;
 	pub const ContractTransactionByteFee: Balance = 10 * MICRO;
 	pub const ContractFee: Balance = 1 * MILLI;
-	pub const TombstoneDeposit: Balance = 1 * COIN;
-	pub const RentByteFee: Balance = 1 * COIN;
-	pub const RentDepositOffset: Balance = 1000 * COIN;
-	pub const SurchargeReward: Balance = 150 * COIN;
+	pub const TombstoneDeposit: Balance = 1 * MILLICENTS;
+	pub const RentByteFee: Balance = 1 * MILLICENTS;
+	pub const RentDepositOffset: Balance = 1000 * MILLICENTS;
+	pub const SurchargeReward: Balance = 150 * MILLICENTS;
 }
 impl contracts::Trait for Runtime {
 	type Currency = Balances;
@@ -394,7 +375,7 @@ parameter_types! {
 	// 365 days * 24 hours * 60 minutes / 5 minutes
 	pub const ErasPerEpoch: EraIndex = 105120;
 	// decimal 9
-	pub const HardCap: Balance = 10_000_000_000 * COIN;
+	pub const HardCap: Balance = 10_000_000_000 * MILLICENTS;
 	pub const GenesisTime: Moment = 1_574_156_000_000;
 }
 impl staking::Trait for Runtime {

srml/kton/Cargo.toml

@@ -22,6 +22,9 @@ darwinia-support = { path = "../support", default-features = false }
 runtime_io = { package = "sr-io", git = 'https://github.com/darwinia-network/substrate.git' }
 substrate-primitives = { git = 'https://github.com/darwinia-network/substrate.git' }
 
+node-runtime = { path = "../../node/runtime" }
+node-primitives = { path = "../../node/primitives" }
+
 [features]
 default = ["std"]
 std = [

srml/kton/src/mock.rs

@@ -1,27 +1,33 @@
+pub use node_runtime::constants::currency::MILLICENTS;
+
 use std::{cell::RefCell, collections::HashSet};
 
 use sr_primitives::{
 	testing::Header,
 	traits::{BlakeTwo256, IdentityLookup},
+	weights::Weight,
 	Perbill,
 };
 use srml_support::{impl_outer_origin, parameter_types};
 use substrate_primitives::H256;
 
 use super::*;
 use crate::{GenesisConfig, Module};
-
-pub const COIN: u64 = 1_000_000_000;
+use node_primitives::Balance;
 
 thread_local! {
 	static SESSION: RefCell<(Vec<AccountId>, HashSet<AccountId>)> = RefCell::new(Default::default());
-	static EXISTENTIAL_DEPOSIT: RefCell<u64> = RefCell::new(0);
+	static EXISTENTIAL_DEPOSIT: RefCell<Balance> = RefCell::new(0);
 }
 
 /// The AccountId alias in this test module.
 pub type AccountId = u64;
+// FIXME:
+//     replace
+//     	  testing::Header.number: u64
+//     with
+//         node_primitives::BlockNumber
 pub type BlockNumber = u64;
-pub type Balance = u64;
 
 impl_outer_origin! {
 	pub enum Origin for Test {}
@@ -31,8 +37,8 @@ impl_outer_origin! {
 #[derive(Clone, PartialEq, Eq, Debug)]
 pub struct Test;
 parameter_types! {
-	pub const BlockHashCount: u64 = 250;
-	pub const MaximumBlockWeight: u32 = 1024;
+	pub const BlockHashCount: BlockNumber = 250;
+	pub const MaximumBlockWeight: Weight = 1024;
 	pub const MaximumBlockLength: u32 = 2 * 1024;
 	pub const AvailableBlockRatio: Perbill = Perbill::one();
 }
@@ -71,7 +77,7 @@ impl Trait for Test {
 }
 
 pub struct ExtBuilder {
-	existential_deposit: u64,
+	existential_deposit: Balance,
 }
 
 impl Default for ExtBuilder {
@@ -81,7 +87,7 @@ impl Default for ExtBuilder {
 }
 
 impl ExtBuilder {
-	pub fn existential_deposit(mut self, existential_deposit: u64) -> Self {
+	pub fn existential_deposit(mut self, existential_deposit: Balance) -> Self {
 		self.existential_deposit = existential_deposit;
 		self
 	}
@@ -94,9 +100,9 @@ impl ExtBuilder {
 		self.set_associated_consts();
 		let mut t = system::GenesisConfig::default().build_storage::<Test>().unwrap();
 		let balance_factor = if self.existential_deposit > 0 {
-			1_000 * COIN
+			1_000 * MILLICENTS
 		} else {
-			1 * COIN
+			1 * MILLICENTS
 		};
 
 		let _ = GenesisConfig::<Test> {

srml/staking/Cargo.toml

@@ -29,9 +29,10 @@ substrate-primitives = { git = 'https://github.com/darwinia-network/substrate.gi
 timestamp = { package = "srml-timestamp", git = 'https://github.com/darwinia-network/substrate.git' }
 rand = "0.7.2"
 
-balances = { package = "darwinia-balances", path = '../balances', default-features = false }
+balances = { package = "darwinia-balances", path = '../balances' }
 kton = { package = "darwinia-kton", path = "../kton" }
 node-runtime = { path = "../../node/runtime" }
+node-primitives = { path = "../../node/primitives" }
 
 [features]
 equalize = []