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.

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

Add integration tests for durable requests #36

Open
rkrzr opened this issue Mar 12, 2018 · 0 comments
Open

Add integration tests for durable requests #36

rkrzr opened this issue Mar 12, 2018 · 0 comments

Comments

@rkrzr
Copy link
Contributor

rkrzr commented Mar 12, 2018

Original issue: #25
Associated PR: #27

This feature has currently no integration tests. We should write some.

Idea from @fatho to test this:

You could spam it with updates to some key /foo/a and then a last update to /foo/b, then ask for the value of /foo/b. If processing all the updates takes long enough, it should sometimes return the old, sometimes the new value for /foo/b without the flag. But with that flag, it should always return the updated value.

Alternatively, we could perhaps also add some artificial delays in the processing of requests with eventual consistency. This would make the tests more deterministic, but it would clutter the code a bit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant