From 492966e0b251119d9548e7795680b0d5a4f13ec5 Mon Sep 17 00:00:00 2001
From: Martin Pool <mbp@sourcefrog.net>
Date: Fri, 10 Nov 2023 17:08:54 -0800
Subject: [PATCH] Clean up

---
 src/lab.rs      | 2 --
 src/manifest.rs | 4 ++--
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/lab.rs b/src/lab.rs
index 7d2c5f82..bb560077 100644
--- a/src/lab.rs
+++ b/src/lab.rs
@@ -66,8 +66,6 @@ pub fn test_mutants(
             "cargo {} failed in an unmutated tree, so no mutants were tested",
             baseline_outcome.last_phase(),
         );
-        // TODO: Maybe should be Err, but it would need to be an error that can map to the right
-        // exit code.
         return Ok(output_mutex
             .into_inner()
             .expect("lock output_dir")
diff --git a/src/manifest.rs b/src/manifest.rs
index 5061abb0..15b8db8c 100644
--- a/src/manifest.rs
+++ b/src/manifest.rs
@@ -161,7 +161,7 @@ mod test {
     use pretty_assertions::assert_eq;
     use toml::Table;
 
-    use super::fix_manifest_toml;
+    use super::{fix_cargo_config_toml, fix_manifest_toml};
 
     #[test]
     fn fix_path_absolute_unchanged() {
@@ -305,7 +305,7 @@ mod test {
                 "/src/other",
             ]"# };
         let source_dir = Utf8Path::new("/Users/jane/src/foo");
-        let fixed_toml = super::fix_cargo_config_toml(cargo_config_toml, source_dir)
+        let fixed_toml = fix_cargo_config_toml(cargo_config_toml, source_dir)
             .unwrap()
             .expect("toml was modified");
         println!("fixed toml:\n{fixed_toml}");