1 files changed, 48 insertions, 9 deletions

diff --git a/tests/unit/db/ItemMapperTest.php b/tests/unit/db/ItemMapperTest.php
index 26c9074dd..adb1dfe7a 100644
--- a/tests/unit/db/ItemMapperTest.php
+++ b/tests/unit/db/ItemMapperTest.php
@@ -236,7 +236,7 @@ class ItemMapperTest extends  \OCA\News\Tests\Unit\Db\MapperTestUtility {
             'AND `items`.`id` < ? ';
         $sql = $this->makeSelectQueryStatus($sql, $this->status);
         $params = [$this->user, $this->id, $this->offset];
-        $this->setMapperResult($sql, $params, $this->rows);
+        $this->setMapperResult($sql, $params, $this->rows, $this->limit);
         $result = $this->mapper->findAllFeed($this->id, $this->limit,
                 $this->offset, $this->status, false, $this->user);
 
@@ -244,12 +244,25 @@ class ItemMapperTest extends  \OCA\News\Tests\Unit\Db\MapperTestUtility {
     }
 
 
+    public function testFindAllFeedNegativeLimit(){
+        $sql = 'AND `items`.`feed_id` = ? ' .
+            'AND `items`.`id` < ? ';
+        $sql = $this->makeSelectQueryStatus($sql, $this->status);
+        $params = [$this->user, $this->id, $this->offset];
+        $this->setMapperResult($sql, $params, $this->rows);
+        $result = $this->mapper->findAllFeed($this->id, -1,
+                $this->offset, $this->status, false, $this->user);
+
+        $this->assertEquals($this->items, $result);
+    }
+
+
     public function testFindAllFeedOldestFirst(){
         $sql = 'AND `items`.`feed_id` = ? ' .
             'AND `items`.`id` > ? ';
         $sql = $this->makeSelectQueryStatus($sql, $this->status, true);
         $params = [$this->user, $this->id, $this->offset];
-        $this->setMapperResult($sql, $params, $this->rows);
+        $this->setMapperResult($sql, $params, $this->rows, $this->limit);
         $result = $this->mapper->findAllFeed($this->id, $this->limit,
                 $this->offset, $this->status, true, $this->user);
 
@@ -261,7 +274,7 @@ class ItemMapperTest extends  \OCA\News\Tests\Unit\Db\MapperTestUtility {
         $sql = 'AND `items`.`feed_id` = ? ';
         $sql = $this->makeSelectQueryStatus($sql, $this->status);
         $params = [$this->user, $this->id];
-        $this->setMapperResult($sql, $params, $this->rows);
+        $this->setMapperResult($sql, $params, $this->rows, $this->limit);
         $result = $this->mapper->findAllFeed($this->id, $this->limit,
                 0, $this->status, false, $this->user);
 
@@ -274,7 +287,7 @@ class ItemMapperTest extends  \OCA\News\Tests\Unit\Db\MapperTestUtility {
             'AND `items`.`id` < ? ';
         $sql = $this->makeSelectQueryStatus($sql, $this->status);
         $params = [$this->user, $this->id, $this->offset];
-        $this->setMapperResult($sql, $params, $this->rows);
+        $this->setMapperResult($sql, $params, $this->rows, $this->limit);
         $result = $this->mapper->findAllFolder($this->id, $this->limit,
                 $this->offset, $this->status, false, $this->user);
 
@@ -282,12 +295,26 @@ class ItemMapperTest extends  \OCA\News\Tests\Unit\Db\MapperTestUtility {
     }
 
 
+    public function testFindAllFolderNegativeLimit(){
+        $sql = 'AND `feeds`.`folder_id` = ? ' .
+            'AND `items`.`id` < ? ';
+        $sql = $this->makeSelectQueryStatus($sql, $this->status);
+        $params = [$this->user, $this->id, $this->offset];
+        $this->setMapperResult($sql, $params, $this->rows);
+        $result = $this->mapper->findAllFolder($this->id, -1,
+                $this->offset, $this->status, false, $this->user);
+
+        $this->assertEquals($this->items, $result);
+    }
+
+
+
     public function testFindAllFolderOldestFirst(){
         $sql = 'AND `feeds`.`folder_id` = ? ' .
             'AND `items`.`id` > ? ';
         $sql = $this->makeSelectQueryStatus($sql, $this->status, true);
         $params = [$this->user, $this->id, $this->offset];
-        $this->setMapperResult($sql, $params, $this->rows);
+        $this->setMapperResult($sql, $params, $this->rows, $this->limit);
         $result = $this->mapper->findAllFolder($this->id, $this->limit,
                 $this->offset, $this->status, true, $this->user);
 
@@ -299,7 +326,7 @@ class ItemMapperTest extends  \OCA\News\Tests\Unit\Db\MapperTestUtility {
         $sql = 'AND `feeds`.`folder_id` = ? ';
         $sql = $this->makeSelectQueryStatus($sql, $this->status);
         $params = [$this->user, $this->id];
-        $this->setMapperResult($sql, $params, $this->rows);
+        $this->setMapperResult($sql, $params, $this->rows, $this->limit);
         $result = $this->mapper->findAllFolder($this->id, $this->limit,
                 0, $this->status, false, $this->user);
 
@@ -311,7 +338,7 @@ class ItemMapperTest extends  \OCA\News\Tests\Unit\Db\MapperTestUtility {
         $sql = 'AND `items`.`id` < ? ';
         $sql = $this->makeSelectQueryStatus($sql, $this->status);
         $params = [$this->user, $this->offset];
-        $this->setMapperResult($sql, $params, $this->rows);
+        $this->setMapperResult($sql, $params, $this->rows, $this->limit);
         $result = $this->mapper->findAll($this->limit,
                 $this->offset, $this->status, false, $this->user);
 
@@ -319,11 +346,23 @@ class ItemMapperTest extends  \OCA\News\Tests\Unit\Db\MapperTestUtility {
     }
 
 
+    public function testFindAllNegativeLimit(){
+        $sql = 'AND `items`.`id` < ? ';
+        $sql = $this->makeSelectQueryStatus($sql, $this->status);
+        $params = [$this->user, $this->offset];
+        $this->setMapperResult($sql, $params, $this->rows, null);
+        $result = $this->mapper->findAll(-1,
+                $this->offset, $this->status, false, $this->user);
+
+        $this->assertEquals($this->items, $result);
+    }
+
+
     public function testFindAllOldestFirst(){
         $sql = 'AND `items`.`id` > ? ';
         $sql = $this->makeSelectQueryStatus($sql, $this->status, true);
         $params = [$this->user, $this->offset];
-        $this->setMapperResult($sql, $params, $this->rows);
+        $this->setMapperResult($sql, $params, $this->rows, $this->limit);
         $result = $this->mapper->findAll($this->limit,
                 $this->offset, $this->status, true, $this->user);
 
@@ -334,7 +373,7 @@ class ItemMapperTest extends  \OCA\News\Tests\Unit\Db\MapperTestUtility {
     public function testFindAllOffsetZero(){
         $sql = $this->makeSelectQueryStatus('', $this->status);
         $params = [$this->user];
-        $this->setMapperResult($sql, $params, $this->rows);
+        $this->setMapperResult($sql, $params, $this->rows, $this->limit);
         $result = $this->mapper->findAll($this->limit,
                 0, $this->status, false, $this->user);