commit | 80a8633c00738d0aa6badaf5eb5af5612e4c6638 | [log] [tgz] |
---|---|---|
author | Vamsavardhana Chillakuru <vamsic007@apache.org> | Sat Dec 09 08:33:52 2006 +0000 |
committer | Vamsavardhana Chillakuru <vamsic007@apache.org> | Sat Dec 09 08:33:52 2006 +0000 |
tree | 20de411eadad8b3a5053025c27f91c80b5130128 | |
parent | 65d24d2c29bb486ac06da5e3794a81ad4b0a6621 [diff] |
GERONIMO-1519 ResourceException.toString() can return null o The method seems to have been introduced to account for a bad testcase. Removed ResourceException.toString() and the corresponding test ResourceExceptionTest.testToString() o For a discussion on this, see http://www.mail-archive.com/dev%40geronimo.apache.org/msg38005.html git-svn-id: https://svn.apache.org/repos/asf/geronimo/specs/branches/1_0@484948 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/geronimo-spec-j2ee-connector/src/java/javax/resource/ResourceException.java b/geronimo-spec-j2ee-connector/src/java/javax/resource/ResourceException.java index 8ad6fc9..bcbe6bf 100644 --- a/geronimo-spec-j2ee-connector/src/java/javax/resource/ResourceException.java +++ b/geronimo-spec-j2ee-connector/src/java/javax/resource/ResourceException.java
@@ -76,9 +76,4 @@ // unit tests revealed that Throwable.initCause is not invoked this.linkedException = ex; } - - public String toString() { - // unit tests revealed that the errorCode is not included - return getMessage(); - } }
diff --git a/geronimo-spec-j2ee-connector/src/test/javax/resource/ResourceExceptionTest.java b/geronimo-spec-j2ee-connector/src/test/javax/resource/ResourceExceptionTest.java index 7861caf..4a757e4 100644 --- a/geronimo-spec-j2ee-connector/src/test/javax/resource/ResourceExceptionTest.java +++ b/geronimo-spec-j2ee-connector/src/test/javax/resource/ResourceExceptionTest.java
@@ -51,12 +51,4 @@ exception.setLinkedException(root); assertSame(root, exception.getLinkedException()); } - - public void testToString() { - ResourceException exception = new ResourceException("problem"); - assertEquals("problem", exception.toString()); - - ResourceException other = new ResourceException("other problem", "123"); - assertEquals("other problem", other.toString()); - } }