单元测试和静态方法

Question

阅读并学习单元测试，尝试理解 下面的文章解释了静态函数调用的困难。

我不太清楚这个问题。我一直认为静态函数是在类中汇总实用函数的好方法。例如，我经常使用静态函数调用来初始化，即：

Init::loadConfig('settings.php');
Init::setErrorHandler(APP_MODE); 
Init::loggingMode(APP_MODE);

// start loading app related objects ..
$app = new App();

// 读完这篇文章后，我现在的目标是这个......

$init = new Init();
$init->loadConfig('settings.php');
$init->loggingMode(APP_MODE);
 // etc ...

但是，我为此类编写的几十个测试是相同的。我什么也没改变，它们仍然全部通过。我做错了什么吗？

该帖子的作者声明如下：

静态方法的基本问题是它们是过程代码。我不知道如何对程序代码进行单元测试。单元测试假设我可以单独实例化我的应用程序的一部分。在实例化过程中，我将依赖项与模拟/友好连接起来，以替换真正的依赖项。对于过程式编程，由于没有对象，因此无需“连接”任何内容，代码和数据是分开的。

现在，我从帖子中了解到静态方法创建依赖关系，但没有直观地理解为什么不能测试静态方法的返回值和常规方法一样容易吗？

我将避免静态方法，但我希望了解静态方法何时有用（如果有的话）。从这篇文章看来，静态方法与全局变量一样邪恶，应该尽可能避免。

有关该主题的任何其他信息或链接将不胜感激。

Answer 1

静态方法本身并不比实例方法更难测试。当方法（静态或其他方式）调用其他静态方法时，就会出现问题，因为您无法隔离正在测试的方法。以下是一个难以测试的典型示例方法：

public function findUser($id) {
    Assert::validIdentifier($id);
    Log::debug("Looking for user $id");  // writes to a file
    Database::connect();                 // needs user, password, database info and a database
    return Database::query(...);         // needs a user table with data
}

您可能想用此方法测试什么？

• 传递除正整数以外的任何值都会引发 InvalidIdentifierException。
• Database::query() 收到正确的标识符。
• 找到时返回匹配的 User，未找到时返回 null。

这些要求很简单，但您还必须设置日志记录、连接到数据库、加载数据等。Database 类应该单独负责测试它是否可以连接和查询。 Log 类应该对日志记录执行相同的操作。 findUser() 不应该处理任何这些，但它必须处理，因为它依赖于它们。

相反，如果上面的方法调用 Database 和 Log 实例上的实例方法，则测试可以传入具有特定于当前测试的脚本化返回值的模拟对象。

function testFindUserReturnsNullWhenNotFound() {
    $log = $this->getMock('Log');  // ignore all logging calls
    $database = $this->getMock('Database', array('connect', 'query');
    $database->expects($this->once())->method('connect');
    $database->expects($this->once())->method('query')
             ->with('<query string>', 5)
             ->will($this->returnValue(null));
    $dao = new UserDao($log, $database);
    self::assertNull($dao->findUser(5));
}

如果 findUser() 忽略调用 connect()，为 $id 传递错误的值 (5< /code> 上面），或者返回 null 以外的任何内容。优点在于不涉及数据库，使测试快速而稳健，这意味着它不会因与测试无关的原因（例如网络故障或错误的样本数据）而失败。它使您能够专注于真正重要的事情：findUser() 中包含的功能。

Answer 2

Sebastian Bergmann 同意 Misko Hevery 的观点，并经常引用他的观点：

• 存根和模拟静态方法

单元测试需要接缝，接缝是我们阻止正常代码路径执行的地方，也是我们实现被测类隔离的方式。 Seams 通过多态性工作，我们重写/实现类/接口，然后以不同的方式连接被测试的类，以便控制执行流程。对于静态方法，没有什么可以覆盖的。是的，静态方法很容易调用，但是如果静态方法调用另一个静态方法，则无法覆盖被调用方法的依赖关系。

静态方法的主要问题是它们引入了耦合，通常是通过对依赖关系进行硬编码到您使用的代码中，使得在单元测试中很难用存根或模拟来替换它们。这违反了开放/封闭原则和依赖倒置原则，其中两个SOLID 原则。

静态被认为是有害的，您说得完全正确。避开他们。

请检查链接以获取更多信息。

更新：请注意，虽然静态仍然被认为是有害的，从 PHPUnit 4.0 开始，已删除存根和模拟静态方法的功能

Answer 3

测试静态方法时我没有看到任何问题（至少在非静态方法中不存在任何问题）。

• 使用依赖注入将模拟对象传递给正在测试的类。
• 模拟静态方法可以使用合适的自动加载器或操作include_path传递给被测类。
• 后期静态绑定处理调用同一类中静态方法的方法。