-
Notifications
You must be signed in to change notification settings - Fork 103
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 Testing Utility for Swarm #100
Comments
That'd be great! |
I'm 👍 , the question is where we would put this utility. |
I think we should start by defining what those would contain. Those are the things that come into my mind:
|
BTW: I would also include a
With warming up I mean to block & wait until swarm is fully synced. |
Yes, well actually @bitwalker and I are planning to refactor all test cases to make Swarm setup more reliable and less work, using his https://github.com/bitwalker/ex_unit_clustered_case library. So maybe hold off doing anything big until this has settled. |
Yeah, revisiting this after thinking on it a bit more, I think a lot of what we have planned with reworking the existing test suite covers the goals stated here. Namely, we would be in a position to test handoff and conflict resolution much more easily, amongst other things. The other big thing coming down the pipe is reworking Swarm to no longer be a singleton process, which would allow us to simulate some things in parallel when we don't need to spin up a cluster of nodes for that purpose. I will leave this open for now though, and we can revisit once some of these changes are in place. |
We recently changed a big part of the supervision of quantum (add job scheduler library) to swarm if it is used in a cluster.
One thing that was missing for us is a testing utility to make sure that the new callbacks for swarm actually work.
I implemented a really easy helper: https://github.com/quantum-elixir/quantum-core/pull/369/files#diff-51021d33872c9243af9315e6b3809e77
Would you be willing so add something like that (but more complete) to the library?
The text was updated successfully, but these errors were encountered: