You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by st...@apache.org on 2006/03/10 15:13:19 UTC

svn commit: r384798 - /incubator/jackrabbit/trunk/jackrabbit/src/test/java/org/apache/jackrabbit/test/api/ExportDocViewTest.java

Author: stefan
Date: Fri Mar 10 06:13:17 2006
New Revision: 384798

URL: http://svn.apache.org/viewcvs?rev=384798&view=rev
Log:
fixing document view export test case: a repository that supports multi-value property serialization is *not* required to escape space characters in single value properties.

Modified:
    incubator/jackrabbit/trunk/jackrabbit/src/test/java/org/apache/jackrabbit/test/api/ExportDocViewTest.java

Modified: incubator/jackrabbit/trunk/jackrabbit/src/test/java/org/apache/jackrabbit/test/api/ExportDocViewTest.java
URL: http://svn.apache.org/viewcvs/incubator/jackrabbit/trunk/jackrabbit/src/test/java/org/apache/jackrabbit/test/api/ExportDocViewTest.java?rev=384798&r1=384797&r2=384798&view=diff
==============================================================================
--- incubator/jackrabbit/trunk/jackrabbit/src/test/java/org/apache/jackrabbit/test/api/ExportDocViewTest.java (original)
+++ incubator/jackrabbit/trunk/jackrabbit/src/test/java/org/apache/jackrabbit/test/api/ExportDocViewTest.java Fri Mar 10 06:13:17 2006
@@ -546,16 +546,6 @@
                 }
             } else {
                 val = prop.getString();
-                if (exportMultivalProps) {
-                    val = escapeValues(val);
-                } else {
-                    // we could not decide if the repository exports multivalued
-                    // properties so we consider both possibilities
-                    String escapedVal = escapeValues(val);
-                    if (escapedVal.equals(attrVal)) {
-                        val = escapedVal;
-                    }
-                }
             }
         }
         if (isBinary && skipBinary) {
@@ -565,8 +555,9 @@
                     " exported although skipBinary is true",
                     "", attrVal);
         } else {
-            assertEquals("Value of property " + prop.getPath() +
-                    " is not exported correctly: ", val, attrVal);
+            assertTrue("Value of property " + prop.getPath() +
+                    " is not exported correctly: " + attrVal,
+                    val.equals(attrVal) || escapeValues(val).equals(attrVal));
         }
     }