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.

Sign up for GitHub

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

Jump to bottom

Improve test coverage for SyncManager #206

Open
frankhinek opened this issue Aug 30, 2023 · 0 comments
Open

Improve test coverage for SyncManager #206

frankhinek opened this issue Aug 30, 2023 · 0 comments
Labels
good first issue Good for newcomers testing related to new or existing tests
Milestone
@web5/api 0.10

Comments

@frankhinek
Copy link
Contributor

Currently tests for SyncManager cover the most basic cases of sync push and pull with a single and multiple identities. Additional tests should be added to cover:

  • sync with multiple remote DWNs
  • sync when records are deleted from local and remote DWNs
  • validate that sync does not introduce any "echoes" wherein a message that already exists on a remote DWN is not pushed again (resulting in an 409 response)
@frankhinek frankhinek added good first issue Good for newcomers testing related to new or existing tests labels Aug 30, 2023
@frankhinek frankhinek added this to the 0.9 milestone Aug 30, 2023
@frankhinek frankhinek mentioned this issue Aug 30, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers testing related to new or existing tests
Projects
None yet
Milestone
@web5/api 0.10
Development

No branches or pull requests
1 participant
@frankhinek