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比赛进行比较。

在我的测试课中，我不能“ - ＆gt; 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并使用它而不是create new Team()，这样你就可以控制你的类了，因为你没有测试实体类而是函数。

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

Answer 2

不要在测试中使用new User()，而应该模拟这些对象。然后，您可以模拟此处的任何方法，因此您无需使用$user->setId()，因为您将能够获得$userMock->method('getId')->willReturn(10)。

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

请参阅https://phpunit.de/manual/current/en/test-doubles.html以获取参考资料。

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

2 个答案: