Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add test for joining a new federated room #31

Merged
merged 15 commits into from
Aug 29, 2022

Conversation

MadLittleMods
Copy link
Contributor

@MadLittleMods MadLittleMods commented Jun 29, 2022

Add test for joining a new federated room and making sure the messages are available (homeserver should backfill).

Synapse changes: matrix-org/synapse#13205, matrix-org/synapse#13320

Dev notes

docker build -t matrixdotorg/synapse -f docker/Dockerfile .

@MadLittleMods MadLittleMods added the T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. label Jun 29, 2022
test/e2e-tests.js Outdated Show resolved Hide resolved
test/README.md Outdated Show resolved Hide resolved
test/e2e-tests.js Outdated Show resolved Hide resolved
MadLittleMods and others added 5 commits August 29, 2022 14:26
…of the last message

All of the messages were at the same time, so the homeserver, just returned the first one
but in reality, it's a series of messages that happen one after the other so it should return
the last one.

We adjust the timestamps so they actually appear as a series. Although we think
about changing the sort on the date tiebreak when looking backwards in Synapse.
@MadLittleMods MadLittleMods marked this pull request as ready for review August 29, 2022 23:12
@MadLittleMods MadLittleMods merged commit e9d13db into main Aug 29, 2022
@MadLittleMods MadLittleMods deleted the madlittlemods/test-for-join-new-federated-room branch April 26, 2023 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant