Remove 'seed' arg from 'with_project' to FinishedTests event

Also polish a bit the output of tests, move test result to stdout to allow filtering out warnings by redirecting stderr to /dev/null.

crates/aiken-project/src/lib.rs

@@ -347,7 +347,7 @@ where
                     .collect();
 
                 self.event_listener
-                    .handle_event(Event::FinishedTests { tests });
+                    .handle_event(Event::FinishedTests { seed, tests });
 
                 if !errors.is_empty() {
                     Err(errors)

crates/aiken-project/src/telemetry.rs

@@ -37,6 +37,7 @@ pub enum Event {
     },
     RunningTests,
     FinishedTests {
+        seed: u32,
         tests: Vec<TestResult<UntypedExpr>>,
     },
     WaitingForBuildDirLock,
@@ -170,7 +171,7 @@ impl EventListener for Terminal {
                     "...".if_supports_color(Stderr, |s| s.bold())
                 );
             }
-            Event::FinishedTests { tests } => {
+            Event::FinishedTests { seed, tests } => {
                 let (max_mem, max_cpu, max_iter) = find_max_execution_units(&tests);
 
                 for (module, results) in &group_by_module(&tests) {
@@ -185,10 +186,22 @@ impl EventListener for Terminal {
                         .collect::<Vec<String>>()
                         .join("\n");
 
-                    let summary = fmt_test_summary(results, true);
+                    let seed_info = if results
+                        .iter()
+                        .any(|t| matches!(t, TestResult::PropertyTestResult { .. }))
+                    {
+                        format!(
+                            "with {opt}={seed} → ",
+                            opt = "--seed".if_supports_color(Stderr, |s| s.bold()),
+                            seed = format!("{seed}").if_supports_color(Stderr, |s| s.bold())
+                        )
+                    } else {
+                        String::new()
+                    };
 
-                    eprintln!(
-                        "{}\n",
+                    let summary = format!("{}{}", seed_info, fmt_test_summary(results, true));
+                    println!(
+                        "{}",
                         pretty::indent(
                             &pretty::open_box(&title, &tests, &summary, |border| border
                                 .if_supports_color(Stderr, |s| s.bright_black())
@@ -319,19 +332,20 @@ fn fmt_test(
     }) = result
     {
         test = format!(
-            "{test}\n{}",
-            pretty::open_box(
-                &pretty::style_if(styled, "counterexample".to_string(), |s| s
-                    .if_supports_color(Stderr, |s| s.red())
-                    .if_supports_color(Stderr, |s| s.bold())
-                    .to_string()),
-                &Formatter::new()
-                    .expr(counterexample, false)
-                    .to_pretty_string(70),
-                "",
-                |s| s.red().to_string()
-            )
-        )
+            "{test}\n{}\n{}\n",
+            "× counterexample"
+                .if_supports_color(Stderr, |s| s.red())
+                .if_supports_color(Stderr, |s| s.bold()),
+            &Formatter::new()
+                .expr(counterexample, false)
+                .to_pretty_string(60)
+                .lines()
+                .map(|line| {
+                    format!("{} {}", "│".if_supports_color(Stderr, |s| s.red()), line)
+                })
+                .collect::<Vec<String>>()
+                .join("\n")
+        );
     }
 
     // Traces

crates/aiken-project/src/watch.rs

@@ -75,12 +75,7 @@ pub fn default_filter(evt: &Event) -> bool {
     }
 }
 
-pub fn with_project<A>(
-    directory: Option<&Path>,
-    seed: u32,
-    deny: bool,
-    mut action: A,
-) -> miette::Result<()>
+pub fn with_project<A>(directory: Option<&Path>, deny: bool, mut action: A) -> miette::Result<()>
 where
     A: FnMut(&mut Project<Terminal>) -> Result<(), Vec<crate::error::Error>>,
 {
@@ -113,23 +108,16 @@ where
             err.report()
         }
 
-        eprintln!(
-            "{}",
-            Summary {
-                warning_count,
-                error_count: errs.len(),
-            }
-        );
-
-        if errs.iter().any(|e| matches!(e, Error::TestFailure { .. })) {
+        if !errs.iter().any(|e| matches!(e, Error::TestFailure { .. })) {
             eprintln!(
-                "                {}══╤══\n{}                  ╰─▶ use {} {} to replay",
-                if errs.len() > 1 { "═" } else { "" },
-                if errs.len() > 1 { " " } else { "" },
-                "--seed".if_supports_color(Stderr, |s| s.bold()),
-                format!("{seed}").if_supports_color(Stderr, |s| s.bold())
+                "{}",
+                Summary {
+                    warning_count,
+                    error_count: errs.len(),
+                }
             );
         }
+
         return Err(ExitFailure::into_report());
     }
 
@@ -162,7 +150,6 @@ where
 pub fn watch_project<F, A>(
     directory: Option<&Path>,
     filter: F,
-    seed: u32,
     debounce: u32,
     mut action: A,
 ) -> miette::Result<()>
@@ -228,13 +215,14 @@ where
         // If we have an event that survived the filter, then we can construct the project and invoke the action
         if latest.is_some() {
             print!("{esc}c", esc = 27 as char);
-            println!(
+            eprint!("{esc}c", esc = 27 as char);
+            eprintln!(
                 "{} ...",
                 "     Watching"
                     .if_supports_color(Stderr, |s| s.bold())
                     .if_supports_color(Stderr, |s| s.purple()),
             );
-            with_project(directory, seed, false, &mut action).unwrap_or(())
+            with_project(directory, false, &mut action).unwrap_or(())
         }
     }
 }

crates/aiken/src/cmd/blueprint/address.rs

@@ -39,7 +39,7 @@ pub fn exec(
         mainnet,
     }: Args,
 ) -> miette::Result<()> {
-    with_project(directory.as_deref(), u32::default(), false, |p| {
+    with_project(directory.as_deref(), false, |p| {
         if rebuild {
             p.build(false, Tracing::silent())?;
         }

crates/aiken/src/cmd/blueprint/apply.rs

@@ -47,7 +47,7 @@ pub fn exec(
         validator,
     }: Args,
 ) -> miette::Result<()> {
-    with_project(None, u32::default(), false, |p| {
+    with_project(None, false, |p| {
         let title = module.as_ref().map(|m| {
             format!(
                 "{m}{}",

crates/aiken/src/cmd/blueprint/hash.rs

@@ -29,7 +29,7 @@ pub fn exec(
         rebuild,
     }: Args,
 ) -> miette::Result<()> {
-    with_project(directory.as_deref(), u32::default(), false, |p| {
+    with_project(directory.as_deref(), false, |p| {
         if rebuild {
             p.build(false, Tracing::silent())?;
         }

crates/aiken/src/cmd/blueprint/policy.rs

@@ -29,7 +29,7 @@ pub fn exec(
         rebuild,
     }: Args,
 ) -> miette::Result<()> {
-    with_project(directory.as_deref(), u32::default(), false, |p| {
+    with_project(directory.as_deref(), false, |p| {
         if rebuild {
             p.build(false, Tracing::silent())?;
         }

crates/aiken/src/cmd/build.rs

@@ -51,23 +51,17 @@ pub fn exec(
     }: Args,
 ) -> miette::Result<()> {
     let result = if watch {
-        watch_project(
-            directory.as_deref(),
-            watch::default_filter,
-            u32::default(),
-            500,
-            |p| {
-                p.build(
-                    uplc,
-                    match filter_traces {
-                        Some(filter_traces) => filter_traces(trace_level),
-                        None => Tracing::All(trace_level),
-                    },
-                )
-            },
-        )
+        watch_project(directory.as_deref(), watch::default_filter, 500, |p| {
+            p.build(
+                uplc,
+                match filter_traces {
+                    Some(filter_traces) => filter_traces(trace_level),
+                    None => Tracing::All(trace_level),
+                },
+            )
+        })
     } else {
-        with_project(directory.as_deref(), u32::default(), deny, |p| {
+        with_project(directory.as_deref(), deny, |p| {
             p.build(
                 uplc,
                 match filter_traces {
@@ -82,8 +76,8 @@ pub fn exec(
 }
 
 #[allow(clippy::type_complexity)]
-pub fn filter_traces_parser(
-) -> MapValueParser<PossibleValuesParser, fn(String) -> fn(TraceLevel) -> Tracing> {
+pub fn filter_traces_parser()
+-> MapValueParser<PossibleValuesParser, fn(String) -> fn(TraceLevel) -> Tracing> {
     PossibleValuesParser::new(["user-defined", "compiler-generated", "all"]).map(
         |s: String| match s.as_str() {
             "user-defined" => Tracing::UserDefined,

crates/aiken/src/cmd/check.rs

@@ -79,27 +79,21 @@ pub fn exec(
     let seed = seed.unwrap_or_else(|| rng.gen());
 
     let result = if watch {
-        watch_project(
-            directory.as_deref(),
-            watch::default_filter,
-            seed,
-            500,
-            |p| {
-                p.check(
-                    skip_tests,
-                    match_tests.clone(),
-                    debug,
-                    exact_match,
-                    seed,
-                    match filter_traces {
-                        Some(filter_traces) => filter_traces(trace_level),
-                        None => Tracing::All(trace_level),
-                    },
-                )
-            },
-        )
+        watch_project(directory.as_deref(), watch::default_filter, 500, |p| {
+            p.check(
+                skip_tests,
+                match_tests.clone(),
+                debug,
+                exact_match,
+                seed,
+                match filter_traces {
+                    Some(filter_traces) => filter_traces(trace_level),
+                    None => Tracing::All(trace_level),
+                },
+            )
+        })
     } else {
-        with_project(directory.as_deref(), seed, deny, |p| {
+        with_project(directory.as_deref(), deny, |p| {
             p.check(
                 skip_tests,
                 match_tests.clone(),

crates/aiken/src/cmd/docs.rs

@@ -29,17 +29,11 @@ pub fn exec(
     }: Args,
 ) -> miette::Result<()> {
     let result = if watch {
-        watch_project(
-            directory.as_deref(),
-            watch::default_filter,
-            u32::default(),
-            500,
-            |p| p.docs(destination.clone()),
-        )
-    } else {
-        with_project(directory.as_deref(), u32::default(), deny, |p| {
+        watch_project(directory.as_deref(), watch::default_filter, 500, |p| {
             p.docs(destination.clone())
         })
+    } else {
+        with_project(directory.as_deref(), deny, |p| p.docs(destination.clone()))
     };
 
     result.map_err(|_| process::exit(1))