fix(rerunners): correct handling of analysis from indexes only.

dnbln · Jan 25, 2024 · be4f344 · be4f344
1 parent 656502e
commit be4f344
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/cargo-difftests/src/main.rs b/cargo-difftests/src/main.rs
@@ -1207,8 +1207,7 @@ pub fn run_analyze_all_from_index(
         results.push(result);
     }
 
-    let out_json = serde_json::to_string(&results)?;
-    println!("{out_json}");
+    action_args.perform_for(&results)?;
 
     Ok(())
 }

diff --git a/cargo-difftests/src/test_rerunner_core.rs b/cargo-difftests/src/test_rerunner_core.rs
@@ -10,7 +10,7 @@ pub struct TestRerunnerInvocation {
 
 impl TestRerunnerInvocation {
     pub fn create_invocation_from<'a>(
-        iter: impl IntoIterator<Item = &'a AnalyzeAllSingleTestGroup>
+        iter: impl IntoIterator<Item = &'a AnalyzeAllSingleTestGroup>,
     ) -> DifftestsResult<Self> {
         let mut tests = vec![];
         let mut groups = vec![];
@@ -20,6 +20,11 @@ impl TestRerunnerInvocation {
                 tests.push(difftest.load_test_desc()?);
             } else if let Some(difftest_group) = &g.difftest_group {
                 groups.push(difftest_group.load_self_json()?);
+            } else {
+                // Most likely came from an index.
+                assert_eq!(g.test_desc.len(), 1);
+                let [test] = g.test_desc.as_slice() else { unreachable!() };
+                tests.push(test.clone());
             }
         }