diff --git a/crates/forge/src/run_tests/messages/mod.rs b/crates/forge/src/run_tests/messages/mod.rs index 9d28624e83..9f091cb4e1 100644 --- a/crates/forge/src/run_tests/messages/mod.rs +++ b/crates/forge/src/run_tests/messages/mod.rs @@ -1,6 +1,6 @@ pub mod collected_tests_count; pub mod latest_blocks_numbers; -pub mod overall_summary; pub mod tests_failure_summary; pub mod tests_run; pub mod tests_summary; +pub mod workspace_summary; diff --git a/crates/forge/src/run_tests/messages/overall_summary.rs b/crates/forge/src/run_tests/messages/workspace_summary.rs similarity index 87% rename from crates/forge/src/run_tests/messages/overall_summary.rs rename to crates/forge/src/run_tests/messages/workspace_summary.rs index ef554701e9..6829515f3b 100644 --- a/crates/forge/src/run_tests/messages/overall_summary.rs +++ b/crates/forge/src/run_tests/messages/workspace_summary.rs @@ -6,11 +6,11 @@ use serde::Serialize; use serde_json::{Value, json}; #[derive(Serialize)] -pub struct OverallSummaryMessage { +pub struct WorkspaceSummaryMessage { summary: TestsSummary, } -impl OverallSummaryMessage { +impl WorkspaceSummaryMessage { pub const LABEL: &str = "Tests summary"; #[must_use] @@ -21,7 +21,7 @@ impl OverallSummaryMessage { } } -impl Message for OverallSummaryMessage { +impl Message for WorkspaceSummaryMessage { fn text(&self) -> String { let styled_label = style(&Self::LABEL).bold().to_string(); LabeledMessage::new(&styled_label, &self.summary.format_summary_message()).text() diff --git a/crates/forge/src/run_tests/package.rs b/crates/forge/src/run_tests/package.rs index a205856d07..83e378a19a 100644 --- a/crates/forge/src/run_tests/package.rs +++ b/crates/forge/src/run_tests/package.rs @@ -9,8 +9,7 @@ use crate::{ combine_configs::combine_configs, run_tests::{ messages::{ - collected_tests_count::CollectedTestsCountMessage, tests_run::TestsRunMessage, - tests_summary::TestsSummaryMessage, + collected_tests_count::CollectedTestsCountMessage, tests_summary::TestsSummaryMessage, }, workspace::WorkspaceDirs, }, @@ -201,11 +200,6 @@ pub async fn run_for_package( for test_target in test_targets { let ui = ui.clone(); - ui.println(&TestsRunMessage::new( - test_target.tests_location, - test_target.test_cases.len(), - )); - let summary = run_for_test_target(test_target, forge_config.clone(), &tests_filter, ui).await?; diff --git a/crates/forge/src/run_tests/test_target.rs b/crates/forge/src/run_tests/test_target.rs index e03e6d874e..085fd76873 100644 --- a/crates/forge/src/run_tests/test_target.rs +++ b/crates/forge/src/run_tests/test_target.rs @@ -14,6 +14,8 @@ use futures::{StreamExt, stream::FuturesUnordered}; use std::sync::Arc; use tokio::sync::mpsc::channel; +use crate::run_tests::messages::tests_run::TestsRunMessage; + #[non_exhaustive] pub enum TestTargetRunResult { Ok(TestTargetSummary), @@ -27,6 +29,11 @@ pub async fn run_for_test_target( tests_filter: &impl TestCaseFilter, ui: Arc, ) -> Result { + ui.println(&TestsRunMessage::new( + tests.tests_location, + tests.test_cases.len(), + )); + let casm_program = tests.casm_program.clone(); let mut tasks = FuturesUnordered::new(); diff --git a/crates/forge/src/run_tests/workspace.rs b/crates/forge/src/run_tests/workspace.rs index a351887d47..a5d9513846 100644 --- a/crates/forge/src/run_tests/workspace.rs +++ b/crates/forge/src/run_tests/workspace.rs @@ -1,7 +1,7 @@ use super::package::RunForPackageArgs; use crate::run_tests::messages::latest_blocks_numbers::LatestBlocksNumbersMessage; -use crate::run_tests::messages::overall_summary::OverallSummaryMessage; use crate::run_tests::messages::tests_failure_summary::TestsFailureSummaryMessage; +use crate::run_tests::messages::workspace_summary::WorkspaceSummaryMessage; use crate::{ ExitStatus, TestArgs, block_number_map::BlockNumberMap, run_tests::package::run_for_package, scarb::build_artifacts_with_scarb, shared_cache::FailedTestsCache, @@ -97,7 +97,7 @@ pub async fn run_for_workspace( .map(|(total, filtered)| total + filtered); } - let overall_summary = OverallSummaryMessage::new(&all_tests, total_filtered_count); + let workspace_summary = WorkspaceSummaryMessage::new(&all_tests, total_filtered_count); let all_failed_tests: Vec = extract_failed_tests(all_tests).collect(); FailedTestsCache::new(&workspace_dirs.cache_dir).save_failed_tests(&all_failed_tests)?; @@ -114,7 +114,7 @@ pub async fn run_for_workspace( if packages_len > 1 { // Add newline to separate summary from previous output ui.print_blank_line(); - ui.println(&overall_summary); + ui.println(&workspace_summary); } if args.exact {