Add Unit Tests for Network & Storage. And minor improvements

[dharanad]

Partially closes #3

Change List

• Added unit tests for Network Layer & Storage, because we all need a little more assurance in life, especially our code.
• Gave our codebase a spa day and a healthy diet.
• Fix typos

[dharanad]

@vaibhawvipul I would also like to suggest few improvement in this PR. If that sounds fine ?
How about dropping the object once we close the connection ?

raft-rs/src/network.rs

Line 121 in 2b1c74b

async fn close(&self) -> Result<(), Box<dyn Error + Send + Sync>> {

And when we delete the file ?

raft-rs/src/storage.rs

Line 102 in 2b1c74b

async fn delete(&self) -> Result<(), Box<dyn Error + Send + Sync>> {

[vaibhawvipul]

@vaibhawvipul I would also like to suggest few improvement in this PR. If that sounds fine ? How about dropping the object once we close the connection ?

raft-rs/src/network.rs

Line 121 in 2b1c74b

async fn close(&self) -> Result<(), Box<dyn Error + Send + Sync>> {

And when we delete the file ?

raft-rs/src/storage.rs

Line 102 in 2b1c74b

async fn delete(&self) -> Result<(), Box<dyn Error + Send + Sync>> {

Connection can be closed because of various reasons. Let's say we want to change the port.

I still would want to keep the logs because later on I want the server to do catch-up, instead of expecting complete logs from other servers, when it is restarted.

Does it make sense?

[vaibhawvipul]

LGTM