Skip to content

Commit dd31e6f

Browse files
test(search): skip TestSearchLimitOffset on RESP3 for standalone too
FT.SEARCH on RESP3 ignores the LIMIT offset server-side (RediSearch#10369), returning [0, offset+count) instead of [offset, offset+count). The existing skip only covered cluster+RESP3, but the same module behaviour reproduces on a standalone Redis Enterprise database, so TestSearchLimitOffset(standalone,Resp3) fails 10/10. Broaden the skip to all RESP3 runs (cluster and standalone) until the module bug is resolved; standalone+RESP3 offset behaviour still needs module-team confirmation.
1 parent 5173f13 commit dd31e6f

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

‎tests/NRedisStack.Tests/Search/SearchTests.cs‎

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -78,12 +78,13 @@ public void TestSearchLimitOffset(string endpointId)
7878
{
7979
SkipClusterPre8(endpointId);
8080

81-
// Cluster + RESP3 ignores the LIMIT offset server-side: FT.SEARCH returns the window
81+
// RESP3 ignores the LIMIT offset server-side: FT.SEARCH returns the window
8282
// [0, offset+count) instead of [offset, offset+count). This is a RediSearch module bug,
83-
// not a client bug (the raw wire reply already contains the wrong documents); skip until fixed.
83+
// not a client bug (the raw wire reply already contains the wrong documents). It reproduces
84+
// on both cluster and standalone Redis Enterprise databases, so skip both until fixed.
8485
// https://github.com/RediSearch/RediSearch/issues/10369
85-
Assert.SkipWhen(endpointId == EndpointsFixture.Env.Cluster && TestContext.Current.GetRunProtocol().IsResp3(),
86-
"RediSearch#10369: FT.SEARCH on cluster+RESP3 ignores the LIMIT offset");
86+
Assert.SkipWhen(TestContext.Current.GetRunProtocol().IsResp3(),
87+
"RediSearch#10369: FT.SEARCH on RESP3 ignores the LIMIT offset (cluster and standalone)");
8788

8889
IDatabase db = GetCleanDatabase(endpointId);
8990
var ft = db.FT();

0 commit comments

Comments
 (0)