Làm cách nào để kiểm tra nhiều ngoại lệ với PHPUnit?

Question

Khi kiểm tra các trường hợp ngoại lệ với PHPUnit, cách tốt nhất để yêu cầu mọi tuyên bố hoặc xác nhận phải ném một ngoại lệ để kiểm tra vượt qua là gì?

tôi về cơ bản muốn làm một cái gì đó như thế này:

public function testExceptions() 
{ 

    $this->setExpectedException('Exception'); 

    foo(-1); //throws exception 
    foo(1); //does not throw exception 

} 

//Test will fail because foo(1) did not throw an exception 

tôi đã đi lên với những điều sau đây, mà không được công việc, nhưng khá xấu xí IMO.

public function testExceptions() 
{ 

    try { 
     foo(-1); 
    } catch (Exception $e) { 
     $hit = true; 
    } 

    if (!isset($hit)) 
     $this->fail('No exception thrown'); 

    unset($hit); 

    try { 
     foo(1); 
    } catch (Exception $e) { 
     $hit = true; 
    } 

    if (!isset($hit)) 
     $this->fail('No exception thrown'); 

    unset($hit); 

} 

Answer 1

Vì ngoại lệ là những sự kiện lớn như vậy trong luồng chương trình, nên thử nghiệm nhiều sự kiện trong một thử nghiệm đơn lẻ là vấn đề. Điều đơn giản nhất là chỉ cần chia nó thành hai bài kiểm tra - điều đầu tiên yêu cầu một ngoại lệ để có thể vượt qua, thứ hai chỉ đơn giản là chạy, và sẽ thất bại nó nó đã ném một. Bạn có thể thêm một số bài kiểm tra khác vào thứ hai nếu bạn muốn (xác nhận một giá trị trả về có thể), nhưng tôi muốn nghiêng để chắc chắn nó vẫn chỉ là một điều thiết yếu, theo cách đặt tên của nó. đang

/** 
* @expectedException Exception 
*/ 
public function testBadFooThrowsException() 
{ 
    // optional, can also do it from the '@expectedException x' 
    //$this->setExpectedException('Exception'); 
    foo(-1); //throws exception -- good. 
} 

public function testFooDoesNotThrowException() 
{ 
    foo(1); //does not throw exception 
} 

Answer 2

Điều này không có ý nghĩa với tôi.

Tôi đoán bạn đang cố gắng thử nghiệm nhiều điều riêng biệt với một trường hợp kiểm tra, đó là thực tiễn không tốt.

Khi foo() ném ngoại lệ dự kiến, trường hợp kiểm tra thành công và bar() sẽ không chạy.

Chỉ cần tạo hai trường hợp thử nghiệm riêng biệt, ít hơn nhiều so với mã bạn đã tạo trong danh sách thứ hai.

Hoặc giải thích lý do tại sao có thể chạy bar(), sau khi foo() không thành công với ngoại lệ, khi nó cũng sẽ ném một ngoại lệ.

Answer 3

Hơi sạch hơn (nhưng tôi vẫn muốn đề nghị tách các xét nghiệm của bạn:

try { 
    foo(-1); 
    $this->fail('No exception thrown'); 
} catch (Exception $e) {} 

Answer 4

Mở rộng về câu trả lời @ dave1010 của, đây là cách tôi giải quyết vấn đề này này cho phép bạn giữ tất cả những ". xác nhận "gọn gàng và gọn gàng trong một thử nghiệm. Bạn chỉ cần xác định một mảng các biến sẽ thất bại trong việc kiểm tra, và sau đó lặp qua từng biến và xem liệu một ngoại lệ có được nâng lên hay không. thử nghiệm đã qua.

<?php 

public function testSetInvalidVariableType() 
{ 
    $invalid_vars = array(
     '',     // Strings 
     array(),   // Arrays 
     true,    // Booleans 
     1,     // Integers 
     new \StdClass  // Objects 
    ); 

    foreach ($invalid_vars as $var) { 
     try { 
      $object->method($var); 
      $this->fail('No exception thrown for variable type "' . gettype($var) . '".'); 
     } catch (\Exception $expected) { 
     } 
    } 
} 

Answer 5

Tôi nghĩ đây là một tình huống rất phổ biến trong thử nghiệm đơn vị. Cách tiếp cận tôi sử dụng trong trường hợp này là sử dụng phpunit dataProviders. Tất cả hoạt động như mong đợi và mã kiểm tra trở nên rõ ràng và súc tích hơn.

class MyTest extends PHPUnit_Framework_TestCase 
{ 
    public function badValues() 
    { 
     return array(
      array(-1), 
      array(1) 
     ); 
    } 


    /** 
    * @dataProvider badValues 
    * @expectedException Exception 
    */ 
    public function testFoo($badValue) 
    { 
     foo($badValue); 
    } 
}