Fix parsing of the top hits aggregation response

The top hits aggregation response wasn't being read to completion causing
other aggregation results on the same level to be skipped.

Closes #1169

Author: gmarz

src/Nest/Resolvers/Converters/Aggregations/AggregationConverter.cs

@@ -78,6 +78,7 @@ private IAggregation GetHitsAggregation(JsonReader reader, JsonSerializer serial
 			var total = o["total"].ToObject<long>();
 			var maxScore = o["max_score"].ToObject<double?>();
 			var hits = o["hits"].Children().OfType<JObject>().Select(s=>s);
+			reader.Read();
 			return new TopHitsMetric(hits) { Total = total, MaxScore = maxScore };
 		}
 

src/Tests/Nest.Tests.Integration/Nest.Tests.Integration.csproj

@@ -183,6 +183,7 @@
     <Compile Include="Mapping\MappingVisitorTests.cs" />
     <Compile Include="Mapping\GetMultipleMappingTests.cs" />
     <Compile Include="Reproduce\Reproduce1079Tests.cs" />
+    <Compile Include="Reproduce\Reproduce1169Tests.cs" />
     <Compile Include="Reproduce\Reproduce769Tests.cs" />
     <Compile Include="Reproduce\Reproduce945Tests.cs" />
     <Compile Include="Reproduce\Reproduce953Tests.cs" />

src/Tests/Nest.Tests.Integration/Reproduce/Reproduce1169Tests.cs

@@ -0,0 +1,67 @@
+using FluentAssertions;
+using Nest.Tests.MockData.Domain;
+using NUnit.Framework;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace Nest.Tests.Integration.Reproduce
+{
+	[TestFixture]
+	public class Reproduce1169Tests : IntegrationTests
+	{
+		[Test]
+		public void TopHitsReturnedFirst()
+		{
+			var response = this.Client.Search<ElasticsearchProject>(s => s
+				.Size(0)
+				.Aggregations(a => a
+					.TopHits("my-top-hits", th => th
+						.Size(1)
+					)
+					.Terms("a", t => t
+						.Field(p => p.Name)
+					)
+				)
+			);
+
+			response.IsValid.Should().BeTrue();
+			
+			var topHits = response.Aggs.TopHitsMetric("my-top-hits");
+			topHits.Should().NotBeNull();
+			topHits.Hits<ElasticsearchProject>().Count().Should().BeGreaterThan(0);
+
+			var terms = response.Aggs.Terms("a");
+			terms.Should().NotBeNull();
+			terms.Items.Count().Should().BeGreaterThan(0);
+		}
+
+		[Test]
+		public void TermsReturnedFirst()
+		{
+			var response = this.Client.Search<ElasticsearchProject>(s => s
+				.Size(0)
+				.Aggregations(a => a
+					.TopHits("my-top-hits", th => th
+						.Size(1)
+					)
+					.Terms("b", t => t
+						.Field(p => p.Name)
+					)
+				)
+			);
+
+			response.IsValid.Should().BeTrue();
+
+			var topHits = response.Aggs.TopHitsMetric("my-top-hits");
+			topHits.Should().NotBeNull();
+			topHits.Hits<ElasticsearchProject>().Count().Should().BeGreaterThan(0);
+
+			var terms = response.Aggs.Terms("b");
+			terms.Should().NotBeNull();
+			terms.Items.Count().Should().BeGreaterThan(0);
+		}
+	}
+}