stubbles\input\broker\RequestBroker::procure() now returns the procur…

…ed object
stubbles · Jun 22, 2015 · 21fd5ca · 21fd5ca
1 parent 3528be3
commit 21fd5ca
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,9 @@
+5.2.1 (2015-06-22)
+------------------
+
+  * `stubbles\input\broker\RequestBroker::procure()` now returns the procured object
+
+
 5.2.0 (2015-06-22)
 ------------------
 

diff --git a/src/main/php/broker/RequestBroker.php b/src/main/php/broker/RequestBroker.php
@@ -88,6 +88,7 @@ public function __construct(array $paramBrokers = [])
      * @param   \stubbles\input\Request  $request
      * @param   object                   $object   the object instance to fill with values
      * @param   string                   $group    restrict procurement to given group
+     * @return  object
      * @throws  \InvalidArgumentException
      */
     public function procure(Request $request, $object, $group = null)
@@ -103,6 +104,8 @@ public function procure(Request $request, $object, $group = null)
                          ->procure($request, $targetMethod->annotation())
             );
         }
+
+        return $object;
     }
 
     /**

diff --git a/src/test/php/broker/RequestBrokerTest.php b/src/test/php/broker/RequestBrokerTest.php
@@ -68,11 +68,10 @@ public function procureNonObjectThrowsInvalidArgumentException()
      */
     public function procuresOnlyThoseInGivenGroup()
     {
-        $object = new BrokerClass();
         $this->request->mapCalls(
                 ['readParam' => ValueReader::forValue('just some string value')]
         );
-        $this->requestBroker->procure($this->request, $object, 'main');
+        $object = $this->requestBroker->procure($this->request, new BrokerClass(), 'main');
         assertFalse($object->isVerbose());
         assertEquals('just some string value', $object->getBar());
         assertNull($object->getBaz());
@@ -94,9 +93,8 @@ public function procuresAllIfNoGroupGiven()
                     )
                 ]
         );
-        $object = new BrokerClass();
         $requestBroker = new RequestBroker(['Mock' => $paramBroker]);
-        $requestBroker->procure($this->request, $object);
+        $object = $requestBroker->procure($this->request, new BrokerClass());
         assertTrue($object->isVerbose());
         assertEquals('just some string value', $object->getBar());
         assertEquals('just another string value', $object->getBaz());