Skip to content

Add unit tests for parse_stripped_state_event - #20136

Open
guillemo12 wants to merge 4 commits into
element-hq:developfrom
guillemo12:feature/state-event-tests
Open

Add unit tests for parse_stripped_state_event#20136
guillemo12 wants to merge 4 commits into
element-hq:developfrom
guillemo12:feature/state-event-tests

Conversation

@guillemo12

@guillemo12 guillemo12 commented Aug 21, 2026

Copy link
Copy Markdown

This PR adds a suite of tests for Synapse state events. It covers key scenarios around creation, updates and state consistency to prevent regressions in event processing and serialization. The primary goal is to increase test coverage and catch issues earlier in the development cycle.

Pull Request Checklist

  • Pull request is based on the develop branch
  • Pull request includes a changelog file. The entry should:
    • Be a short description of your change which makes sense to users. "Fixed a bug that prevented receiving messages from other servers." instead of "Moved X method from EventStore to EventWorkerStore.".
    • Use markdown where necessary, mostly for code blocks.
    • End with either a period (.) or an exclamation mark (!).
    • Start with a capital letter.
    • Feel free to credit yourself, by adding a sentence "Contributed by @github_username." or "Contributed by [Your Name]." to the end of the entry.
  • Code style is correct (run the linters)

@guillemo12
guillemo12 requested a review from a team as a code owner August 21, 2026 10:13

@reivilibre reivilibre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@reivilibre
reivilibre self-requested a review August 27, 2026 13:56

@reivilibre reivilibre left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

CI is failing, are you happy to look into it?

@guillemo12

Copy link
Copy Markdown
Author

I think the issue was an improperly formatted line ending; I've just fixed it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants