Merge pull request #3238 from rbtcollins/test-bonanza

Use a single integration test binary

Author: rbtcollins

ci/run.bash

@@ -61,24 +61,17 @@ build_test() {
   cmd="$1"
   shift
   download_pkg_test "${cmd}"
-  target_cargo "${cmd}" --bin rustup-init
-  target_cargo "${cmd}" --lib --all
-  if [ "build" != "${cmd}" ]; then
-    target_cargo "${cmd}" --doc --all
-  fi
-
-  if [ "build" = "${cmd}" ]; then
-    target_cargo "${cmd}" --test dist
+    if [ "build" = "${cmd}" ]; then
+    target_cargo "${cmd}" --workspace --all-targets
   else
     #  free runners have 2 or 3(mac) cores
-    target_cargo "${cmd}" --test dist -- --test-threads 2
+    target_cargo "${cmd}" --workspace --tests -- --test-threads 2
+  fi
+
+  if [ "build" != "${cmd}" ]; then
+    target_cargo "${cmd}" --doc --workspace
   fi
 
-  find tests -maxdepth 1 -type f ! -path '*/dist.rs' -name '*.rs' \
-  | sed -e 's@^tests/@@;s@\.rs$@@g' \
-  | while read -r test; do
-      target_cargo "${cmd}" --test "${test}"
-  done
 }
 
 if [ -z "$SKIP_TESTS" ]; then

tests/mock/clitools.rs

@@ -806,24 +806,6 @@ where
     !(run || self_cmd || version || (is_update && !no_self_update))
 }
 
-#[deprecated]
-pub fn run<I, A>(config: &Config, name: &str, args: I, env: &[(&str, &str)]) -> SanitizedOutput
-where
-    I: IntoIterator<Item = A> + Clone,
-    A: AsRef<OsStr>,
-{
-    config.run(name, args, env)
-}
-
-#[deprecated]
-pub fn run_subprocess<I, A>(config: &Config, name: &str, args: I, env: &[(&str, &str)]) -> Output
-where
-    I: IntoIterator<Item = A>,
-    A: AsRef<OsStr>,
-{
-    config.run_subprocess(name, args, env)
-}
-
 #[derive(Copy, Clone, Eq, PartialEq)]
 enum RlsStatus {
     Available,

tests/channel-rust-nightly-example.toml renamed to tests/suite/channel-rust-nightly-example.toml

@@ -1,8 +1,6 @@
 //! Yet more cli test cases. These are testing that the output
 //! is exactly as expected.
 
-pub mod mock;
-
 use crate::mock::clitools::{self, set_current_dist_date, with_update_server, Config, Scenario};
 use rustup::for_host;
 use rustup::test::this_host_triple;

tests/channel-rust-nightly-example2.toml renamed to tests/suite/channel-rust-nightly-example2.toml

@@ -1,7 +1,5 @@
 //! Tests of the interactive console installer
 
-pub mod mock;
-
 use std::env::consts::EXE_SUFFIX;
 use std::io::Write;
 use std::process::Stdio;

tests/cli-ui/rustup-init/rustup-init_help_flag_stdout.toml renamed to tests/suite/cli-ui/rustup-init/rustup-init_help_flag_stdout.toml

@@ -1,8 +1,6 @@
 //! Test cases of the rustup command that do not depend on the
 //! dist server, mostly derived from multirust/test-v2.sh
 
-pub mod mock;
-
 use std::str;
 use std::{env::consts::EXE_SUFFIX, path::Path};
 

tests/cli-ui/rustup-init/rustup-init_sh_help_flag_stdout.toml renamed to tests/suite/cli-ui/rustup-init/rustup-init_sh_help_flag_stdout.toml

@@ -1,7 +1,6 @@
 //! This file contains tests relevant to Rustup's handling of updating PATHs.
 //! It depends on self-update working, so if absolutely everything here breaks,
 //! check those tests as well.
-pub mod mock;
 
 // Prefer omitting actually unpacking content while just testing paths.
 const INIT_NONE: [&str; 4] = ["rustup-init", "-y", "--default-toolchain", "none"];

tests/cli-ui/rustup/rustup_check_cmd_help_flag_stdout.toml renamed to tests/suite/cli-ui/rustup/rustup_check_cmd_help_flag_stdout.toml

@@ -1,7 +1,5 @@
 //! Test cases for new rustup UI
 
-pub mod mock;
-
 use std::env::consts::EXE_SUFFIX;
 use std::fs;
 use std::path::{PathBuf, MAIN_SEPARATOR};
@@ -10,7 +8,10 @@ use rustup::for_host;
 use rustup::test::this_host_triple;
 use rustup::utils::raw;
 
-use crate::mock::clitools::{self, Config, Scenario};
+use crate::mock::{
+    self,
+    clitools::{self, Config, Scenario},
+};
 
 macro_rules! for_host_and_home {
     ($config:ident, $s: expr) => {

tests/cli-ui/rustup/rustup_completions_cmd_help_flag_stdout.toml renamed to tests/suite/cli-ui/rustup/rustup_completions_cmd_help_flag_stdout.toml

@@ -1,7 +1,5 @@
 //! Testing self install, uninstall and update
 
-pub mod mock;
-
 use std::env;
 use std::env::consts::EXE_SUFFIX;
 use std::fs;
@@ -14,9 +12,10 @@ use rustup::test::{this_host_triple, with_saved_path};
 use rustup::utils::{raw, utils};
 use rustup::{for_host, Notification, DUP_TOOLS, TOOLS};
 
-use crate::clitools::output_release_file;
-use crate::mock::clitools::{self, self_update_setup, Config, Scenario};
-use crate::mock::dist::calc_hash;
+use crate::mock::{
+    clitools::{self, output_release_file, self_update_setup, Config, Scenario},
+    dist::calc_hash,
+};
 
 const TEST_VERSION: &str = "1.1.1";
 

tests/cli-ui/rustup/rustup_component_cmd_add_cmd_help_flag_stdout.toml renamed to tests/suite/cli-ui/rustup/rustup_component_cmd_add_cmd_help_flag_stdout.toml

@@ -23,20 +23,20 @@ fn rustup_init_ui_doc_text_tests() {
     let project_root = PathBuf::from(env!("CARGO_MANIFEST_DIR"));
     t.register_bin("rustup-init", &rustup_init);
     t.register_bin("rustup-init.sh", &project_root.join("rustup-init.sh"));
-    t.case("tests/cli-ui/rustup-init/*.toml");
+    t.case("tests/suite/cli-ui/rustup-init/*.toml");
     #[cfg(target_os = "windows")]
     {
-        // On non-windows, we don't use rustup-init.sh, so skip the test.
-        t.skip("tests/cli-ui/rustup-init/rustup-init_sh_help_flag_stdout.toml");
+        // On windows, we don't use rustup-init.sh, so skip the test.
+        t.skip("tests/suite/cli-ui/rustup-init/rustup-init_sh_help_flag_stdout.toml");
     }
 
     // On non-windows, we don't use rustup-init.sh, so skip the test.
     #[cfg(not(target_os = "windows"))]
     {
         let rustup_init_help_toml =
-            project_root.join("tests/cli-ui/rustup-init/rustup-init_help_flag_stdout.toml");
-        let rustup_init_sh_help_toml =
-            project_root.join("tests/cli-ui/rustup-init/rustup-init_sh_help_flag_stdout.toml");
+            project_root.join("tests/suite/cli-ui/rustup-init/rustup-init_help_flag_stdout.toml");
+        let rustup_init_sh_help_toml = project_root
+            .join("tests/suite/cli-ui/rustup-init/rustup-init_sh_help_flag_stdout.toml");
 
         #[derive(Debug, PartialEq, Eq, serde::Deserialize, serde::Serialize)]
         struct Stdout {

tests/cli-ui/rustup/rustup_component_cmd_list_cmd_help_flag_stdout.toml renamed to tests/suite/cli-ui/rustup/rustup_component_cmd_list_cmd_help_flag_stdout.toml

@@ -1,8 +1,6 @@
 //! Test cases of the rustup command, using v1 manifests, mostly
 //! derived from multirust/test-v2.sh
 
-pub mod mock;
-
 use std::fs;
 
 use rustup::for_host;

tests/cli-ui/rustup/rustup_component_cmd_remove_cmd_help_flag_stdout.toml renamed to tests/suite/cli-ui/rustup/rustup_component_cmd_remove_cmd_help_flag_stdout.toml

@@ -1,8 +1,6 @@
 //! Test cases of the rustup command, using v2 manifests, mostly
 //! derived from multirust/test-v2.sh
 
-pub mod mock;
-
 use std::fs;
 use std::io::Write;
 

tests/cli-ui/rustup/rustup_default_cmd_help_flag_stdout.toml renamed to tests/suite/cli-ui/rustup/rustup_default_cmd_help_flag_stdout.toml

@@ -1,8 +1,6 @@
 // Tests of installation and updates from a v2 Rust distribution
 // server (mocked on the file system)
 
-pub mod mock;
-
 use std::cell::Cell;
 use std::collections::HashMap;
 use std::env;

tests/cli-ui/rustup/rustup_doc_cmd_help_flag_stdout.toml renamed to tests/suite/cli-ui/rustup/rustup_doc_cmd_help_flag_stdout.toml

@@ -1,5 +1,3 @@
-pub mod mock;
-
 use std::fs::File;
 use std::io::Write;
 

tests/cli-ui/rustup/rustup_help_cmd_stdout.toml renamed to tests/suite/cli-ui/rustup/rustup_help_cmd_stdout.toml

@@ -0,0 +1,13 @@
+mod cli_exact;
+mod cli_inst_interactive;
+mod cli_misc;
+mod cli_paths;
+mod cli_rustup;
+mod cli_self_upd;
+mod cli_ui;
+mod cli_v1;
+mod cli_v2;
+mod dist;
+mod dist_install;
+mod dist_manifest;
+mod dist_transactions;

tests/cli-ui/rustup/rustup_help_flag_stdout.toml renamed to tests/suite/cli-ui/rustup/rustup_help_flag_stdout.toml

@@ -0,0 +1,2 @@
+mod mock;
+mod suite;