You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@jackrabbit.apache.org by al...@apache.org on 2010/07/22 14:46:57 UTC

svn commit: r966626 - /jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/query/LimitAndOffsetTest.java

Author: alexkli
Date: Thu Jul 22 12:46:57 2010
New Revision: 966626

URL: http://svn.apache.org/viewvc?rev=966626&view=rev
Log:
JCR-2684: Setting Query.setOffset() passed the results total returns negative getSize() instead of zero
- adding no-nodes-found-check to test case

Modified:
    jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/query/LimitAndOffsetTest.java

Modified: jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/query/LimitAndOffsetTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/query/LimitAndOffsetTest.java?rev=966626&r1=966625&r2=966626&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/query/LimitAndOffsetTest.java (original)
+++ jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/query/LimitAndOffsetTest.java Thu Jul 22 12:46:57 2010
@@ -142,6 +142,7 @@ public class LimitAndOffsetTest extends 
         query.setOffset(10);
         result = query.execute();
         nodes = result.getNodes();
+        assertFalse(nodes.hasNext());
         assertEquals(0, nodes.getSize());
         assertEquals(3, ((QueryResultImpl) result).getTotalSize());