Add test suite to ensure the correctness of `AsyncRuntime` implementation.

[github-actions[bot]]

👋 Thanks for opening this issue!

Get help or engage by:

• /help : to print help messages.
• /assignme : to assign this issue to you.

[SteveLauC]

I wonder what kinds of tests we should add for an implementation of AsyncRuntime, for this test suite, I think we should verify the functions and primitives will work correctly.

I also want to run some integration tests to check if Openraft would work with the runtime as well.

Thoughts on this?

[drmingdrmer]

For example for Mutex, it should provide exclusive accessing to some value. Such as: https://github.com/drmingdrmer/openraft/blob/bc4b4e54661308ae47b88b39373f82c54dd5780c/openraft/src/sync/mutex.rs#L130-L168

And similar to other components. At least the very basic behavior should be guaranteed by these tests. There is also a test suite to ensure the behaviors of RaftLogStorage:

https://github.com/drmingdrmer/openraft/blob/113-mutex/openraft/src/testing/suite.rs

[SteveLauC]

At least the very basic behavior should be guaranteed by these tests.

Will give this a try this morning.

There is also a test suite to ensure the behaviors of RaftLogStorage

Yeah, I am thinking about putting the test suite for logs and state machine in a module log, and adding another module runtime for the tests of AsyncRuntime.

[SteveLauC]

Looks like this can be closed as well. 😀

[drmingdrmer]

Aye!