Restore append before checking error (#1782)

elastic · Apr 18, 2024 · b2c2c7a · b2c2c7a
1 parent 6eda8fe
commit b2c2c7a
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/cmd/testrunner.go b/cmd/testrunner.go
@@ -365,11 +365,15 @@ func testTypeCommandActionFactory(runner testrunner.TestRunner) cobraext.Command
 				RunTestsOnly:               runTestsOnly,
 				RunIndependentElasticAgent: runIndependentElasticAgent,
 			})
+
+			// Results must be appended even if there is an error, since there could be
+			// tests (e.g. system tests) that return both error and results.
+			results = append(results, r...)
+
 			if err != nil {
 				return fmt.Errorf("error running package %s tests: %w", testType, err)
 			}
 
-			results = append(results, r...)
 		}
 
 		format := testrunner.TestReportFormat(reportFormat)