Replace getListSize with getList

This is not so performant, but as discussed in github it'll do for now.
magento · Dec 4, 2017 · 10df2ce · 10df2ce
1 parent ca50c9d
commit 10df2ce
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/app/code/Magento/Indexer/Console/Command/IndexerStatusCommand.php b/app/code/Magento/Indexer/Console/Command/IndexerStatusCommand.php
@@ -103,7 +103,7 @@ private function getPendingCount(Mview\ViewInterface $view)
 
         $state = $view->getState();
 
-        $pendingCount = $changelog->getListSize($state->getVersionId(), $currentVersionId);
+        $pendingCount = count($changelog->getList($state->getVersionId(), $currentVersionId));
 
         $pendingString = "<error>$pendingCount</error>";
         if ($pendingCount <= 0) {

diff --git a/app/code/Magento/Indexer/Test/Unit/Console/Command/IndexerStatusCommandTest.php b/app/code/Magento/Indexer/Test/Unit/Console/Command/IndexerStatusCommandTest.php
@@ -27,14 +27,14 @@ class IndexerStatusCommandTest extends AbstractIndexerCommandCommonSetup
      */
     private function attachViewToIndexerMock($indexerMock, array $data)
     {
-         /** @var \Magento\Framework\Mview\View\Changelog|\PHPUnit_Framework_MockObject_MockObject $stub */
+         /** @var \Magento\Framework\Mview\View\Changelog|\PHPUnit_Framework_MockObject_MockObject $changelog */
         $changelog = $this->getMockBuilder(\Magento\Framework\Mview\View\Changelog::class)
             ->disableOriginalConstructor()
             ->getMock();
 
         $changelog->expects($this->any())
-            ->method('getListSize')
-            ->willReturn($data['view']['changelog']['list_size']);
+            ->method('getList')
+            ->willReturn(range(0, $data['view']['changelog']['list_size']-1));
 
         /** @var \Magento\Indexer\Model\Mview\View\State|\PHPUnit_Framework_MockObject_MockObject $stateMock */
         $stateMock = $this->getMockBuilder(\Magento\Indexer\Model\Mview\View\State::class)