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.
     #[track_caller]
     pub fn stream_capture_stdout(
@@ -231,7 +222,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();
     }
 
@@ -663,7 +654,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.

src/etc/completions/x.fish

@@ -293,7 +293,7 @@ complete -c x -n "__fish_x_using_subcommand doc" -l skip-std-check-if-no-downloa
 complete -c x -n "__fish_x_using_subcommand doc" -s h -l help -d 'Print help (see more with \'--help\')'
 complete -c x -n "__fish_x_using_subcommand test" -l test-args -d 'extra arguments to be passed for the test tool being used (e.g. libtest, compiletest or rustdoc)' -r
 complete -c x -n "__fish_x_using_subcommand test" -l compiletest-rustc-args -d 'extra options to pass the compiler when running compiletest tests' -r
-complete -c x -n "__fish_x_using_subcommand test" -l extra-checks -d 'comma-separated list of other files types to check (accepts py, py:lint, py:fmt, shell)' -r
+complete -c x -n "__fish_x_using_subcommand test" -l extra-checks -d 'comma-separated list of other files types to check (accepts py, py:lint, py:fmt, shell, shell:lint, cpp, cpp:fmt, spellcheck, spellcheck:fix)' -r
 complete -c x -n "__fish_x_using_subcommand test" -l compare-mode -d 'mode describing what file the actual ui output will be compared to' -r
 complete -c x -n "__fish_x_using_subcommand test" -l pass -d 'force {check,build,run}-pass tests to this mode' -r
 complete -c x -n "__fish_x_using_subcommand test" -l run -d 'whether to execute run-* tests' -r