Skip to content

Commit 13eeb3a

Browse files
committed
Test fixes
1 parent 57769f8 commit 13eeb3a

2 files changed

Lines changed: 6 additions & 5 deletions

File tree

src/Lucene.Net.Tests/Store/TestMultiMMap.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2613,13 +2613,13 @@ public void TestOpenSlice_OutOfBounds_Throws()
26132613
using var slicer = mmapDir.CreateSlicer("f", NewIOContext(Random));
26142614

26152615
// Negative offset.
2616-
Assert.Throws<ArgumentException>(() => slicer.OpenSlice("neg-off", -1, 4));
2616+
Assert.Throws<ArgumentOutOfRangeException>(() => slicer.OpenSlice("neg-off", -1, 4));
26172617
// Negative length.
2618-
Assert.Throws<ArgumentException>(() => slicer.OpenSlice("neg-len", 0, -1));
2618+
Assert.Throws<ArgumentOutOfRangeException>(() => slicer.OpenSlice("neg-len", 0, -1));
26192619
// offset + length past end of file.
2620-
Assert.Throws<ArgumentException>(() => slicer.OpenSlice("past-end", 8, 10));
2620+
Assert.Throws<ArgumentOutOfRangeException>(() => slicer.OpenSlice("past-end", 8, 10));
26212621
// offset alone past end of file.
2622-
Assert.Throws<ArgumentException>(() => slicer.OpenSlice("off-past-end", 17, 0));
2622+
Assert.Throws<ArgumentOutOfRangeException>(() => slicer.OpenSlice("off-past-end", 17, 0));
26232623

26242624
// Edge: offset == length == 0 is fine on a non-empty file.
26252625
using (var s = slicer.OpenSlice("empty", 0, 0)) { Assert.AreEqual(0L, s.Length); }

src/Lucene.Net/Store/MMapDirectory.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -246,7 +246,8 @@ public IndexInputSlicerAnonymousClass(MMapDirectory outerInstance, string file,
246246
public override IndexInput OpenSlice(string? sliceDescription, long offset, long length)
247247
{
248248
outerInstance.EnsureOpen();
249-
if ((ulong)offset >= (ulong)mapping.Length)
249+
// LUCENENET NOTE: TestSeekSliceZero invariant allows 0 offset with 0 length
250+
if (offset != 0 && (ulong)offset >= (ulong)mapping.Length)
250251
throw new ArgumentOutOfRangeException(nameof(offset),
251252
$"slice() {sliceDescription ?? "(null)"} offset out of bounds: " +
252253
$"offset={offset},length={length},fileLength={mapping.Length}: {this}");

0 commit comments

Comments
 (0)