-
Notifications
You must be signed in to change notification settings - Fork 12
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
Initial supervision tests #43
Comments
Hmm here's another weird case I thought of: A Wait, just thinking, nm. If you make a call to a daemon actor the only way this could happen is if you were to stop iterating a remote streaming function (any other type of call should error on the Probably needs tests for all of this right now lol! See #87 for further discussion on how this might be solved. Either way we need tests to verify the current behavior. |
In an effort towards #43. This completes the first major "bullets" worth of tests described in that issue.
In an effort towards #43. This completes the first major bullet's worth of tests described in that issue.
Aha! Catching more undefined behavior with this endevour :D Working on the nested I've created #88 to address this. |
Add a test to verify that `trio.MultiError`s are properly propagated up a simple actor nursery tree. We don't have any exception marshalling between processes (yet) so we can't validate much more then a simple 2-depth tree. This satisfies the final bullet in #43. Note I've limited the number of subactors per layer to around 5 since any more then this seems to break the `multiprocessing` forkserver; zombie subprocesses seem to be blocking teardown somehow... Also add a single depth fast fail test just to verify that it's the nested spawning that triggers this forkserver bug.
#42 brings in proper
trio.MultiError
support and more deterministic cancellation semantics.More tests are needed to ensure this system is rock solid before moving onto adding different supervision strategies as in #22.
Some I can think of off-hand that aren't in the test suite are:
run_in_actor()
, xstart_actor()
start_actor()
andrun_in_actor()
MultiError
up subactor nursery treesMore to come...
The text was updated successfully, but these errors were encountered: