Skip to content

Commit

Permalink
Fix tests
Browse files Browse the repository at this point in the history
  • Loading branch information
@QuentinI
QuentinI committed Oct 10, 2024
1 parent 4346865 commit f502edd
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions crates/marketplace/src/testing/integration.rs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ where
global_state.clone(),
Arc::clone(&hooks),
builder_key_pair,
Duration::from_secs(60),
Duration::from_millis(500),
)
.into_app()
.expect("Failed to create builder tide-disco app");
Expand Down Expand Up @@ -197,13 +197,13 @@ cross_tests!(
Ignore: true,
Metadata: {
let mut metadata = TestDescription::default();
metadata.overall_safety_properties.transaction_threshold = 1;
metadata.txn_description = hotshot_testing::txn_task::TxnTaskDescription::RoundRobinTimeBased(Duration::from_millis(500));
metadata.validate_transactions = hotshot_testing::test_builder::nonempty_block_threshold((90,100));

Check failure on line 200 in crates/marketplace/src/testing/integration.rs

View workflow job for this annotation

GitHub Actions / clippy

field assignment outside of initializer for an instance created with Default::default() 

error: field assignment outside of initializer for an instance created with Default::default()
   --> crates/marketplace/src/testing/integration.rs:200:9
    |
200 |         metadata.validate_transactions = hotshot_testing::test_builder::nonempty_block_threshold((90,100));
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
note: consider initializing the variable with `TestDescription::<hotshot_example_types::state_types::TestTypes, hotshot_example_types::node_types::PushCdnImpl, hotshot_example_types::node_types::MarketplaceTestVersions> { validate_transactions: hotshot_testing::test_builder::nonempty_block_threshold((90,100)), txn_description: hotshot_testing::txn_task::TxnTaskDescription::RoundRobinTimeBased(Duration::from_millis(100)), completion_task_description: CompletionTaskDescription::TimeBasedCompletionTaskBuilder(
                  TimeBasedCompletionTaskDescription {
                      duration: Duration::from_secs(120),
                  },
              ), ..Default::default() }` and removing relevant reassignments
   --> crates/marketplace/src/testing/integration.rs:199:9
    |
199 |         let mut metadata = TestDescription::default();
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#field_reassign_with_default

Check failure on line 200 in crates/marketplace/src/testing/integration.rs

View workflow job for this annotation

GitHub Actions / clippy

field assignment outside of initializer for an instance created with Default::default() 

error: field assignment outside of initializer for an instance created with Default::default()
   --> crates/marketplace/src/testing/integration.rs:200:9
    |
200 |         metadata.validate_transactions = hotshot_testing::test_builder::nonempty_block_threshold((90,100));
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
note: consider initializing the variable with `TestDescription::<hotshot_example_types::state_types::TestTypes, hotshot_example_types::node_types::Libp2pImpl, hotshot_example_types::node_types::MarketplaceTestVersions> { validate_transactions: hotshot_testing::test_builder::nonempty_block_threshold((90,100)), txn_description: hotshot_testing::txn_task::TxnTaskDescription::RoundRobinTimeBased(Duration::from_millis(100)), completion_task_description: CompletionTaskDescription::TimeBasedCompletionTaskBuilder(
                  TimeBasedCompletionTaskDescription {
                      duration: Duration::from_secs(120),
                  },
              ), ..Default::default() }` and removing relevant reassignments
   --> crates/marketplace/src/testing/integration.rs:199:9
    |
199 |         let mut metadata = TestDescription::default();
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#field_reassign_with_default

Check failure on line 200 in crates/marketplace/src/testing/integration.rs

View workflow job for this annotation

GitHub Actions / clippy

field assignment outside of initializer for an instance created with Default::default() 

error: field assignment outside of initializer for an instance created with Default::default()
   --> crates/marketplace/src/testing/integration.rs:200:9
    |
200 |         metadata.validate_transactions = hotshot_testing::test_builder::nonempty_block_threshold((90,100));
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
note: consider initializing the variable with `TestDescription::<hotshot_example_types::state_types::TestTypes, hotshot_example_types::node_types::MemoryImpl, hotshot_example_types::node_types::MarketplaceTestVersions> { validate_transactions: hotshot_testing::test_builder::nonempty_block_threshold((90,100)), txn_description: hotshot_testing::txn_task::TxnTaskDescription::RoundRobinTimeBased(Duration::from_millis(100)), completion_task_description: CompletionTaskDescription::TimeBasedCompletionTaskBuilder(
                  TimeBasedCompletionTaskDescription {
                      duration: Duration::from_secs(120),
                  },
              ), ..Default::default() }` and removing relevant reassignments
   --> crates/marketplace/src/testing/integration.rs:199:9
    |
199 |         let mut metadata = TestDescription::default();
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#field_reassign_with_default
    = note: `-D clippy::field-reassign-with-default` implied by `-D warnings`
    = help: to override `-D warnings` add `#[allow(clippy::field_reassign_with_default)]`
metadata.txn_description = hotshot_testing::txn_task::TxnTaskDescription::RoundRobinTimeBased(Duration::from_millis(100));

metadata.completion_task_description =
CompletionTaskDescription::TimeBasedCompletionTaskBuilder(
TimeBasedCompletionTaskDescription {
duration: Duration::from_secs(60),
duration: Duration::from_secs(120),
},
);
metadata.overall_safety_properties.num_successful_views = 45;
Expand Down

0 comments on commit f502edd

Please sign in to comment.