chart/5.3.5
fisx
tagged this
23 May 10:09
* Refactor. * Refactor. * Refactor. * Make test case fail where it should pass. * Cleanup failing test cases. - add old run traces (with approved device) back * FUTUREWORK. * Changelog. * Make failing test case pass. Pending LH devices means user *has* given consent, not the opposite! * Fix terminology. * Make default settings explicit. * Renames; fix more LH logic. transitioning from disabled to pending doesn't block any connections, but from pending to active does. * Rename. * Fixup HEAD~2 * Remove dead code. * testLHMessageExchange: Assert that message exchange works during all stages of multiple people approving the device * Fix comment wording * Delete redundant constraints * testLHMessageExchange: Ensure correct client is used to send messages * testLHNoConsentBlockOne2OneConv: Break the test in 2 for simplicity Also fix assertions about what happens when LH devices are pending * brig: Do not cause LH conflict when a user has pending device while creating connections * testLHPreventAddingNonConsentingUsers: Users are only kicked after approving the LH device * hlint * integration/connectTwoUsers: Don't worry too much if users are already connected * galley: Allow non-lh-consenting users to be added to a conv when lh is pending on a member --------- Co-authored-by: Akshay Mankar <[email protected]>