Skip to content

test(transport): tcp,quic should allow multiple local addresses #2194

@vladopajic

Description

@vladopajic

we should extend basic_test with additional test case that asserts that transport can be started with multiple addresses.
this should work for tcp and quic and for all other transports if it works.

do this after #2193 is merged


test cases:

  • default test case: where multiple addresses are provided; dial from clients and ensure that dialed connection works
    example: test_quic.nim::should allow multiple local addresse
  • cancel dialing: same as above, but here we dial (with timeout so it gets cancelled) and do not drive accept so connection is not created (dial is cancelled). in this case we need to ensure that all already established connections will still work and that connections that we attempted to dial will not work (future will be canceled).
  • cancel accept: similar case to above, here we want to ensure that allready established connections will still work and that no new connections will be made because accept fut was cancelled. example: basic_test.nim::handle accept cancellation
  • stopping transport while accepting: - should stop transport, close all connections; future that was blocked on accept should throw - transport stopped
  • please consider and suggest other edge cases that can happen when stopping/canceling transport/futures

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

Status

new

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions