@@ -454,12 +454,10 @@ function () use (&$indexers) {
454454 public function testReindexAllWithSharedIndexers ()
455455 {
456456 $ indexId = 'indexer_internal_name ' ;
457-
458457 // Ensure dependencies considered up-to-date so reset flag is true
459458 $ this ->dependencyInfoProviderMock
460459 ->method ('getIndexerIdsToRunBefore ' )
461460 ->willReturn ([]);
462-
463461 // Configure current indexer config to have a shared_index
464462 $ sharedIndex = 'shared_idx ' ;
465463 $ this ->configMock
@@ -482,7 +480,6 @@ public function testReindexAllWithSharedIndexers()
482480 default => ['shared_index ' => null ],
483481 };
484482 });
485-
486483 // Provide available indexers map for getSharedIndexers() iteration
487484 $ this ->configMock
488485 ->method ('getIndexers ' )
@@ -492,7 +489,6 @@ public function testReindexAllWithSharedIndexers()
492489 'indexer_b ' => [],
493490 'indexer_c ' => [],
494491 ]);
495-
496492 // Prepare shared indexer mocks returned by factory
497493 $ sharedIndexerA = $ this ->createMock (Indexer::class);
498494 $ sharedViewA = $ this ->createMock (ViewInterface::class);
@@ -514,7 +510,6 @@ public function testReindexAllWithSharedIndexers()
514510 ->expects ($ this ->exactly (2 ))
515511 ->method ('create ' )
516512 ->willReturnOnConsecutiveCalls ($ sharedIndexerA , $ sharedIndexerB );
517-
518513 // Model state expectations
519514 $ this ->workingStateProvider ->method ('isWorking ' )->willReturnOnConsecutiveCalls (false , true );
520515 $ stateMock = $ this ->createPartialMock (
@@ -529,20 +524,17 @@ public function testReindexAllWithSharedIndexers()
529524 $ stateMock ->expects ($ this ->any ())->method ('getStatus ' )->willReturn ('idle ' );
530525 $ stateMock ->expects ($ this ->exactly (3 ))->method ('save ' )->willReturnSelf ();
531526 $ this ->stateFactoryMock ->expects ($ this ->once ())->method ('create ' )->willReturn ($ stateMock );
532-
533527 // Current indexer view
534528 $ this ->viewMock ->expects ($ this ->once ())->method ('isEnabled ' )->willReturn (true );
535529 $ this ->viewMock ->expects ($ this ->once ())->method ('suspend ' );
536530 $ this ->viewMock ->expects ($ this ->once ())->method ('resume ' );
537-
538531 // Action execution
539532 $ actionMock = $ this ->createPartialMock (
540533 ActionInterface::class,
541534 ['executeFull ' , 'executeList ' , 'executeRow ' ]
542535 );
543536 $ this ->actionFactoryMock ->expects ($ this ->once ())->method ('create ' )->with ('Some \\Class \\Name ' )
544537 ->willReturn ($ actionMock );
545-
546538 // Prepare model with ID set and required data
547539 $ this ->model ->setId ($ indexId );
548540 $ this ->model ->setData ([
0 commit comments