[do not merge][lexical-yjs][lexical-playground] Chore: Add test for collaboration undo bug #6670
Conversation
mike-atticus
requested review from
zurfyx,
fantactuka,
acywatson,
Fetz,
ivailop7,
Sahejkm and
potatowagon
as code owners
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
facebook-github-bot
added
the
CLA Signed
size-limit report 📦
|
| @@ -230,4 +230,92 @@ test.describe('Collaboration', () => { | |||
| focusPath: [1, 1, 0], | |||
| }); | |||
| }); | |||
|
|
|||
| test('Undo with two collaborators editing same paragraph', async ({ | |||
There was a problem hiding this comment.
this is failing on the ci :(
the failure is reproducible across different envs so i dont think its a flaky failure
There was a problem hiding this comment.
That's the point of this PR, to demonstrate that bug
There was a problem hiding this comment.
Yea I created this test in case it helps others with reproducing the bug and confirming a fix, but it's not something that can be merged on its own. Let me know if I should switch this to a draft PR for now, if that's clearer for PR queue management.
There was a problem hiding this comment.
i see, thanks for filing a detailed issue. if its not meant to be merged, converting to draft or adding a [do not merge] to title would work
mike-atticus
changed the title
Description
Adds a test for the issue #6614 where using undo in a collaborative session results in the Lexical editor content getting out of sync with the Yjs doc, which then leads to loss of document content.
Test plan
Before
No tests to detect the bugs that lead to unexpected loss of editor content after using undo.
After
Playwright test added which uses the side-by-side collaborative playground editors to reproduce the bug after using undo.
Note that this test actually detects two separate bugs: