diff --git a/src/test/org/apache/commons/lang3/exception/ExceptionUtilsTest.java b/src/test/org/apache/commons/lang3/exception/ExceptionUtilsTest.java index d357b2abd..0a45bcfa0 100644 --- a/src/test/org/apache/commons/lang3/exception/ExceptionUtilsTest.java +++ b/src/test/org/apache/commons/lang3/exception/ExceptionUtilsTest.java @@ -63,7 +63,6 @@ public class ExceptionUtilsTest extends TestCase { private Throwable withCause; private Throwable withoutCause; private Throwable jdkNoCause; - private ExceptionWithCause selfCause; private ExceptionWithCause cyclicCause; public ExceptionUtilsTest(String name) { @@ -76,8 +75,6 @@ public void setUp() { nested = new NestableException(withoutCause); withCause = new ExceptionWithCause(nested); jdkNoCause = new NullPointerException(); - selfCause = new ExceptionWithCause(null); - selfCause.setCause(selfCause); ExceptionWithCause a = new ExceptionWithCause(null); ExceptionWithCause b = new ExceptionWithCause(a); a.setCause(b); @@ -90,7 +87,6 @@ protected void tearDown() throws Exception { nested = null; withCause = null; jdkNoCause = null; - selfCause = null; cyclicCause = null; } @@ -157,7 +153,6 @@ public void testGetCause_Throwable() { assertSame(withoutCause, ExceptionUtils.getCause(nested)); assertSame(nested, ExceptionUtils.getCause(withCause)); assertSame(null, ExceptionUtils.getCause(jdkNoCause)); - assertSame(selfCause, ExceptionUtils.getCause(selfCause)); assertSame(cyclicCause.getCause(), ExceptionUtils.getCause(cyclicCause)); assertSame(((ExceptionWithCause) cyclicCause.getCause()).getCause(), ExceptionUtils.getCause(cyclicCause.getCause())); assertSame(cyclicCause.getCause(), ExceptionUtils.getCause(((ExceptionWithCause) cyclicCause.getCause()).getCause())); @@ -187,7 +182,6 @@ public void testGetRootCause_Throwable() { assertSame(withoutCause, ExceptionUtils.getRootCause(nested)); assertSame(withoutCause, ExceptionUtils.getRootCause(withCause)); assertSame(null, ExceptionUtils.getRootCause(jdkNoCause)); - assertSame(null, ExceptionUtils.getRootCause(selfCause)); assertSame(((ExceptionWithCause) cyclicCause.getCause()).getCause(), ExceptionUtils.getRootCause(cyclicCause)); } @@ -217,7 +211,6 @@ public void testGetThrowableCount_Throwable() { assertEquals(2, ExceptionUtils.getThrowableCount(nested)); assertEquals(3, ExceptionUtils.getThrowableCount(withCause)); assertEquals(1, ExceptionUtils.getThrowableCount(jdkNoCause)); - assertEquals(1, ExceptionUtils.getThrowableCount(selfCause)); assertEquals(3, ExceptionUtils.getThrowableCount(cyclicCause)); } @@ -253,12 +246,6 @@ public void testGetThrowables_Throwable_jdkNoCause() { assertSame(jdkNoCause, throwables[0]); } - public void testGetThrowables_Throwable_selfCause() { - Throwable[] throwables = ExceptionUtils.getThrowables(selfCause); - assertEquals(1, throwables.length); - assertSame(selfCause, throwables[0]); - } - public void testGetThrowables_Throwable_recursiveCause() { Throwable[] throwables = ExceptionUtils.getThrowables(cyclicCause); assertEquals(3, throwables.length); @@ -300,12 +287,6 @@ public void testGetThrowableList_Throwable_jdkNoCause() { assertSame(jdkNoCause, throwables.get(0)); } - public void testGetThrowableList_Throwable_selfCause() { - List throwables = ExceptionUtils.getThrowableList(selfCause); - assertEquals(1, throwables.size()); - assertSame(selfCause, throwables.get(0)); - } - public void testGetThrowableList_Throwable_recursiveCause() { List throwables = ExceptionUtils.getThrowableList(cyclicCause); assertEquals(3, throwables.size());