如何測試一個集合類使用PHPUnit

Question

我有類似這樣的（爲簡潔一些邏輯刪除）類：

class FooCollection { 
    protected $_foos; 

    public function __construct() { 
     $this->_foos = new SplObjectStorage(); 
    } 

    public function addFoo(FooInterface $foo) { 
     $this->_foos->attach($foo); 
    } 

    public function removeFoo(FooInterface $foo) { 
     $this->_foos->detach($foo); 
    } 
} 

我想用PHPUnit的測試addFoo()和removeFoo()方法，我是想知道做這件事最好的策略是什麼？據我所知，我只有幾個選項：

1. 添加一個方法hasFoo(FooInterface $foo)並檢查後添加。
2. 添加方法getFoos()，直接返回SplObjectStorage實例，並檢查添加後是否在$foo之內。
3. 嘗試removeFoo($foo)後addFoo($foo)並檢查是否有異常。
4. 將$_foos設爲公共屬性，並在添加後直接檢查（壞，壞，壞...）。

選項＃1和＃2正在改變公共界面僅用於測試目的，我不確定我對此的看法。它們看起來像是非常普通且有用的方法，但在我的具體情況中，我從來沒有必要檢查集合中是否存在特定的實例，也沒有檢索所有實例，所以它真的只會膨脹。另外，似乎如果我在一次測試中測試接口的多個部分，我並不真正測試一個「單元」，但這或多或少只是一個哲學掛斷。

選項＃3對我來說似乎很尷尬。

選項＃4是一個非常糟糕的主意，我不應該把它列出來，因爲即使它在這裏被建議，我也不會這樣做。

Answer 1

你沒有發佈一個公共訪問器來獲取集合，但我相信你有一個，否則添加/刪除foos到一個公共無法訪問的數組是沒有意義的。所以，你可以嘗試像（PHPUnit的3.6，PHP 5.4）：

public function setUp() 
{ 
    $this->NumbersCollection = new NumbersCollection; 
} 

public function tearDown() 
{ 
    unset($this->NumbersCollection); 
} 

public function testNumbersCollection() 
{ 
    $this->NumbersCollection->addNumber(1); 
    $this->NumbersCollection->addNumber(2); 

    $this->assertSame(3, $this->NumbersCollection->sum()); 
    $this->assertSame(2, $this->NumbersCollection->product()); 

    $this->NumbersCollection->removeNumber(1); 

    $this->NumbersCollection->addNumber(7); 

    $this->assertSame(9, $this->NumbersCollection->sum()); 
    $this->assertSame(14, $this->NumbersCollection->product()); 
} 

Answer 2

爲什麼不創建您在傳遞給構造一個模擬SplObjectStorage對象？然後你可以斷言在模擬上調用了attach和detach方法。

function testAttachFOO() { 
    $mockStorage = $this->getMockBuilder('SplObjectStorage') 
        ->setMethods(array('attach')) 
        ->getMock(); 

    $mockFoo = $this->getMock('FooInterface'); 

    $mockStorage->expects($this->once()) 
     ->method('attach') 
     ->with($mockFoo); 

    $collection = new FooCollection($mockStorage); 

    $collection->addFoo($mockFoo); 
} 

和removeFoo類似的東西。

這樣做的確需要您更改構造函數，以便可以注入依賴項。但是IMO使得代碼更清晰。也使測試更容易。

所以構造變爲：

public function __construct(SPLObjectStorage $storage) { 
    $this->_foos = $storage; 
} 

如果類變得很難建立這樣做，這是一個跡象，表明類是做得太多了，應該重構到更多更小的類。