Question

我最近开始进行单元测试，我想知道，我应该编写100％代码覆盖率的单元测试吗？

当我最终编写比生产代码更多的单元测试代码时，这似乎是徒劳的。

我正在编写一个PHP Codeigniter项目，有时似乎我编写了这么多代码来测试一个小函数。

例如，此单元测试

public function testLogin(){
    //setup
    $this->CI->load->library("form_validation");
    $this->realFormValidation=new $this->CI->form_validation;
    $this->CI->form_validation=$this->getMock("CI_Form_validation");
    $this->realAuth=new $this->CI->auth;
    $this->CI->auth=$this->getMock("Auth",array("logIn"));
    $this->CI->auth->expects($this->once())
                   ->method("logIn")
                   ->will($this->returnValue(TRUE));

    //test
    $this->CI->form_validation->expects($this->once())
        ->method("run")
        ->will($this->returnValue(TRUE));
    $_POST["login"]=TRUE;
    $this->CI->login();
    $out = $this->CI->output->get_headers();
    //check new header ends with dashboard
    $this->assertStringEndsWith("dashboard",$out[0][0]);

    //tear down
    $this->CI->form_validation=$this->realFormValidation;
    $this->CI->auth=$this->realAuth;

}
public function badLoginProvider(){
    return array(
        array(FALSE,FALSE),
        array(TRUE,FALSE)
    );
}
/**
 * @dataProvider badLoginProvider
 */
public function testBadLogin($formSubmitted,$validationResult){
    //setup
    $this->CI->load->library("form_validation");
    $this->realFormValidation=new $this->CI->form_validation;
    $this->CI->form_validation=$this->getMock("CI_Form_validation");

    //test
    $this->CI->form_validation->expects($this->any())
        ->method("run")
        ->will($this->returnValue($validationResult));
    $_POST["login"]=$formSubmitted;
    $this->CI->login();
    //check it went to the login page
    $out = output();
    $this->assertGreaterThan(0, preg_match('/Login/i', $out));

    //tear down
    $this->CI->form_validation=$this->realFormValidation;
}

对于此生产代码

public function login(){
    if($this->input->post("login")){
        $this->load->library('form_validation');
        $username=$this->input->post('username');
        $this->form_validation->set_rules('username', 'Username', 'required');
        $this->form_validation->set_rules('password', 'Password', "required|callback_userPassCheck[$username]");
        if ($this->form_validation->run()===FALSE) {
            $this->load->helper("form");
            $this->load->view('dashboard/login');
        }
        else{
            $this->load->model('auth');
            echo "valid";
            $this->auth->logIn($this->input->post('username'),$this->input->post('password'),$this->input->post('remember_me'));
            $this->load->helper('url');
            redirect('dashboard');
        }
    }
    else{
        $this->load->helper("form");
        $this->load->view('dashboard/login');
    }
}

我哪里错了？

Answer 1

在我看来，测试代码不仅仅是生产代码，这是正常的。但是测试代码往往是直截了当的，一旦你掌握它，它就像编写测试一样简单。

话虽如此，如果您发现您的测试代码太复杂而无法编写/覆盖生产代码中的所有执行路径，那么这是重构的一个很好的指标：您的方法可能太长，或者尝试做几个事情，或有这么多外部依赖等...

另一点是，拥有高测试覆盖率是好的，但不需要100％或一些非常高的数字。有时代码中没有逻辑，就像代码只是将任务委托给其他人一样。在这种情况下，您可以跳过测试它们并使用@codeCoverageIgnore注释在代码覆盖率中忽略它们。

Answer 2

在我看来，它的逻辑是测试代码更多，因为你必须测试多个场景，必须提供测试数据，你必须检查每个案例的数据。

通常，80％的测试覆盖率是一个很好的价值。在大多数情况下，没有必要测试100％的代码，因为你不应该测试例如setter和getter。不要只测试统计数据;）

我应该对每一段代码进行单元测试

2 个答案: