Remove FIXME in test since we won't fix it this way

This commit is contained in:
Torsten Grote
2021-06-30 16:57:32 -03:00
parent dc3ba3d8f0
commit 9668f62c6a

View File

@@ -233,9 +233,12 @@ public class TransportKeyAgreementIntegrationTest
// but not via the new duplex transport // but not via the new duplex transport
assertTrue(alice.getKeyManager() assertTrue(alice.getKeyManager()
.canSendOutgoingStreams(bobId, SIMPLEX_TRANSPORT_ID)); .canSendOutgoingStreams(bobId, SIMPLEX_TRANSPORT_ID));
// FIXME normally Alice should not be able to already send streams // Normally, Alice should not be able to send streams already.
// assertFalse(alice.getKeyManager() // However, she does already derive keys for the transport.
// .canSendOutgoingStreams(bobId, newTransportId)); // The UI checks RemovableDriveManager#isTransportSupportedByContact()
// in practice to prevent sending streams that Bob can't decrypt.
assertTrue(alice.getKeyManager()
.canSendOutgoingStreams(bobId, newTransportId));
// Bob restarts and comes back with the new transport. // Bob restarts and comes back with the new transport.
bob = restartWithNewTransport(bob, bobDir, bobIdentity); bob = restartWithNewTransport(bob, bobDir, bobIdentity);