Fix unit tests - Githubissues

y-crdt / ydotnet

.NET bindings for yrs.

MIT License

35 stars 8 forks source link

Fix unit tests #71

Closed LSViana closed 10 months ago

LSViana commented 10 months ago

Fixed the issues by:

Replacing the direct UndoEventNative parameter with an nint (which is then used to read the struct instance) in the ObserveAddedCallback and ObservePoppedCallback delegates
Add [StructLayout(Sequential)] to the struct UndoEventNative (optional, this was meant to fix a warning)

Closes #70.

LSViana commented 10 months ago

I won't wait for a reviewer because the fix is simple and should unblock moving this repo into the y-crdt organization soon (since the tests are passing now).