Conversation
There was a problem hiding this comment.
Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.
Comments suppressed due to low confidence (1)
src/libraries/System.Net.Http.WinHttpHandler/tests/FunctionalTests/BidirectionStreamingTest.cs:99
- [nitpick] Consider adding a comment specifying when to re-enable this test (e.g. for .NET 9.0+), so that test coverage is restored once the underlying issue is fixed.
[ActiveIssue("https://github.com/dotnet/runtime/issues/112700")]
|
Tagging subscribers to this area: @dotnet/ncl |
|
Test only change, tell mode, adding servicing approved. |
|
/azp list |
|
/azp run runtime |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
Disabling this test will have no effect about the issue, it will keep popping from some other test, because issue is on the hot path. |
|
I second Ahmets concern, are we sure this happens only on the test that is being disabled? |
47dd1c8 to
818c1a8
Compare
|
/azp run runtime |
1 similar comment
|
/azp run runtime |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Disabling test failing on assert in #112700.
This is already fixed in .NET 9.0+ by #93984.
cc @wfurt @karelz
Customer Impact
Regression
Testing
CI
Risk
Very low - test only change.