dotnet / runtime

.NET is a cross-platform runtime for cloud, mobile, desktop, and IoT apps.
https://docs.microsoft.com/dotnet/core/
MIT License
15.07k stars 4.69k forks source link

Add tests with LoopbackServer for web sockets #73849

Open greenEkatherine opened 2 years ago

greenEkatherine commented 2 years ago

We should increase test scenarios for web sockets that are running with LoopbackServer -- first of all, for web sockets over HTTP/2 and then for HTTP/1.1.

Now many tests are running only in outerloop, which runs on request and not in every CI. This option is unavailable for web sockets over HTTP/2 because the external server has not yet supported it.

To achieve more test coverage, Http2LoopbackServer should be adapted to work with web sockets i.e., it should have a special stream wrapper to decode received data easily.

ghost commented 2 years ago

Tagging subscribers to this area: @dotnet/ncl See info in area-owners.md if you want to be subscribed.

Issue Details
We should increase test scenarios for web sockets that are running with LoopbackServer -- first of all, for web sockets over HTTP/2 and then for HTTP/1.1. Now many tests are running only in outerloop, which runs on request and not in every CI. This option is unavailable for web sockets over HTTP/2 because the external server has not yet supported it. To achieve more test coverage, Http2LoopbackServer should be adapted to work with web sockets i.e., it should have a special stream wrapper to decode received data easily.
Author: greenEkatherine
Assignees: greenEkatherine
Labels: `area-System.Net.Http`, `test-enhancement`
Milestone: -