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

Test how the user directory handles remote users #11105

Open
DMRobertson opened this issue Oct 18, 2021 · 0 comments
Open

Test how the user directory handles remote users #11105

DMRobertson opened this issue Oct 18, 2021 · 0 comments
Labels
T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.

Comments

@DMRobertson
Copy link
Contributor

All of the unit tests check the behaviour of the directory for local users. The implementation of the directory has many code paths that differ for local and remote users. We should have test coverage of these.

Fetching actual profiles over federation to properly fix #5677 will be involved and require mocking if we're going to make it a unit test. But it should be fairly straightforward to add tests that inspect the room-sharing tables for remote users. And thankfully, we don't have the (painful) business of directory exclusion to worry about for remote users.

@clokep clokep added the T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. label Oct 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
Projects
None yet
Development

No branches or pull requests

2 participants