You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@abdera.apache.org by da...@apache.org on 2008/01/04 00:11:34 UTC

svn commit: r608670 - /incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java

Author: dandiep
Date: Thu Jan  3 15:11:31 2008
New Revision: 608670

URL: http://svn.apache.org/viewvc?rev=608670&view=rev
Log:
Fix JCR test. (seems Sanitizer changed? Why isn't it respecting my 'filler' argument anymore?)

Modified:
    incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java

Modified: incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java
URL: http://svn.apache.org/viewvc/incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java?rev=608670&r1=608669&r2=608670&view=diff
==============================================================================
--- incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java (original)
+++ incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java Thu Jan  3 15:11:31 2008
@@ -108,7 +108,7 @@
     //prettyPrint(abdera, res.getDocument());
 
     IRI location = res.getLocation();
-    assertEquals(colUri + "/Some-Entry", location.toString());
+    assertEquals(colUri + "/Some_Entry", location.toString());
 
     // GET the entry
     res = client.get(location.toString());



Re: svn commit: r608670 - /incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java

Posted by Dan Diephouse <da...@mulesource.com>.
OK that makes sense. No worries!

James M Snell wrote:
> Sorry, I'd forgotten about this one.  For space characters, the
> Sanitizer now inserts _'s. For everything else it uses the filler
> character.  The reason for the change was to simplify the case when the
> only thing that really needs replaced are the spaces... If that causes a
> problem for you, let me know and I'll change it back.
>
> - James
>
> dandiep@apache.org wrote:
>   
>> Author: dandiep
>> Date: Thu Jan  3 15:11:31 2008
>> New Revision: 608670
>>
>> URL: http://svn.apache.org/viewvc?rev=608670&view=rev
>> Log:
>> Fix JCR test. (seems Sanitizer changed? Why isn't it respecting my 'filler' argument anymore?)
>>
>> Modified:
>>     incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java
>>
>> Modified: incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java
>> URL: http://svn.apache.org/viewvc/incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java?rev=608670&r1=608669&r2=608670&view=diff
>> ==============================================================================
>> --- incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java (original)
>> +++ incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java Thu Jan  3 15:11:31 2008
>> @@ -108,7 +108,7 @@
>>      //prettyPrint(abdera, res.getDocument());
>>  
>>      IRI location = res.getLocation();
>> -    assertEquals(colUri + "/Some-Entry", location.toString());
>> +    assertEquals(colUri + "/Some_Entry", location.toString());
>>  
>>      // GET the entry
>>      res = client.get(location.toString());
>>
>>
>>
>>     


-- 
Dan Diephouse
MuleSource
http://mulesource.com | http://netzooid.com/blog


Re: svn commit: r608670 - /incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java

Posted by James M Snell <ja...@gmail.com>.
Sorry, I'd forgotten about this one.  For space characters, the
Sanitizer now inserts _'s. For everything else it uses the filler
character.  The reason for the change was to simplify the case when the
only thing that really needs replaced are the spaces... If that causes a
problem for you, let me know and I'll change it back.

- James

dandiep@apache.org wrote:
> Author: dandiep
> Date: Thu Jan  3 15:11:31 2008
> New Revision: 608670
> 
> URL: http://svn.apache.org/viewvc?rev=608670&view=rev
> Log:
> Fix JCR test. (seems Sanitizer changed? Why isn't it respecting my 'filler' argument anymore?)
> 
> Modified:
>     incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java
> 
> Modified: incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java
> URL: http://svn.apache.org/viewvc/incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java?rev=608670&r1=608669&r2=608670&view=diff
> ==============================================================================
> --- incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java (original)
> +++ incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java Thu Jan  3 15:11:31 2008
> @@ -108,7 +108,7 @@
>      //prettyPrint(abdera, res.getDocument());
>  
>      IRI location = res.getLocation();
> -    assertEquals(colUri + "/Some-Entry", location.toString());
> +    assertEquals(colUri + "/Some_Entry", location.toString());
>  
>      // GET the entry
>      res = client.get(location.toString());
> 
> 
> 

Re: svn commit: r608670 - /incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java

Posted by James M Snell <ja...@gmail.com>.
Sorry, I'd forgotten about this one.  For space characters, the
Sanitizer now inserts _'s. For everything else it uses the filler
character.  The reason for the change was to simplify the case when the
only thing that really needs replaced are the spaces... If that causes a
problem for you, let me know and I'll change it back.

- James

dandiep@apache.org wrote:
> Author: dandiep
> Date: Thu Jan  3 15:11:31 2008
> New Revision: 608670
> 
> URL: http://svn.apache.org/viewvc?rev=608670&view=rev
> Log:
> Fix JCR test. (seems Sanitizer changed? Why isn't it respecting my 'filler' argument anymore?)
> 
> Modified:
>     incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java
> 
> Modified: incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java
> URL: http://svn.apache.org/viewvc/incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java?rev=608670&r1=608669&r2=608670&view=diff
> ==============================================================================
> --- incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java (original)
> +++ incubator/abdera/java/trunk/jcr/src/test/java/org/apache/abdera/jcr/JcrCollectionProviderTest.java Thu Jan  3 15:11:31 2008
> @@ -108,7 +108,7 @@
>      //prettyPrint(abdera, res.getDocument());
>  
>      IRI location = res.getLocation();
> -    assertEquals(colUri + "/Some-Entry", location.toString());
> +    assertEquals(colUri + "/Some_Entry", location.toString());
>  
>      // GET the entry
>      res = client.get(location.toString());
> 
> 
>