Merge pull request #34 from nunit/issue-33

Fix error due to uninitialized property
nunit · Nov 2, 2024 · 8d2c732 · 8d2c732
2 parents 4b03425 + d43fa35
commit 8d2c732
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/GitVersion.yml b/GitVersion.yml
@@ -1,5 +1,5 @@
 # This copy of GitVersion.yml is used in building the recipe package itself.
-next-version: 1.2.0
+next-version: 1.2.1
 mode: ContinuousDelivery
 legacy-semver-padding: 5
 build-metadata-padding: 5

diff --git a/recipe/test-reports.cake b/recipe/test-reports.cake
@@ -3,16 +3,14 @@ public class PackageTestReport
 	public PackageTest Test;
 	public ActualResult Result;
 	public ITestRunner Runner;
-	public List<string> Errors;
-	public List<string> Warnings;
+	public List<string> Errors = new List<string>();
+	public List<string> Warnings = new List<string>();
 
 	public PackageTestReport(PackageTest test, ActualResult actualResult, ITestRunner runner = null)
 	{
 		Test = test;
 		Result = actualResult;
 		Runner = runner;
-		Errors = new List<string>();
-		Warnings = new List<string>();
 
 		var expectedResult = test.ExpectedResult;
 
@@ -59,9 +57,9 @@ public class PackageTestReport
 	{
 		Test = test;
 		Result = null;
-		Errors = new List<string>();
+        Runner = runner;
+
 		Errors.Add($"     {ex.Message}");
-		Runner = runner;
 	}
 
 	public void Display(int index, TextWriter writer)