在单元测试期间根据实体 ID 比较实体

Question

我正在尝试为我的应用程序创建单元测试，但我想获得一些建议。

我有这些方法：

/**
 * This methods check if the user can apply to the team, it search if he have a username for the game and if he doesn't already applied for another team in the same tournament
 * @param User $user
 * @param Team $team
 * @return bool|string
 */
public function canApply(User $user, Team $team) {
    if ($user->getGamingUsername($team->getTournament()->getGame()) === null) {
        return "Vous devez avoir un nom d'utilisateur pour pouvoir vous inscrire, renseignez le dans \"Mon profil\"";
    } else if (false !== $teamAlreadyIn = $this->isAlreadyApplicant($user, $team)) {
        return "Vous avez déjà postulé pour une équipe pour ce tournoi : 
        <a href=\"".$this->router->generate("mgd_team_show", array("id" => $teamAlreadyIn->getId()))."\">".htmlspecialchars($teamAlreadyIn->getName())."</a>";
    }
    return true;
}

/**
 * This method search if the user is already in a team for the same tournament than the one passed in argument
 * @param User $user
 * @param Team $team
 * @return bool|Team|mixed
 */
public function isAlreadyApplicant($user, Team $team) {
    if (!$user || !$this->authorizationChecker->isGranted("ROLE_USER")) {
        return false;
    }

    foreach ($user->getApplications() as $userTeam) {
        /** @var Team $userTeam */
        if ($userTeam->getTournament()->getId() === $team->getTournament()->getId()) {
            return $userTeam;
        }
    }

    foreach ($user->getTeams() as $userTeam) {
        /** @var Team $userTeam */
        if ($userTeam->getTournament()->getId() === $team->getTournament()->getId()) {
            return $userTeam;
        }
    }

    foreach ($user->getManagedTeam() as $userTeam) {
        /** @var Team $userTeam */
        if ($userTeam->getTournament()->getId() === $team->getTournament()->getId()) {
            return $userTeam;
        }
    }
    return false;
}

如您所见，第一个 (canApply) 调用第二个 (isAlreadyApplicant)。

但是当我尝试测试 canApply 时，我遇到了一些麻烦：这个方法调用 isAlreadyApplicant，在这个方法中，我根据比赛的 id 进行比较。

在我的测试类中，我不能“->setId()”，因为它是一个私有方法。那我该怎么处理呢？ 从我的数据库中获取元素会更好吗？

目前，我的一个 testMethod 看起来像这样：

/**
 * The user is connected and try to apply to a team and applied for another team for another game
 */
public function testCanApplySecondTeamAnotherGame() {
    $user = new User();
    $game = new Game();
    $anotherGame = new Game();

    $team = new Team();
    $anotherTeam = new Team();

    $tournament = new TournamentTeam();
    $anotherTournament = new TournamentTeam();

    $team->setTournament($tournament);
    $anotherTeam->setTournament($anotherTournament);

    $tournament->setGame($game);
    $anotherTournament->setGame($anotherGame);

    $game->setName("TestGame");
    $anotherGame->setName("AnotherGame");

    $gamingProfile = new GamingProfile();
    $gamingProfile->setGame($game);
    $gamingProfile->setUsername("TestGameUsername");

    $anotherGamingProfile = new GamingProfile();
    $anotherGamingProfile->setGame($anotherGame);
    $anotherGamingProfile->setUsername("TestAnotherGameUsername");

    $user->setGamingProfiles(new ArrayCollection(array($gamingProfile, $anotherGamingProfile)));

    $user->addApplication($anotherTeam);
    $user->addTeam($anotherTeam);

    $router = $this->createMock(Router::class);
    $authorizationChecker = $this->createMock(AuthorizationCheckerInterface::class);
    $authorizationChecker->method("isGranted")->willReturn(true);

    $applicationChecker = new ApplicationChecker($router, $authorizationChecker);
    //Here, we try to apply to a team
    $this->assertTrue($applicationChecker->canApply($user, $team));

}

如果您有任何问题，请不要犹豫！ 祝你有美好的一天！

Answer 1

这段代码耦合度更高，很难以简单的方式进行测试。

我建议你采取方法isAlreadyApplicant 并进入服务例如。

在依赖注入之后将新服务注入到你的实际服务中（我希望是一个服务）。

Dependency Injection Documentation

所以现在您可以使用方法isAlreadyApplicant 模拟新服务。

这是用于单元测试的。

---

如果您需要功能测试，您可以创建一个类，设置 id 并使用它而不是创建 new Team()，这样您就可以控制您的类，因为您测试的不是实体类而是函数。

---

另一种解决方案是模拟库以将私有属性设置到您的测试中，但我不喜欢这种解决方案

Answer 2

您应该模拟这些对象，而不是在测试中使用new User()。然后你可以在其中模拟任何方法，所以你不需要使用$user->setId()，因为你可以得到$userMock->method('getId')->willReturn(10)。

因此，与其使用 setter 实例化新对象和设置值，不如模拟这些对象并存根 getter。

参考https://phpunit.de/manual/current/en/test-doubles.html。