Skip to content

Adding more tests to WebTransport #42786

Open
@Daniel-Genkin-MS-2

Description

@Daniel-Genkin-MS-2

Is there an existing issue for this?

  • I have searched the existing issues

Is your feature request related to a problem? Please describe the problem.

I implemented some basic unit tests as well as some handshake tests in my PR which added WebTransport over HTTP/3 to Kestrel. However, adding more InMemory and some Functional tests would be good to fill in the gaps.

Describe the solution you'd like

Some good tests would be:

  • verifying that the data is properly stripped of stream headers.
  • verifying that the logs throughout the connection are reliable.
  • verifying that streams are disposed of properly in all cases where the connection, session, or stream is closed or aborted.

Some good integration tests would be:

  • starting a connection, accepting a unidirectional stream, reading data, closing.
  • starting a connection, accepting a bidirectional stream, reading data, sending data, closing.
  • starting a connection, establishing an output unidirectional stream, sending data, closing.
  • starting a connection, verifying that datagrams are not supported and handled elegantly.

Additional context

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    area-networkingIncludes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions