覆盖存根中的父函数

Question

对于你们中的一些人来说，这可能是一件容易的事。我正在尝试在我拥有的小型数据库连接类上测试受保护的方法。

相关代码如下：

class DbConnect{

/**
 *    Connexion  MSSQL local
 */
protected function localConnect($localconfig){
    $connectionInfo = array("UID" => $localconfig->uid, 
                            "PWD" =>$localconfig->pwd, 
                            "Database"=> $localconfig->DB);

   $this->localConnection = sqlsrv_connect($localconfig->serverName,
                                           $connectionInfo);

   if( $this->localConnection === false ){
       $sql_error = sqlsrv_errors();
       throw new DBException("Error in DB Connection.\r\n
                              SQL ERROR:" . $sql_error);
   }
}
}

为了测试该方法，我有一个好主意（可能来自此处某处的帖子）来子类化并从那里调用。我在测试文件的底部创建了一个子类。我显然无法覆盖该方法对公共的可见性，因此决定在存根中采用另一种方法：声明一个调用父级受保护的 localConnect 方法的公共方法：

 class DBConnectStub extends DBconnect{

   public function callLocalConnect($localConfig){
        parent::localConnect($localConfig);
    }
}

我的测试现在看起来像这样：

/**
 * @expectedException DBException
 */
public function test_localConnectError(){

  $localconfig = (object) array ( 'serverName' => 'nohost', 
                                   'uid' => 'nouid',
                                  'pwd' => 'noPwd',
                                  'DB' => 'noDB'

                         );  

  $db = DbConnectStub::getInstance($localconfig, array());
  $db->callLocalConnect($localConfig);
  unset($db);

}

奇怪的部分，当我运行测试时，php 吐出：

致命错误：调用 C:\tirelinkCRMsync\test 中未定义的方法 DbConnect::callLocalConnect() \tirelinkCRMSync\DBConnectTest.php 第 82 行。

该对象已正确实例化，但为什么未定义该方法，肯定有一个细节让我困惑。这种方法有效还是有更好的方法？

Answer 1

我正在尝试测试受保护的方法[...]

不要

就这么简单。只是不要。受保护的方法不是类公共 API 的一部分，因此在尝试确保类正常工作时，不应假设它们的工作方式。

您应该能够在不调整测试的情况下更改代码（公共函数的实现）。这就是您的测试制作的目的，以便您可以更改代码并你确信它仍然有效。当您同时更改代码和测试时，您无法确定您的代码仍然像以前一样工作！

请参阅：Sebastian Bergmann - 测试您的隐私.html

所以：仅仅因为可以测试受保护和私有的属性和方法并不意味着这是一件“好事”。

以及： 最佳使用 PHPUnit 测试受保护方法的实践 - 在抽象类上

这篇文章还提到，使用

$method = new ReflectionMethod(
    'Foo', 'doSomethingPrivate'
);
$method->setAccessible(TRUE);

Which 比为每个要测试的方法创建子类更容易。

---

迂腐的侧节点：

恕我直言，它应该是 $this->localConnect 而不是 parent::localConnect 因为parent::仅用于调用相同 父类的方法。 （没什么大不了的，只是令人困惑，至少对我来说）。

Answer 2

这可能是一个愚蠢的问题，但是您是否重写了 DbConnectStub::getInstance 以使其返回 Stub 实例？

class DBConnectStub extends DBconnect{
 public static function getInstance ()
 {
    //whatever process to create the instance (and not the parent method call that will return a DBConnect instance)
 } 

 public function callLocalConnect($localConfig){
    parent::localConnect($localConfig);
 }
}