delete -> remove

junit-team · Nov 9, 2015 · 80332bc · 80332bc
1 parent 1d6a7d5
commit 80332bc
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/junit-engine-api/src/main/java/org/junit/gen5/engine/AbstractTestDescriptor.java b/junit-engine-api/src/main/java/org/junit/gen5/engine/AbstractTestDescriptor.java
@@ -53,7 +53,7 @@ protected void removeChild(AbstractTestDescriptor abstractTestDescriptor) {
 
 	protected void removeFromHierarchy() {
 		if (isRoot())
-			return;
+			throw new UnsupportedOperationException("You cannot remove the root of a hierarchy.");
 		if (parent instanceof AbstractTestDescriptor)
 			((AbstractTestDescriptor) parent).removeChild(this);
 		setParent(null);
@@ -75,8 +75,8 @@ public final Set<TestDescriptor> getChildren() {
 
 	@Override
 	public void accept(Visitor visitor) {
-		Runnable delete = () -> removeFromHierarchy();
+		Runnable remove = () -> removeFromHierarchy();
-		visitor.visit(this, delete);
+		visitor.visit(this, remove);
 		new LinkedHashSet<>(getChildren()).forEach(child -> child.accept(visitor));
 	}
 

diff --git a/junit-engine-api/src/main/java/org/junit/gen5/engine/TestDescriptor.java b/junit-engine-api/src/main/java/org/junit/gen5/engine/TestDescriptor.java
@@ -44,7 +44,7 @@ default boolean isRoot() {
 
 	interface Visitor {
 
-		void visit(TestDescriptor descriptor, Runnable deleteCurrent);
+		void visit(TestDescriptor descriptor, Runnable remove);
 	}
 
 	void accept(Visitor visitor);