Merge pull request #2061 from epourail/fix-totalitems-on-empty-collection

Initialize the 'totalCount' property to the default value to 0.0

Author: dunglas

src/GraphQl/Resolver/Factory/CollectionResolverFactory.php

@@ -107,7 +107,7 @@ public function __invoke(string $resourceClass = null, string $rootClass = null,
                 $offset = 1 + (int) $after;
             }
 
-            $data = ['edges' => [], 'pageInfo' => ['endCursor' => null, 'hasNextPage' => false]];
+            $data = ['totalCount' => 0.0, 'edges' => [], 'pageInfo' => ['endCursor' => null, 'hasNextPage' => false]];
             if ($collection instanceof PaginatorInterface && ($totalItems = $collection->getTotalItems()) > 0) {
                 $data['pageInfo']['endCursor'] = base64_encode((string) ($totalItems - 1));
                 $data['pageInfo']['hasNextPage'] = $collection->getCurrentPage() !== $collection->getLastPage() && (float) $collection->count() === $collection->getItemsPerPage();

tests/GraphQl/Resolver/Factory/CollectionResolverFactoryTest.php

@@ -54,7 +54,7 @@ public function testCreateCollectionResolverNoCollection(bool $paginationEnabled
     public function paginationProvider(): array
     {
         return [
-            [true, ['edges' => [], 'pageInfo' => ['endCursor' => null, 'hasNextPage' => false]]],
+            [true, ['totalCount' => 0.0, 'edges' => [], 'pageInfo' => ['endCursor' => null, 'hasNextPage' => false]]],
             [false, []],
         ];
     }
@@ -133,6 +133,7 @@ public function testCreateCollectionResolver(string $cursor, array $expectedCurs
 
         $this->assertEquals(
             [
+                'totalCount' => 0.0,
                 'edges' => [['node' => 'normalizedObject1', 'cursor' => $expectedCursors[0]], ['node' => 'normalizedObject2', 'cursor' => $expectedCursors[1]]],
                 'pageInfo' => ['endCursor' => null, 'hasNextPage' => false],
             ],