Tfboe\FmLib\Tests\Unit\Service\RankingSystem\EloRankingTest::testGetChanges()

Complexity

Conditions	10
Paths	144

Size

Total Lines	68
Code Lines	45

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
  public function testGetChanges(bool $isPlayed, int $gameResult, array $playerInfos)
  {
    $repository = $this->createStub(ObjectRepository::class, ['findBy' => []]);
    /** @var EntityManagerInterface $entityManager */
    $entityManager = $this->createStub(EntityManagerInterface::class, ['getRepository' => $repository]);
    $service = $this->service($entityManager, $this->getObjectCreator());
    /** @var EloRanking $player1 */

    /** @var Player[] $players */
    $players = [];
    $playersAArray = [];
    $playersBArray = [];
    for ($i = 0; $i < count($playerInfos); $i++) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

      $players[$i] = $this->createStub(Player::class, ['getId' => $i]);
      if ($i < count($playerInfos) / 2) {
        $playersAArray[] = $players[$i];
      } else {
        $playersBArray[] = $players[$i];
      }
    }

    $playersA = new ArrayCollection($playersAArray);
    $playersB = new ArrayCollection($playersBArray);
    $game = $this->createStub(Game::class, [
      'getPlayersA' => $playersA,
      'getPlayersB' => $playersB,
    ]);

    $entriesArray = [];
    for ($i = 0; $i < count($playerInfos); $i++) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

      $entriesArray[$i] = $this->getRankingSystemListEntry($service, $players[$i]);
    }
    $entries = new ArrayCollection($entriesArray);
    $list = $this->createStub(RankingSystemList::class, ['getEntries' => $entries]);
    /** @var RankingSystemListInterface $list */
    foreach ($entries as $entry) {
      /** @var RankingSystemListEntryInterface $entry */
      $entry->setRankingSystemList($list);
    }
    for ($i = 0; $i < count($playerInfos); $i++) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

      $info = $playerInfos[$i];
      $entry = $list->getEntries()[$i];
      $entry->setPoints($info['points']);
      $entry->setPlayedGames($info['played']);
      $entry->setNumberRankedEntities($info['ranked']);
      $entry->setRatedGames($info['rated']);
      if (array_key_exists('provisoryRanking', $info)) {
        $list->getEntries()[$i]->setProvisoryRanking($info['provisoryRanking']);
      }
    }
    $game->method('isPlayed')->willReturn($isPlayed);
    $game->method('getResult')->willReturn($gameResult);

    /** @var RankingSystemChangeInterface[] $changes */
    $changes = static::callProtectedMethod($service, 'getChanges', [$game, $list]);
    self::assertEquals(count($playerInfos), count($changes));
    foreach ($players as $player) {
      $exists = false;
      foreach ($changes as $change) {
        if ($change->getPlayer() === $player) {
          $exists = true;
          break;
        }
      }
      self::assertTrue($exists);
    }
    /** @var Game $game */
    $this->assertChanges($changes, $playerInfos, $game);
  }
}