Address PR comments

Author: kashifsoofi

tests/Meilisearch.Tests/DocumentTests.cs

@@ -643,46 +643,31 @@ await index.GetDocumentsAsync<Movie>(new DocumentsQuery()
             documents.Results.Last().Id.Should().Be("11");
         }
 
-        [Fact]
-        public async Task GetMultipleExistingDocumentWithAscSort()
-        {
-            var index = await _fixture.SetUpBasicIndexWithIntId("GetOneExistingDocumentWithIntegerIdTest");
-            var updateSortable = await index.UpdateSortableAttributesAsync(new[] { "id" });
-            updateSortable.TaskUid.Should().BeGreaterOrEqualTo(0);
-            await index.WaitForTaskAsync(updateSortable.TaskUid);
-
-            var documents =
-                await index.GetDocumentsAsync<MovieWithIntId>(new DocumentsQuery()
-                {
-                    Sort = new List<string> { "id:asc" }
-                });
-
-            Assert.Equal(7, documents.Results.Count());
-            documents.Results.Should().BeInAscendingOrder(x => x.Id);
-        }
-
-        [Fact]
-        public async Task GetMultipleExistingDocumentWithDescSort()
+        [Theory]
+        [InlineData("id:asc", true)]
+        [InlineData("id:desc", false)]
+        public async Task GetMultipleExistingDocumentsWithIdSort(string sort, bool ascending)
         {
-            var index = await _fixture.SetUpBasicIndexWithIntId("GetOneExistingDocumentWithIntegerIdTest");
+            var index = await _fixture.SetUpBasicIndexWithIntId($"GetMultipleExistingDocumentsWithIdSort_{(ascending ? "Asc" : "Desc")}Test");
             var updateSortable = await index.UpdateSortableAttributesAsync(new[] { "id" });
             updateSortable.TaskUid.Should().BeGreaterOrEqualTo(0);
             await index.WaitForTaskAsync(updateSortable.TaskUid);
 
-            var documents =
-                await index.GetDocumentsAsync<MovieWithIntId>(new DocumentsQuery()
-                {
-                    Sort = new List<string> { "id:desc" }
-                });
+            var results = (await index.GetDocumentsAsync<MovieWithIntId>(
+                new DocumentsQuery { Sort = new List<string> { sort } }
+            )).Results.ToList();
 
-            Assert.Equal(7, documents.Results.Count());
-            documents.Results.Should().BeInDescendingOrder(x => x.Id);
+            Assert.Equal(7, results.Count);
+            if (ascending)
+                results.Should().BeInAscendingOrder(x => x.Id);
+            else
+                results.Should().BeInDescendingOrder(x => x.Id);
         }
 
         [Fact]
-        public async Task GetMultipleExistingDocumentWithMultiSort()
+        public async Task GetMultipleExistingDocumentsWithMultiSort()
         {
-            var index = await _fixture.SetUpBasicIndexWithIntId("GetOneExistingDocumentWithIntegerIdTest");
+            var index = await _fixture.SetUpBasicIndexWithIntId("GetMultipleExistingDocumentsWithMultiSort");
             var updateFilterable = await index.UpdateFilterableAttributesAsync(new[] { "genre" });
             updateFilterable.TaskUid.Should().BeGreaterOrEqualTo(0);
             await index.WaitForTaskAsync(updateFilterable.TaskUid);
@@ -691,20 +676,29 @@ public async Task GetMultipleExistingDocumentWithMultiSort()
             updateSortable.TaskUid.Should().BeGreaterOrEqualTo(0);
             await index.WaitForTaskAsync(updateSortable.TaskUid);
 
-            var documents =
-                await index.GetDocumentsAsync<MovieWithIntId>(new DocumentsQuery()
+            var results = (await index.GetDocumentsAsync<MovieWithIntId>(
+                new DocumentsQuery
                 {
                     Filter = "genre IN ['SF','Action']",
                     Sort = new List<string> { "genre:asc", "name:desc" }
-                });
+                }
+            )).Results.ToList();
 
-            Assert.Equal(4, documents.Results.Count());
-            var first = documents.Results.First();
+            Assert.Equal(4, results.Count);
+            var first = results.First();
             first.Genre.Should().Be("Action");
             first.Name.Should().Be("Spider-Man");
-            var last = documents.Results.Last();
+            var last = results.Last();
             last.Genre.Should().Be("SF");
             last.Name.Should().Be("Harry Potter");
+
+            results.Should().BeInAscendingOrder(x => x.Genre);
+            var split = results.FindIndex(x => x.Genre == "SF");
+            split.Should().BeGreaterThan(0); // at least one Action before SF
+            results.Take(split).Should().OnlyContain(x => x.Genre == "Action");
+            results.Skip(split).Should().OnlyContain(x => x.Genre == "SF");
+            results.Take(split).Should().BeInDescendingOrder(x => x.Name);
+            results.Skip(split).Should().BeInDescendingOrder(x => x.Name);
         }
 
         [Fact]