Memory Pool not thread safe ? about zenject HOT 4 OPEN

@Mathijs-Bakker I see what you meant. The test code given only works in NUnit 3 which has multithread capability. If you try to run it in Unity , Assert does not support async. If you run it outside unity , you need to expose StaticMemoryPoolRegistry which is currently locked behind a UNITY_EDITOR directive (I think this should be changed as this is very handy tool to perform test code)

from zenject.

Sorry but I did not succeed to run your tests. But looking at your code I see:
In your test you have a Thread.Sleep().
Wouldn't Task.Delay() a better option?
As Task.Delay acts in a very different way than Thread.Sleep. Task.Delay will create a task which will complete after a time delay. Task.Delay is not blocking the calling thread.

But yeah, Zenject is not totally thread safe in certain cases:
issue#35

from zenject.

Sorry but I did not succeed to run your tests

As in you got an error ? Because that is the current problem with the test. It should not fail.

But looking at your code I see:
In your test you have a Thread.Sleep().
Wouldn't Task.Delay() a better option?

I do not know if it is better. The test as written replicates the same error that I have in my project. (and I do not use Thread.Sleep() in the project)

from zenject.

Sorry but I did not succeed to run your tests

I meant the testing code.

from zenject.