diff options
Diffstat (limited to 'crates/tor-proto/src/relay/channel')
| -rw-r--r-- | crates/tor-proto/src/relay/channel/create_handler.rs | 24 |
1 files changed, 6 insertions, 18 deletions
diff --git a/crates/tor-proto/src/relay/channel/create_handler.rs b/crates/tor-proto/src/relay/channel/create_handler.rs index d4a73a609..00b381dbf 100644 --- a/crates/tor-proto/src/relay/channel/create_handler.rs +++ b/crates/tor-proto/src/relay/channel/create_handler.rs @@ -814,12 +814,8 @@ mod test { conn_inspector.client_cell().await.unwrap().msg().cmd(), ChanCmd::DESTROY, ); - // TODO(relay): I think the relay shouldn't be sending a DESTROY back to the client. - // https://gitlab.torproject.org/tpo/core/arti/-/work_items/2648 - assert_eq!( - conn_inspector.relay_cell().await.unwrap().msg().cmd(), - ChanCmd::DESTROY, - ); + // The relay shouldn't be sending a DESTROY back to the client + assert!(conn_inspector.try_relay_cell().is_none()); }); } @@ -903,12 +899,8 @@ mod test { conn_inspector.client_cell().await.unwrap().msg().cmd(), ChanCmd::DESTROY, ); - // TODO(relay): I think the relay shouldn't be sending a DESTROY back to the client. - // https://gitlab.torproject.org/tpo/core/arti/-/work_items/2648 - assert_eq!( - conn_inspector.relay_cell().await.unwrap().msg().cmd(), - ChanCmd::DESTROY, - ); + // The relay shouldn't be sending a DESTROY back to the client + assert!(conn_inspector.try_relay_cell().is_none()); } }); } @@ -960,12 +952,8 @@ mod test { conn_inspector.client_cell().await.unwrap().msg().cmd(), ChanCmd::DESTROY, ); - // TODO(relay): I think the relay shouldn't be sending a DESTROY back to the client. - // https://gitlab.torproject.org/tpo/core/arti/-/work_items/2648 - assert_eq!( - conn_inspector.relay_cell().await.unwrap().msg().cmd(), - ChanCmd::DESTROY, - ); + // The relay shouldn't be sending a DESTROY back to the client + assert!(conn_inspector.try_relay_cell().is_none()); } }); } |