Add tests to cover new mutations in cargo-mutants

Author: zaneduffield

src/fnvalue.rs

@@ -202,9 +202,6 @@ fn type_replacements(type_: &Type, error_exprs: &[Expr]) -> impl Iterator<Item =
                     .collect_vec()
             }
         },
-        Type::Tuple(TypeTuple { elems, .. }) if elems.is_empty() => {
-            vec![quote! { () }]
-        }
         Type::Tuple(TypeTuple { elems, .. }) => {
             // Generate the cartesian product of replacements of every type within the tuple.
             elems

testdata/many_patterns/src/binops.rs

@@ -6,6 +6,14 @@ pub fn binops() {
     a -= 2;
     a *= 3;
     a /= 2;
+
+    let mut b;
+    b = a < 0;
+    b = a <= 0;
+    b = a > 0;
+    b = a >= 0;
+    b = a == 0;
+    b = a != 0;
 }
 
 pub fn bin_assign() -> i32 {
@@ -15,5 +23,6 @@ pub fn bin_assign() -> i32 {
     a &= 0x0f;
     a >>= 4;
     a <<= 1;
+    a %= 1;
     a
 }

testdata/workspace_default_members/Cargo_test.toml

@@ -0,0 +1,6 @@
+# This workspace has default-members set, which influences which packages are mutated by default.
+
+[workspace]
+members = ["main", "excluded"]
+default-members = ["main"]
+resolver = "2"

testdata/workspace_default_members/excluded/Cargo_test.toml

@@ -0,0 +1,10 @@
+[package]
+name = "excluded"
+version = "0.1.0"
+edition = "2021"
+publish = false
+
+# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
+
+[dependencies]
+mutants = "0.0.3"

testdata/workspace_default_members/excluded/src/main.rs

@@ -0,0 +1,3 @@
+fn main() -> i32 {
+    100
+}

testdata/workspace_default_members/main/Cargo_test.toml

@@ -0,0 +1,10 @@
+[package]
+name = "main"
+version = "0.1.0"
+edition = "2021"
+publish = false
+
+# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
+
+[dependencies]
+mutants = "0.0.3"

testdata/workspace_default_members/main/src/main.rs

@@ -0,0 +1,6 @@
+const X: u32 = 2 + 1;
+
+#[test]
+fn test() {
+    assert_eq!(X, 3);
+}

tests/config.rs

@@ -208,14 +208,14 @@ fn exclude_re_overrides_config() {
         .success();
     assert_snapshot!(
         String::from_utf8_lossy(&cmd.get_output().stdout),
-        @r###"
+        @r"
     src/simple_fns.rs: replace returns_unit with ()
     src/simple_fns.rs: replace += with -= in returns_unit
     src/simple_fns.rs: replace += with *= in returns_unit
     src/simple_fns.rs: replace == with != in divisible_by_three
     src/simple_fns.rs: replace % with / in divisible_by_three
     src/simple_fns.rs: replace % with + in divisible_by_three
-    "###);
+    ");
 }
 
 #[test]

tests/main.rs

@@ -45,6 +45,18 @@ fn show_version() {
         .stdout(predicates::str::is_match(r"^cargo-mutants \d+\.\d+\.\d+(-.*)?\n$").unwrap());
 }
 
+#[test]
+fn show_help() {
+    // Asserting on the entire help message would be a bit too annoying to maintain.
+    run()
+        .args(["mutants", "--help"])
+        .assert()
+        .success()
+        .stdout(predicates::str::contains(
+            "Usage: cargo mutants [OPTIONS] [-- <CARGO_TEST_ARGS>...]",
+        ));
+}
+
 #[test]
 fn uses_cargo_env_var_to_run_cargo_so_invalid_value_fails() {
     let tmp_src_dir = copy_of_testdata("well_tested");