fix parser for new reponse in FT.PROFILE redis#306

Author: atakavci

src/NRedisStack/ResponseParser.cs

@@ -673,7 +673,14 @@ public static Dictionary<string, RedisResult> ToStringRedisResultDictionary(this
             foreach (var pair in res)
             {
                 var arr = (RedisResult[])pair!;
-                dict.Add(arr[0].ToString()!, arr[1]);
+                if (arr.Length > 1)
+                {
+                    dict.Add(arr[0].ToString()!, arr[1]);
+                }
+                else
+                {
+                    dict.Add(arr[0].ToString()!, null);
+                }
             }
             return dict;
         }

src/NRedisStack/Search/AggregationResult.cs

@@ -14,8 +14,9 @@ internal AggregationResult(RedisResult result, long cursorId = -1)
     {
         var arr = (RedisResult[])result!;
 
-        // the first element is always the number of results
-        TotalResults = (long)arr[0];
+        //  this statement below is not true as explained in the document https://redis.io/docs/latest/commands/ft.aggregate/#return
+        // // the first element is always the number of results
+        // TotalResults = (long)arr[0];
 
         _results = new Dictionary<string, RedisValue>[arr.Length - 1];
         for (int i = 1; i < arr.Length; i++)
@@ -33,7 +34,7 @@ internal AggregationResult(RedisResult result, long cursorId = -1)
 
             _results[i - 1] = cur;
         }
-
+        TotalResults = _results.Length;
         CursorId = cursorId;
     }
 

tests/NRedisStack.Tests/Search/SearchTests.cs

@@ -507,7 +507,7 @@ public void TestApplyAndFilterAggregations()
 
         // actual search
         AggregationResult res = ft.Aggregate(index, r);
-        Assert.Equal(3, res.TotalResults);
+        Assert.Equal(2, res.TotalResults);
 
         Row r1 = res.GetRow(0);
         Assert.Equal("def", r1.GetString("name"));
@@ -2685,7 +2685,7 @@ public void TestProfile()
         var searchRes = profileSearch.Item1;
         var searchDet = profileSearch.Item2;
 
-        Assert.Equal(5, searchDet.Count);
+        Assert.Equal(6, searchDet.Count);
         Assert.Equal(2, searchRes.Documents.Count);
 
 
@@ -2694,8 +2694,8 @@ public void TestProfile()
         var profileAggregate = ft.ProfileAggregate(index, aggReq);
         var aggregateRes = profileAggregate.Item1;
         var aggregateDet = profileAggregate.Item2;
-        Assert.Equal(5, aggregateDet.Count);
-        Assert.Equal(1, aggregateRes.TotalResults);
+        Assert.Equal(6, aggregateDet.Count);
+        Assert.Equal(2, aggregateRes.TotalResults);
     }
 
     [SkipIfRedis(Is.Enterprise)]
@@ -2715,16 +2715,16 @@ public async Task TestProfileAsync()
         var searchRes = profileSearch.Item1;
         var searchDet = profileSearch.Item2;
 
-        Assert.Equal(5, searchDet.Count);
+        Assert.Equal(6, searchDet.Count);
         Assert.Equal(2, searchRes.Documents.Count);
 
         // check using AggregationRequest
         var aggReq = new AggregationRequest("*").Load(FieldName.Of("t")).Apply("startswith(@t, 'hel')", "prefix");
         var profileAggregate = await ft.ProfileAggregateAsync(index, aggReq);
         var aggregateRes = profileAggregate.Item1;
         var aggregateDet = profileAggregate.Item2;
-        Assert.Equal(5, aggregateDet.Count);
-        Assert.Equal(1, aggregateRes.TotalResults);
+        Assert.Equal(6, aggregateDet.Count);
+        Assert.Equal(2, aggregateRes.TotalResults);
     }
 
     [Fact]