[dna-commits] DNA SVN: r817 - in trunk/dna-jcr/src: test/java/org/jboss/dna/jcr and 1 other directory.

dna-commits at lists.jboss.org dna-commits at lists.jboss.org
Mon Apr 13 15:23:32 EDT 2009


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();
     }




More information about the dna-commits mailing list