Skip to content

[release/8.0] Fix ResponseCompressionMiddleware tests #60884

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: release/8.0
Choose a base branch
from
Open

[release/8.0] Fix ResponseCompressionMiddleware tests #60884

Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
3976a49
Fix ResponseCompressionMiddleware tests
wtgodbe Mar 11, 2025
e59d900
Update ResponseCompressionMiddlewareTest.cs
wtgodbe Mar 11, 2025
ec4420c
Update ResponseCompressionMiddlewareTest.cs
wtgodbe Mar 11, 2025
995f94b
Update content length assertion in test
wtgodbe Mar 11, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions src/Middleware/ResponseCompression/test/ResponseCompressionMiddlewareTest.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1325,6 +1325,8 @@ private static void CheckResponseNotCompressed(HttpResponseMessage response, lon
Assert.NotNull(response.Content.Headers.GetValues(HeaderNames.ContentMD5));
Assert.Empty(response.Content.Headers.ContentEncoding);
Assert.Equal(expectedBodyLength, response.Content.Headers.ContentLength);
// Check that compressed size is within a reasonable range
Assert.InRange(response.Content.Headers.ContentLength, (int)(expectedBodyLength! - 5), (int)(expectedBodyLength! + 5));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Assert.InRange(response.Content.Headers.ContentLength, (int)(expectedBodyLength! - 5), (int)(expectedBodyLength! + 5));
Assert.InRange(response.Content.Headers.ContentLength, (int)(expectedBodyLength.Value - 5), (int)(expectedBodyLength.Value + 5));

}

private static void AssertLog(WriteContext log, LogLevel level, string message)
Expand Down
Loading