Merge pull request #377 from FriendsOfCake/josegonzalez-patch-1

Fix issue where tested table does not return a mocked BehaviorRegistry

.travis.yml

@@ -50,7 +50,7 @@ before_script:
 
 script:
   - sh -c "if [ '$COVERALLS' = '1' ]; then phpunit --stderr --coverage-clover build/logs/clover.xml; fi"
-  - sh -c "if [ '$COVERALLS' = '1' ]; then php vendor/bin/coveralls -c .coveralls.yml -v; fi"
+  - sh -c "if [ '$COVERALLS' = '1' ]; then php vendor/bin/coveralls -v; fi"
   - sh -c "if [ '$DEFAULT' = '1' ]; then phpunit --stderr; fi"
   - sh -c "if [ '$PHPCS' = '1' ]; then vendor/bin/phpcs -n -p --extensions=php --standard=vendor/cakephp/cakephp-codesniffer/CakePHP --ignore=vendor --ignore=docs . ; fi"
 

tests/TestCase/Listener/SearchListenerTest.php

@@ -70,15 +70,24 @@ public function testInjectSearch()
         $eventManager = new \Cake\Event\EventManager();
         $controller = new \Cake\Controller\Controller($request, $response, 'Search', $eventManager);
 
+        $behaviorRegistryMock = $this->getMockBuilder('\Cake\ORM\BehaviorRegistry')
+            ->setMockClassName('BehaviorRegistry')
+            ->setMethods(['hasMethod'])
+            ->getMock();
+        $behaviorRegistryMock->expects($this->once())
+            ->method('hasMethod')
+            ->will($this->returnValue(true));
+
         $tableMock = $this->getMockBuilder('\Cake\ORM\Table')
             ->setMockClassName('SearchTables')
-            ->setMethods(['filterParams'])
+            ->setMethods(['filterParams', 'behaviors'])
             ->getMock();
         $tableMock->expects($this->once())
             ->method('filterParams')
-            ->will($this->returnCallback(function () use ($params) {
-                return $params;
-            }));
+            ->will($this->returnValue($params));
+        $tableMock->expects($this->once())
+            ->method('behaviors')
+            ->will($this->returnValue($behaviorRegistryMock));
 
         \Cake\ORM\TableRegistry::set('Search', $tableMock);