You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by ju...@apache.org on 2013/03/26 08:07:12 UTC

svn commit: r1460995 - in /jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api: SetPropertyStringTest.java SetPropertyValueTest.java

Author: jukka
Date: Tue Mar 26 07:07:11 2013
New Revision: 1460995

URL: http://svn.apache.org/r1460995
Log:
JCR-3543: TCK does not allow a property to be re-bound to a different definition

Remove too eager test cases

Modified:
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyStringTest.java
    jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyValueTest.java

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyStringTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyStringTest.java?rev=1460995&r1=1460994&r2=1460995&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyStringTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyStringTest.java Tue Mar 26 07:07:11 2013
@@ -307,25 +307,6 @@ public class SetPropertyStringTest exten
     }
 
     /**
-     * Tests if <code>Node.setProperty(String, String[])</code> throws a {@link
-     * javax.jcr.ValueFormatException} when trying to set an existing
-     * single-valued property to a multi-value
-     */
-    public void testSetSingleStringArrayValueFormatException() throws Exception {
-        // prerequisite: existing single-valued STRING property
-        if (!testNode.hasProperty(propertyName1)) {
-            testNode.setProperty(propertyName1, s1);
-            testNode.getParent().save();
-        }
-
-        try {
-            testNode.setProperty(propertyName1, sArray1);
-            fail("setProperty(singleValueProperty, String[]) not throwing a ValueFormatException");
-        } catch (ValueFormatException success) {
-        }
-    }
-
-    /**
      * Tests if removing a <code>String[]</code> property with
      * <code>Node.setProperty(String, null)</code> works with
      * <code>Session.save()</code>
@@ -421,25 +402,6 @@ public class SetPropertyStringTest exten
     }
 
     /**
-     * Tests if <code>Node.setProperty(String, String[], int)</code> throws a
-     * {@link javax.jcr.ValueFormatException} when trying to set an existing
-     * single-value property to a multi-value
-     */
-    public void testSetSingleStringArrayValueFormatExceptionWithPropertyType() throws Exception {
-        // prerequisite: existing single-valued STRING property
-        if (!testNode.hasProperty(propertyName1)) {
-            testNode.setProperty(propertyName1, s1);
-            testNode.getParent().save();
-        }
-
-        try {
-            testNode.setProperty(propertyName1, sArray1, PropertyType.STRING);
-            fail("setProperty(singleValueProperty, String[], int) not throwing a ValueFormatException");
-        } catch (ValueFormatException success) {
-        }
-    }
-
-    /**
      * Tests if removing a <code>String[]</code> property with
      * <code>Node.setProperty(String, null, int)</code> works with
      * <code>Session.save()</code>

Modified: jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyValueTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyValueTest.java?rev=1460995&r1=1460994&r2=1460995&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyValueTest.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-tests/src/main/java/org/apache/jackrabbit/test/api/SetPropertyValueTest.java Tue Mar 26 07:07:11 2013
@@ -332,25 +332,6 @@ public class SetPropertyValueTest extend
     }
 
     /**
-     * Tests if <code>Node.setProperty(String, Value[])</code> throws a {@link
-     * javax.jcr.ValueFormatException} when trying to set an existing
-     * single-valued property to a multi-value
-     */
-    public void testSetSingleValueArrayValueFormatException() throws Exception {
-        // prerequisite: existing single-valued property
-        if (!testNode.hasProperty(propertyName1)) {
-            testNode.setProperty(propertyName1, v1);
-            testNode.getParent().save();
-        }
-
-        try {
-            testNode.setProperty(propertyName1, vArray1);
-            fail("setProperty(singleValueProperty, Value[]) not throwing a ValueFormatException");
-        } catch (ValueFormatException success) {
-        }
-    }
-
-    /**
      * Tests if removing a <code>Value[]</code> property with
      * <code>Node.setProperty(String, null)</code> works with
      * <code>Session.save()</code>
@@ -472,25 +453,6 @@ public class SetPropertyValueTest extend
     }
 
     /**
-     * Tests if <code>Node.setProperty(String, Value[], int)</code> throws a
-     * {@link javax.jcr.ValueFormatException} when trying to set an existing
-     * single-valued property to a multi-value
-     */
-    public void testSetSingleValueArrayValueFormatExceptionWithPropertyType() throws Exception {
-        // prerequisite: existing single-valued property
-        if (!testNode.hasProperty(propertyName1)) {
-            testNode.setProperty(propertyName1, v1);
-            testNode.getParent().save();
-        }
-
-        try {
-            testNode.setProperty(propertyName1, vArray1, PropertyType.STRING);
-            fail("setProperty(singleValueProperty, Value[], int) not throwing a ValueFormatException");
-        } catch (ValueFormatException success) {
-        }
-    }
-
-    /**
      * Tests if removing a <code>Value[]</code> property with
      * <code>Node.setProperty(String, null, int)</code> works with
      * <code>Session.save()</code>