Author: rhauch
Date: 2009-04-13 15:23:31 -0400 (Mon, 13 Apr 2009)
New Revision: 817
Modified:
trunk/dna-jcr/src/main/java/org/jboss/dna/jcr/AbstractJcrNode.java
trunk/dna-jcr/src/test/java/org/jboss/dna/jcr/AbstractJcrNodeTest.java
Log:
DNA-348 AbstractJcrNode.checkin and .checkout Throws the Wrong Exception Type
Applied the patch as is, which corrects the exception that is thrown from the methods.
Modified: trunk/dna-jcr/src/main/java/org/jboss/dna/jcr/AbstractJcrNode.java
===================================================================
--- trunk/dna-jcr/src/main/java/org/jboss/dna/jcr/AbstractJcrNode.java 2009-04-13 19:20:51
UTC (rev 816)
+++ trunk/dna-jcr/src/main/java/org/jboss/dna/jcr/AbstractJcrNode.java 2009-04-13 19:23:31
UTC (rev 817)
@@ -1341,21 +1341,21 @@
/**
* {@inheritDoc}
*
- * @throws UnsupportedOperationException always
+ * @throws UnsupportedRepositoryOperationException always
* @see javax.jcr.Node#checkin()
*/
- public final Version checkin() {
- throw new UnsupportedOperationException();
+ public final Version checkin() throws UnsupportedRepositoryOperationException {
+ throw new UnsupportedRepositoryOperationException();
}
/**
* {@inheritDoc}
*
- * @throws UnsupportedOperationException always
+ * @throws UnsupportedRepositoryOperationException always
* @see javax.jcr.Node#checkout()
*/
- public final void checkout() {
- throw new UnsupportedOperationException();
+ public final void checkout() throws UnsupportedRepositoryOperationException {
+ throw new UnsupportedRepositoryOperationException();
}
/**
Modified: trunk/dna-jcr/src/test/java/org/jboss/dna/jcr/AbstractJcrNodeTest.java
===================================================================
--- trunk/dna-jcr/src/test/java/org/jboss/dna/jcr/AbstractJcrNodeTest.java 2009-04-13
19:20:51 UTC (rev 816)
+++ trunk/dna-jcr/src/test/java/org/jboss/dna/jcr/AbstractJcrNodeTest.java 2009-04-13
19:23:31 UTC (rev 817)
@@ -239,12 +239,12 @@
node.cancelMerge(null);
}
- @Test( expected = UnsupportedOperationException.class )
+ @Test( expected = UnsupportedRepositoryOperationException.class )
public void shoudNotAllowCheckin() throws Exception {
node.checkin();
}
- @Test( expected = UnsupportedOperationException.class )
+ @Test( expected = UnsupportedRepositoryOperationException.class )
public void shoudNotAllowCheckout() throws Exception {
node.checkout();
}
Show replies by date