Implemented specific tests for collect

Author: RWOverdijk

src/AssetManager/Resolver/AggregateResolver.php

@@ -60,7 +60,7 @@ public function collect()
         $collection = array();
 
         foreach ($this->queue as $resolver) {
-            if (!method_exists($resolver, 'collect') {
+            if (!method_exists($resolver, 'collect')) {
                 continue;
             }
 

tests/AssetManagerTest/Resolver/AggregateResolverTest.php

@@ -8,6 +8,11 @@
 
 class AggregateResolverTest extends PHPUnit_Framework_TestCase
 {
+    public function setUp()
+    {
+        require_once __DIR__ . '/../../_files/ResolverWithCollect.php';
+    }
+
     public function testResolve()
     {
         $resolver = new AggregateResolver();
@@ -46,9 +51,9 @@ public function testResolve()
 
     public function testCollect()
     {
-        $resolver = new AggregateResolver();
-
-        $lowPriority = $this->getMock('AssetManager\Resolver\ResolverInterface');
+        /* Tests for interfaces that _do_ implement the `collect` method. */
+        $resolver    = new AggregateResolver();
+        $lowPriority = $this->getMock('ResolverWithCollect');
         $lowPriority
             ->expects($this->exactly(2))
             ->method('collect')
@@ -57,7 +62,7 @@ public function testCollect()
 
         $this->assertContains('one', $resolver->collect());
 
-        $highPriority = $this->getMock('AssetManager\Resolver\ResolverInterface');
+        $highPriority = $this->getMock('ResolverWithCollect');
         $highPriority
             ->expects($this->once())
             ->method('collect')
@@ -69,5 +74,21 @@ public function testCollect()
         $this->assertContains('three', $collection);
 
         $this->assertCount(3, $collection);
+
+        /* Tests for interfaces that _don't_ implement the `collect` method. */
+        $resolver    = new AggregateResolver();
+        $lowPriority = $this->getMock('AssetManager\Resolver\ResolverInterface');
+
+        $resolver->attach($lowPriority);
+
+        $this->assertEquals(array(), $resolver->collect());
+
+        $highPriority = $this->getMock('AssetManager\Resolver\ResolverInterface');
+        $resolver->attach($highPriority, 1000);
+
+        $collection = $resolver->collect();
+        $this->assertEquals(array(), $collection);
+
+        $this->assertCount(0, $collection);
     }
 }

tests/_files/ResolverWithCollect.php

@@ -0,0 +1,14 @@
+<?php
+
+use AssetManager\Resolver\ResolverInterface;
+
+class ResolverWithCollect implements ResolverInterface
+{
+    public function resolve($path)
+    {
+    }
+
+    public function collect()
+    {
+    }
+}