From 9ec4d88fbe7da35305d4609de66ed17d3f618e4c Mon Sep 17 00:00:00 2001 From: Ben Adams Date: Sat, 19 Nov 2016 14:15:37 +0000 Subject: [PATCH] Improve Skip coverage --- .../MemoryPoolIteratorTests.cs | 21 ++++++++++++------- 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/test/Microsoft.AspNetCore.Server.KestrelTests/MemoryPoolIteratorTests.cs b/test/Microsoft.AspNetCore.Server.KestrelTests/MemoryPoolIteratorTests.cs index 1416f652df..0cb46de20d 100644 --- a/test/Microsoft.AspNetCore.Server.KestrelTests/MemoryPoolIteratorTests.cs +++ b/test/Microsoft.AspNetCore.Server.KestrelTests/MemoryPoolIteratorTests.cs @@ -518,20 +518,25 @@ namespace Microsoft.AspNetCore.Server.KestrelTests public void SkipThrowsWhenSkippingMoreBytesThanAvailableInMultipleBlocks() { // Arrange - var block = _pool.Lease(); - block.End += 3; + var firstBlock = _pool.Lease(); + firstBlock.End += 3; - var nextBlock = _pool.Lease(); - nextBlock.End += 2; - block.Next = nextBlock; + var middleBlock = _pool.Lease(); + middleBlock.End += 1; + firstBlock.Next = middleBlock; - var scan = block.GetIterator(); + var finalBlock = _pool.Lease(); + finalBlock.End += 2; + middleBlock.Next = finalBlock; + + var scan = firstBlock.GetIterator(); // Act/Assert Assert.ThrowsAny(() => scan.Skip(8)); - _pool.Return(block); - _pool.Return(nextBlock); + _pool.Return(firstBlock); + _pool.Return(middleBlock); + _pool.Return(finalBlock); } [Theory]