Skip to content

Commit

Permalink
move chain-test-utils to chain-impl-mockchain
Browse files Browse the repository at this point in the history
These methods are not used elsewhere so now:

- They are moved to chain-impl-mockchain.
- chain-test-utils is removed.
  • Loading branch information
eugene-babichenko committed Jul 21, 2021
1 parent 925dbd4 commit 9c82f77
Show file tree
Hide file tree
Showing 12 changed files with 22 additions and 33 deletions.
1 change: 0 additions & 1 deletion Cargo.toml
Expand Up @@ -9,7 +9,6 @@ members = [
"chain-crypto",
"chain-network",
"chain-storage",
"chain-test-utils",
"chain-impl-mockchain",
"cardano-legacy-address",
"sparse-array",
Expand Down
3 changes: 0 additions & 3 deletions chain-impl-mockchain/Cargo.toml
Expand Up @@ -26,7 +26,6 @@ thiserror = "1.0"
lazy_static = { version = "1.3.0", optional = true }
cardano-legacy-address = { path= "../cardano-legacy-address" }
rand_chacha = { version = "0.3", optional = true }
chain-test-utils = { path = "../chain-test-utils", optional = true }
rayon = "1.5.0"
criterion = { version = "0.3.0", optional = true }
rand = "0.8"
Expand All @@ -36,7 +35,6 @@ property-test-api = [
"chain-crypto/property-test-api",
"chain-time/property-test-api",
"chain-addr/property-test-api",
"chain-test-utils",
"quickcheck",
"quickcheck_macros",
"lazy_static",
Expand All @@ -48,7 +46,6 @@ with-bench = ["criterion","property-test-api"]
quickcheck = "0.9"
quickcheck_macros = "0.9"
chain-core = { path = "../chain-core"}
chain-test-utils = { path = "../chain-test-utils" }
chain-crypto = { path = "../chain-crypto", features=["property-test-api"]}
chain-time = { path = "../chain-time", features=["property-test-api"]}
chain-addr = { path = "../chain-addr", features=["property-test-api"]}
Expand Down
12 changes: 6 additions & 6 deletions chain-impl-mockchain/src/block/test.rs
Expand Up @@ -2,6 +2,8 @@
use crate::block::Header;
#[cfg(test)]
use crate::header::HeaderDesc;
#[cfg(test)]
use crate::testing::serialization::{serialization_bijection, serialization_bijection_r};
use crate::{
block::{Block, BlockVersion, HeaderRaw},
fragment::{Contents, ContentsBuilder, Fragment},
Expand All @@ -10,26 +12,24 @@ use crate::{
#[cfg(test)]
use chain_core::property::{Block as _, Deserialize, HasHeader as _, Serialize};
#[cfg(test)]
use chain_test_utils::property;
#[cfg(test)]
use quickcheck::TestResult;
use quickcheck::{Arbitrary, Gen};

quickcheck! {
fn headerraw_serialization_bijection(b: HeaderRaw) -> TestResult {
property::serialization_bijection(b)
serialization_bijection(b)
}

fn header_serialization_bijection(b: Header) -> TestResult {
property::serialization_bijection_r(b)
serialization_bijection_r(b)
}

fn block_serialization_bijection(b: Block) -> TestResult {
property::serialization_bijection(b)
serialization_bijection(b)
}

fn block_serialization_bijection_r(b: Block) -> TestResult {
property::serialization_bijection_r(b)
serialization_bijection_r(b)
}

fn block_properties(block: Block) -> TestResult {
Expand Down
4 changes: 3 additions & 1 deletion chain-impl-mockchain/src/fragment/test.rs
@@ -1,6 +1,8 @@
use super::*;
use crate::config::ConfigParam;
#[cfg(test)]
use crate::testing::serialization::serialization_bijection_r;
#[cfg(test)]
use quickcheck::TestResult;
use quickcheck::{Arbitrary, Gen};
use quickcheck_macros::quickcheck;
Expand Down Expand Up @@ -35,7 +37,7 @@ fn fragment_serialization_bijection(b: Fragment) -> TestResult {

quickcheck! {
fn initial_ents_serialization_bijection(config_params: ConfigParams) -> TestResult {
chain_test_utils::property::serialization_bijection_r(config_params)
serialization_bijection_r(config_params)
}
}

Expand Down
4 changes: 3 additions & 1 deletion chain-impl-mockchain/src/header/test.rs
Expand Up @@ -2,6 +2,8 @@ use super::*;
use crate::chaintypes::ChainLength;
use crate::header::{BftProof, BftSignature, Common, GenesisPraosProof, KesSignature};
use crate::key::BftLeaderId;
#[cfg(test)]
use crate::testing::serialization::serialization_bijection_r;
use chain_crypto::{
self, AsymmetricKey, Ed25519, RistrettoGroup2HashDh, SecretKey, SumEd25519_12,
VerifiableRandomFunction,
Expand All @@ -13,7 +15,7 @@ use quickcheck::{Arbitrary, Gen};

quickcheck! {
fn header_serialization_bijection(b: Header) -> TestResult {
chain_test_utils::property::serialization_bijection_r(b)
serialization_bijection_r(b)
}
}

Expand Down
4 changes: 2 additions & 2 deletions chain-impl-mockchain/src/key.rs
Expand Up @@ -404,9 +404,9 @@ impl Readable for GenesisPraosLeader {
#[cfg(any(test, feature = "property-test-api"))]
mod tests {
use super::*;
use chain_crypto::{testing, PublicKey, RistrettoGroup2HashDh, SecretKey, SumEd25519_12};
#[cfg(test)]
use chain_test_utils::property::serialization_bijection;
use crate::testing::serialization::serialization_bijection;
use chain_crypto::{testing, PublicKey, RistrettoGroup2HashDh, SecretKey, SumEd25519_12};
use lazy_static::lazy_static;
#[cfg(test)]
use quickcheck::TestResult;
Expand Down
1 change: 1 addition & 0 deletions chain-impl-mockchain/src/testing/mod.rs
Expand Up @@ -12,3 +12,4 @@ pub use builders::*;
pub use data::KeysDb;
pub use gen::{TestGen, VoteTestGen};
pub use ledger::{ConfigBuilder, LedgerBuilder, TestLedger, UtxoDb};
pub mod serialization;
File renamed without changes.
8 changes: 5 additions & 3 deletions chain-impl-mockchain/src/transaction/test.rs
Expand Up @@ -5,6 +5,8 @@ use super::{
#[cfg(test)]
use crate::certificate::OwnerStakeDelegation;
use crate::key::{EitherEd25519SecretKey, SpendingSignature};
#[cfg(test)]
use crate::testing::serialization::serialization_bijection_r;
use chain_crypto::{testing::arbitrary_secret_key, Ed25519, SecretKey, Signature};
#[cfg(test)]
use quickcheck::TestResult;
Expand All @@ -13,18 +15,18 @@ use quickcheck_macros::quickcheck;

quickcheck! {
fn transaction_encode_decode(transaction: Transaction<NoExtra>) -> TestResult {
chain_test_utils::property::serialization_bijection_r(transaction)
serialization_bijection_r(transaction)
}
fn stake_owner_delegation_tx_encode_decode(transaction: Transaction<OwnerStakeDelegation>) -> TestResult {
chain_test_utils::property::serialization_bijection_r(transaction)
serialization_bijection_r(transaction)
}
/*
fn certificate_tx_encode_decode(transaction: Transaction<Address, Certificate>) -> TestResult {
chain_core::property::testing::serialization_bijection_r(transaction)
}
*/
fn signed_transaction_encode_decode(transaction: Transaction<NoExtra>) -> TestResult {
chain_test_utils::property::serialization_bijection_r(transaction)
serialization_bijection_r(transaction)
}
}

Expand Down
4 changes: 2 additions & 2 deletions chain-impl-mockchain/src/update.rs
Expand Up @@ -396,6 +396,8 @@ impl Readable for SignedUpdateVote {
mod tests {
use super::*;
#[cfg(test)]
use crate::testing::serialization::serialization_bijection;
#[cfg(test)]
use crate::{
config::ConfigParam,
testing::{
Expand All @@ -414,8 +416,6 @@ mod tests {
#[cfg(test)]
use chain_addr::Discrimination;
#[cfg(test)]
use chain_test_utils::property::serialization_bijection;
#[cfg(test)]
use quickcheck::TestResult;
use quickcheck::{Arbitrary, Gen};
use quickcheck_macros::quickcheck;
Expand Down
13 changes: 0 additions & 13 deletions chain-test-utils/Cargo.toml

This file was deleted.

1 change: 0 additions & 1 deletion chain-test-utils/src/lib.rs

This file was deleted.

0 comments on commit 9c82f77

Please sign in to comment.