3

我在为它编写单元测试时重构一个类。有一种情况是我的一个方法完全调用了另一个对象的方法，该方法注入到我正在测试的这个类中。

所以我必须模拟我注入类的对象。

现在，问题是，是否值得为这个特定方法编写单元测试？编写单元测试似乎很奇怪，它调用其他对象的方法，该对象本身必须被模拟，那么我为什么要测试这个方法呢？

测试的目的不是检查方法的功能是否按预期工作吗？如果是，那么当我嘲笑它所拥有的一切并且该特定方法没有任何东西可以测试时，我为什么要测试？

我真的很困惑！

我坚持的方法是这个（用于自定义会话处理）：

public function write($sessionId, $sessionData)
{
    $sth = $this->databaseConnection->prepare("INSERT INTO `{$this->DBTableName}` (`session_id`,`session_name`,`session_data`) VALUES(:session_id, :session_name, :session_data) ON DUPLICATE KEY UPDATE `session_data`=:session_data;");
    $sth->bindValue(':session_id', $sessionId);
    $sth->bindValue(':session_name', $this->sessionName);
    $sth->bindValue(':session_data', $sessionData);

    return $sth->execute();
}

这里也是这段代码的链接：http: //pastebin.com/1FBeU6mb

顺便说一句，我刚开始为我的课程编写测试，我是这个测试领域的初学者并且没有经验。

提前致谢。

4

3 回答 3

2

我对 php 不是很熟悉，但看起来您只是在构建和执行数据库查询，对吧？

除非这个方法和数据库之间有我看不到的其他层，否则这里真的没有什么值得嘲笑的。所以你是对的，在这里嘲笑并没有给你很多价值。而且从某种意义上说，测试这种方法的价值是有限的，因为您实际上只是在测试数据库层，通常我们可以假设它已经正确且稳定，因此不需要测试。

一般来说，模拟的价值在于它允许您通过假设其他方法正在做什么来简化测试，并且允许您不必间接测试那些其他方法。

在选择测试 write 方法时，事实证明您正在测试的是您有正确的步骤来返回正确的结果。而已。我不熟悉 php 模拟框架，但我知道对于其他语言的框架，您可以设置所谓的期望，它可以让您指定将在具有某些参数的某个对象上调用某个方法。您甚至可以经常指定它们的执行顺序。这里的要点是您正在测试您的对象发送的传出消息，而不是这些消息的返回值。

与此测试所需的维护相比，这是否有价值由您决定。

于 2013-02-22T22:16:45.313 回答
0

您正在测试是否将正确的参数传递给您准备好的语句。此外，您应该测试该write方法是否返回准备好的语句结果。

如果您不对此进行测试，那么您的应用程序可能会以多种方式中断。

  • 重命名或删除方法参数 ( $sessionId, $sessionData)
  • 重命名您的财产$this->sessionName
  • 删除其中一个bindValue呼叫。
  • 绑定别名的错误命名。它们应该与您的查询匹配。
  • 返回的结果不是execute().

等等等等

所以是的，测试这个是很好的做法。

于 2013-02-22T22:09:26.890 回答
0

假设您的示例描述了一个SessionHandler类，它看起来类似于：

class SessionHandler
{
    public function __construct($sessionTableName, $sessionName, \PDO $databaseConnection)
    { 
        $this->DBTableName = $sessionTableName;
        $this->sessionName = $sessionName;
        $this->databaseConnection = $databaseConnection;
    }

    // among others, your method write($sessionId, $sessionData) follows
}

这可以涵盖方法write()：

public function testWriteInsertsOrUpdatesSessionData()
{
    /**
     * initialize a few explaining variables which we can refer to 
     * later when arranging test doubles and eventually act
     */
    $sessionTableName = 'sessions';
    $sessionName = 'foobarbaz';

    $sessionId = 'foo';
    $sessionData = serialize([
        'bar' => 'baz',
    ]);

    $executed = true;

    /**
     * create a test double for the statement that we expect to be returned 
     * from `PDO::prepare()`
     */
    $statement = $this->createMock(\PDOStatement::class);

    /**
     * set up expectations towards which methods should be invoked 
     * on the statement, specifying their order
     */
    $statement
        ->expects($this->at(0))
        ->method('bindValue')
        ->with(
            $this->identicalTo(':session_id'),
            $this->identicalTo(sessionId)
        );

    $statement
        ->expects($this->at(1))
        ->method('bindValue')
        ->with(
            $this->identicalTo(':session_name'),
            $this->identicalTo($sessionName)
        );

    $statement
        ->expects($this->at(2))
        ->method('bindValue')
        ->with(
            $this->identicalTo(':session_data'),
            $this->identicalTo(sessionData)
        );

    $statement
        ->expects($this->at(3))
        ->method('execute')
        ->willReturn($executed);

    /**
     * create a test double for the database connection we inject
     * into SessionHandler during construction
     */
    $databaseConnection = $this->createMock(\PDO::class);

    $databaseConnection
        ->expects($this->once())
        ->method('prepare')
        ->with($this->identicalTo(sprintf(
            'INSERT INTO `%s` (`session_id`,`session_name`,`session_data`) VALUES(:session_id, :session_name, :session_data) ON DUPLICATE KEY UPDATE `session_data`=:session_data;',
            $sessionTableName
        )))
        ->willReturn($statement);

    $sessionHandler = new SessionHandler(
        $sessionTableName,
        $sessionName,
        $databaseConnection
    );

    $result = $sessionHandler->write(
        $sessionId,
        $sessionData
    );

    $this->assertSame($executed, $result);
}

供参考，请参阅：

于 2017-08-28T20:31:38.933 回答