Skip to content

Commit d060bb0

Browse files
committed
fix test.
clarify doc comment i now think the XTRIM documentation is saying that an entry at exactly MINID is kept. https://redis.io/docs/latest/commands/xtrim/
1 parent 97903a1 commit d060bb0

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/StackExchange.Redis/Interfaces/IDatabase.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2779,7 +2779,7 @@ IEnumerable<SortedSetEntry> SortedSetScan(
27792779
/// Trim the stream to a specified minimum timestamp.
27802780
/// </summary>
27812781
/// <param name="key">The key of the stream.</param>
2782-
/// <param name="minId">All entries less than minId will be removed.</param>
2782+
/// <param name="minId">All entries with an id (timestamp) earlier minId will be removed.</param>
27832783
/// <param name="flags">The flags to use for this operation.</param>
27842784
/// <returns>The number of messages removed from the stream.</returns>
27852785
/// <remarks><seealso href="https://redis.io/topics/streams-intro"/></remarks>

tests/StackExchange.Redis.Tests/StreamTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1932,7 +1932,7 @@ public void StreamTrimMinId()
19321932
var numRemoved = db.StreamTrim(key, 1111111111, CommandFlags.None);
19331933
var len = db.StreamLength(key);
19341934

1935-
Assert.Equal(2, numRemoved);
1935+
Assert.Equal(1, numRemoved);
19361936
Assert.Equal(1, len);
19371937
}
19381938

0 commit comments

Comments
 (0)