PHPUnit、複数の値をテストする

Question

同じ属性に対して複数の値をテストするにはどうすればよいですか？

class Test { 

    private $_optionalValue = null; 

    function setValue(String $optionalValue) 
    { 
     $this->_optionalValue = $optionalValue; 
    } 
} 

だからここに、 "$ _optionalValueは、" NULLまたはユーザー定義の値かもしれないが、私はこのようなPHPUnitでチェックする場合：$ optionalValueがStringではないので

$optionalValue = PHPUnit_Util_Class::getObjectAttribute($my_object, '_optionalValue'); 

$this->assertThat(
    $optionalValue, 
    $this->logicalXor(
     $this->assertNull($optionalValue), 
     $this->logicalAnd(
      $this->assertAttributeInternalType('string', '_optionalValue', $optionalValue), 
      $this->assertRegExp('/[0-9]{2}:[0-9]{2}:[0-9]{2}/', (string) $optionalValue) 
     ) 
    ) 
); 

ザ・正規表現アサーションが失敗しました（デフォルトではヌルです）

Answer 1

オブジェクトのプライベートプロパティをテストしていますが、これは一般的に避けるべきです。なぜなら、これはユニットの内部であり、そのことに気を付けるべきではないからです。

ユニットがそのクラスの値を検証する必要がある場合は、一般的に何らかの値を検証する必要があります。

したがって、検証のロジックを独自のユニットにカプセル化することができます。バリデーター：

class FooValueValidator implements Validator { 
    /** 
    * @var string 
    */ 
    private $value; 

    /** 
    * @var string 
    */ 
    private $regex = '/[0-9]{2}:[0-9]{2}:[0-9]{2}/'; 

    public function __construct($value) { 
     $this->value = $value; 
    } 

    /** 
    * @return bool 
    */ 
    public function isValid() { 

     if (is_null($this->value)) { 
      return TRUE; 
     } 

     if (!is_string($this->value)) { 
      return FALSE; 
     } 

     $result = preg_match($this->pattern, $this->value); 
     if (FALSE === $result) { 
      throw new Exception(sprintf('Regular expression failed.')); 
     } 
     return (bool) $result; 
    } 
} 

バリデーターの単位テストを書くことができます。あなたは、あなたのバリデーターが働いていることを知っていますし、どこでも好きな場所で使うことができます。

class Test { 

    private $_optionalValue = null; 

    /** 
    * @var Validator 
    */ 
    private $_validator; 

    public function __construct(Validator $validator) { 
     $this->_validator = $validator; 
    } 

    function setValue(String $optionalValue) 
    { 
     if (!$this->validator->isValid($optionalValue)) { 
      throw new InvalidArgumentException(sprintf('Invalid value "%s".', $optionalValue)); 
     } 
     $this->_optionalValue = $optionalValue; 
    } 
} 

Answer 2

あなたはassertThatにお電話内部のアサーションを作っているが、あなたは構築し、代わりに、制約に渡す必要があります。 assertで始まるすべてのメソッドは、すぐに値を評価し、不一致の例外をスローします。各アサーションには、対応する制約クラスがあり、一部はファクトリメソッドがあります。

$optionalValue = PHPUnit_Util_Class::getObjectAttribute($my_object, '_optionalValue'); 

$this->assertThat(
    $optionalValue, 
    $this->logicalXor(
     $this->isNull(), 
     $this->logicalAnd(
      new PHPUnit_Framework_Constraint_IsType('string'), 
      new PHPUnit_Framework_Constraint_PCREMatch('/[0-9]{2}:[0-9]{2}:[0-9]{2}/') 
     ) 
    ) 
); 

ところで、私はあなたがオフに優れているa）は、このように内部状態をテストしていないことに同意しないとあなたが期待する値を知っているように、b）は、あなたのテストを設計する必要があります。各テストでは、システムを単一の期待状態にする必要があります。乱数を使用するコードでも、スタブを使用して固定シーケンスを置き換える必要があります。複数の可能性を可能にするテストは疑わしい。