Ensure Executions returns an immutable list

sbrannen · sbrannen · commit 9bb778614fdd · 2018-12-19T13:48:01.000+01:00
Issue: #1356
diff --git a/junit-platform-testkit/src/main/java/org/junit/platform/testkit/engine/Executions.java b/junit-platform-testkit/src/main/java/org/junit/platform/testkit/engine/Executions.java
@@ -56,7 +56,7 @@ private Executions(Stream<Execution> executions, String category) {
 		Preconditions.notNull(events, "Event list must not be null");
 		Preconditions.containsNoNullElements(events, "Event list must not contain null elements");
 
-		this.executions = createExecutions(events);
+		this.executions = Collections.unmodifiableList(createExecutions(events));
 		this.category = category;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ private Executions(Stream<Execution> executions, String category) {`
`56`	`56`	`Preconditions.notNull(events, "Event list must not be null");`
`57`	`57`	`Preconditions.containsNoNullElements(events, "Event list must not contain null elements");`
`58`	`58`
`59`		`- this.executions = createExecutions(events);`
	`59`	`+ this.executions = Collections.unmodifiableList(createExecutions(events));`
`60`	`60`	`this.category = category;`
`61`	`61`	`}`
`62`	`62`