make mark_as_executed private

Author: Shourya742

src/bootstrap/src/utils/exec.rs

@@ -211,15 +211,6 @@ impl<'a> BootstrapCommand {
         exec_ctx.as_ref().start(self, OutputMode::Capture, OutputMode::Print)
     }
 
-    /// Spawn the command in background, while capturing and returns a handle to stream the output.
-    #[track_caller]
-    pub fn stream_capture(
-        &'a mut self,
-        exec_ctx: impl AsRef<ExecutionContext>,
-    ) -> Option<StreamingCommand> {
-        exec_ctx.as_ref().stream(self, OutputMode::Capture, OutputMode::Capture)
-    }
-
     /// Spawn the command in background, while capturing and returning stdout, and printing stderr.
     /// Returns None in dry-mode
     #[track_caller]
@@ -232,7 +223,7 @@ impl<'a> BootstrapCommand {
 
     /// Mark the command as being executed, disarming the drop bomb.
     /// If this method is not called before the command is dropped, its drop will panic.
-    pub fn mark_as_executed(&mut self) {
+    fn mark_as_executed(&mut self) {
         self.drop_bomb.defuse();
     }
 
@@ -664,7 +655,7 @@ impl ExecutionContext {
 
         let stdout = child.stdout.take();
         let stderr = child.stderr.take();
-        return Some(StreamingCommand { child, stdout, stderr });
+        Some(StreamingCommand { child, stdout, stderr })
     }
 }
 

src/bootstrap/src/utils/render_tests.rs

@@ -34,11 +34,6 @@ pub(crate) fn try_run_tests(
     cmd: &mut BootstrapCommand,
     stream: bool,
 ) -> bool {
-    if builder.config.dry_run() {
-        cmd.mark_as_executed();
-        return true;
-    }
-
     if !run_tests(builder, cmd, stream) {
         if builder.fail_fast {
             crate::exit!(1);
@@ -54,7 +49,9 @@ pub(crate) fn try_run_tests(
 fn run_tests(builder: &Builder<'_>, cmd: &mut BootstrapCommand, stream: bool) -> bool {
     builder.verbose(|| println!("running: {cmd:?}"));
 
-    let mut streaming_command = cmd.stream_capture_stdout(&builder.config.exec_ctx).unwrap();
+    let Some(mut streaming_command) = cmd.stream_capture_stdout(&builder.config.exec_ctx) else {
+        return true;
+    };
 
     // This runs until the stdout of the child is closed, which means the child exited. We don't
     // run this on another thread since the builder is not Sync.