tests: added unit test in fedimint-core/src/runtime.rs

fedimint / fedimint

Federated E-Cash Mint

https://fedimint.org/

MIT License

536 stars 209 forks source link

tests: added unit test in fedimint-core/src/runtime.rs #5141

Open sivasathyaseeelan opened 2 weeks ago

sivasathyaseeelan commented 2 weeks ago

Fix

This PR adds unit test fin fedimint/fedimint-core/src/runtime.rs and closes #5140

sivasathyaseeelan commented 2 weeks ago

Please don't use sleep, or at least not with 1 second values. CI time is a scarce resource. Where you really need to guarantee sleep, maybe use 1 millisecond. where you need coordination between threads use https://docs.rs/tokio/latest/tokio/sync/oneshot/index.html to unblock things.

Hi @dpc, I have changed from 1 second to 1 millisecond. Can you please review this pr again?