Merge pull request #20 from GrandLTU/top-hits-fix

Fix top hits aggregation to respect set parameters.

src/Aggregation/TopHitsAggregation.php

@@ -153,6 +153,6 @@ class TopHitsAggregation extends AbstractAggregation
             }
         );
 
-        return $fd;
+        return $this->processArray($fd);
     }
 }

tests/Aggregation/TopHitsAggregationTest.php

@@ -39,4 +39,23 @@ class TopHitsAggregationTest extends \PHPUnit_Framework_TestCase
 
         $this->assertEquals($expected, $aggregation->toArray());
     }
+
+    /**
+     * Tests if adding parameters has any effect.
+     */
+    public function testParameters()
+    {
+        $topHitsAggregation = new TopHitsAggregation('test');
+        $topHitsAggregation->addParameter('_source', ['include' => ['title']]);
+        $expectedAgg = new \stdClass();
+        $expectedAgg->sort = [];
+        $expectedAgg->_source = ['include' => ['title']];
+        $expected = [
+            'agg_test' => [
+                'top_hits' => $expectedAgg,
+            ],
+        ];
+
+        $this->assertEquals($expected, $topHitsAggregation->toArray());
+    }
 }