Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Support MSC3086 asserted identity #5886

Merged
merged 7 commits into from
Apr 28, 2021
Merged

Support MSC3086 asserted identity #5886

merged 7 commits into from
Apr 28, 2021

Conversation

dbkr
Copy link
Member

@dbkr dbkr commented Apr 19, 2021

Add support for switching calls between rooms based on MSC3086 asserted identity events. Requires config option documented in element-hq/element-web#17008.

Also includes our first tests for CallHandler which introduces a bit more stuff for testing. Updates to the new method to create call objects, then stubs this method out in the tests.

Requires matrix-org/matrix-js-sdk#1674

dbkr added 4 commits April 19, 2021 20:30
This is out first CallHandler test(!) Switches react-sdk to use
createCall on the client object so we can stub this out in the test.
Add a bunch more stubs to the test client.

There's more stuff in this test that has scope to be used more
widely, like waiting for a certain dispatch and mocking out rooms
with particular sets of users in them: we could consider moving these
out to test utils if we wanted.
@dbkr dbkr marked this pull request as ready for review April 23, 2021 14:38
@dbkr dbkr requested a review from a team April 23, 2021 15:27
Copy link
Collaborator

@jryans jryans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, and thanks for adding tests here! 😄

src/CallHandler.tsx Outdated Show resolved Hide resolved
test/CallHandler-test.ts Outdated Show resolved Hide resolved
dbkr and others added 3 commits April 27, 2021 18:55
@dbkr dbkr merged commit c95c1ae into develop Apr 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants