Initialise hasNext to false if the initial collection is empty/null

Author: Ndiritu

src/Tasks/PageIterator.php

@@ -57,7 +57,7 @@ public function __construct($response, RequestAdapter $requestAdapter, ?array $c
 
         if ($page !== null) {
             $this->currentPage = $page;
-            $this->hasNext = true;
+            $this->hasNext = !(empty($page->getValue()) || is_null($page->getValue()));
         }
     }
 
@@ -137,7 +137,7 @@ public static function convertToPage($response): ?PageResult
 
         $value = null;
         if (is_array($response)) {
-            $value = $response['value'] ?? ['value' => []];
+            $value = $response['value'] ?? [];
         } elseif ($response instanceof Parsable &&
             method_exists($response, 'getValue')) {
             $value = $response->getValue();

tests/Tasks/PageIteratorTest.php

@@ -224,4 +224,47 @@ public function testCanGetFromParsablePageObject(): void
         $this->assertEquals(2, $count);
         $this->assertInstanceOf(User::class, $data[0]);
     }
+
+    public function testHasNextIsFalseWithEmptyOrNullInitialCollection(): void
+    {
+        $pageIterator = new PageIterator([], $this->mockRequestAdapter);
+        $this->assertFalse($pageIterator->hasNext());
+
+        $pageIterator = new PageIterator([1, 2, 3], $this->mockRequestAdapter);
+        $this->assertFalse($pageIterator->hasNext());
+
+        $pageIterator = new PageIterator(['value' => []], $this->mockRequestAdapter);
+        $this->assertFalse($pageIterator->hasNext());
+
+        $pageIterator = new PageIterator(['value' => null], $this->mockRequestAdapter);
+        $this->assertFalse($pageIterator->hasNext());
+
+        $pageIterator = new PageIterator((object) [], $this->mockRequestAdapter);
+        $this->assertFalse($pageIterator->hasNext());
+
+        $pageIterator = new PageIterator((object) [1, 2], $this->mockRequestAdapter);
+        $this->assertFalse($pageIterator->hasNext());
+
+        $pageIterator = new PageIterator((object) [], $this->mockRequestAdapter);
+        $this->assertFalse($pageIterator->hasNext());
+
+        $usersResponse = new UsersResponse();
+        $usersResponse->setValue([]);
+        $pageIterator = new PageIterator($usersResponse, $this->mockRequestAdapter);
+        $this->assertFalse($pageIterator->hasNext());
+    }
+
+    public function testHasNextInitialisedToTrueWhenValueHasItems(): void
+    {
+        $pageIterator = new PageIterator(['value' => [1, 2, 3]], $this->mockRequestAdapter);
+        $this->assertTrue($pageIterator->hasNext());
+
+        $pageIterator = new PageIterator((object) ['value' => [1]], $this->mockRequestAdapter);
+        $this->assertTrue($pageIterator->hasNext());
+
+        $usersResponse = new UsersResponse();
+        $usersResponse->setValue([1, 2]);
+        $pageIterator = new PageIterator($usersResponse, $this->mockRequestAdapter);
+        $this->assertTrue($pageIterator->hasNext());
+    }
 }