element-hq / element-hq/dendrite

Fix failing /sync tests

Open
#1,324 1 comment 0 reactions 0 assignees View on GitHub
are-we-synapse-yet C-Sync-API help wanted T-Defect
Dominant language
Go
Stars
967
Forks
102
PR merge metrics
No merged PRs in 30d

Description

*This issue was originally created by [**@kegsay**](https://github.com/kegsay) at .*

They should be passing but for some reason are not.

Sytests:
```
× Newly joined room has correct timeline in incremental sync
× Newly joined room includes presence in incremental sync
× When user joins a room the state is included in the next sync
× When user joins a room the state is included in a gapped sync
× When user joins and leaves a room in the same batch, the full state is still included in the next sync
× Current state appears in timeline in private history with many messages after
```

Contributor guide

Open the contributing guide

Research direction

Start by running the six listed Sytests for incremental sync and inspect the /sync implementation and its handling of joined-room state, presence, gapped syncs, and private history. Done means all six currently failing Sytests pass without regressions.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
backend-api-design, testing
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.