[英]How to made the code coverage to 100%
最近我开始写PHPUnit测试。 这是我的模型代码。(我使用了CodeIgniter 3)。
Account_model.php
class Account_model extends CI_Model
{
...
public function select_by_seq($seq = '', $select_columns = [])
{
try {
$bind = [':a_seq' => $seq];
// check $select_colums is exist in table
if ($this->check_column($select_columns) === false)
{
throw new Exception('columns illegal', 201);
}
...
$sql = "select ....
from {$this->db->dbprefix('account')}
where a_seq = :a_seq";
$query = $this->db->query($sql, $bind);
// ===== this always not runing. =====
if ($query === false)
{
// ===== this always not runing. =====
throw new Exception('sql errors', 301);
}
else
{
return $query->result_array();
}
}
catch (Exception $error)
{
// set error log
$this->set_error_log($error->getCode() . $error->getMessage());
}
return false;
}
}
这是我的测试Account_model_test.php
class Account_model_test extends TestCase
{
public static function setUpBeforeClass()
{
parent::setUpBeforeClass();
$CI =& get_instance();
}
public function setUp()
{
$this->resetInstance();
loader('model', 'account_model');
$this->obj = $this->CI->account_model;
}
public function test_select_by_seq()
{
$result = $this->obj->select_by_seq(
$seq = '20160830'
);
$this->assertCount(1, $result);
}
public function test_select_by_seq_with_illegal_column()
{
$result = $this->obj->select_by_seq(
$seq = '20160830',
$select_columns = ['illegal']
);
$this->assertFalse($result);
}
...
}
因为我自己写SQL。 我建立了PHPUnit测试, if ($query === false)
不能解决这个问题。 然后我的代码覆盖率没有达到100%。 这个问题让我认为100%对于单元测试非常重要吗? 还是我必须修改我的模型代码? 感谢您的帮助。
始终保持100%的代码覆盖率是很好的。 但是就像您说的那样,在某些情况下很难做到这一点。 因此,对于大多数项目而言,代码覆盖率大于70%相当不错。 有关项目所需的最小代码覆盖率,请参见此链接 。 但是,您应该尝试将业务逻辑提取到其自己的类(存储库)并对其进行单元测试。
如果您也向我们展示您的测试文件,那将是很好的,因此我们可以更清楚地了解您已经拥有的文件。 不知不觉中,我建议使用查询功能的Mock,因此它可以返回false
作为结果。 您可以在此处找到有关Mocks的更多信息
同样如前所述:您不应专注于100%的代码覆盖率。
@codeCoverageIgnore
,@codeCoverageIgnoreStart
和@codeCoverageIgnoreEnd
批注可用于从覆盖率分析中排除代码行。 -PHPUnit手册
PHPUnit 文档中有示例。
简而言之,在*Start
*End
批注中包装要被代码覆盖率忽略的代码:
if (false) {
// @codeCoverageIgnoreStart
print '*';
// @codeCoverageIgnoreEnd
}
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.