More coverage for tests on client

Hoverbear / old-raft-rs

[Incomplete] A Raft implementation in Rust

https://hoverbear.github.io/raft-rs/raft/

MIT License

266 stars 41 forks source link

Closed Hoverbear closed 9 years ago

Hoverbear commented 9 years ago

Tests all possible outcomes for a proposal against a standalone thread.

cc @danburkert r?

danburkert commented 9 years ago

Awesome! Things are shaping up, pretty great.

Hoverbear commented 9 years ago

@danburkert Should be cleaned up. :)

Hoverbear commented 9 years ago

// TODO:

Make sure there is a fail if the client is pointed to a leader outside of the cluster.
The // TODO can be replaced with assert!(cluster.insert(address).is_none())

Hoverbear commented 9 years ago

@danburkert I improved the tests and added a fail condition when the directed-to leader is not in the cluster.

danburkert commented 9 years ago

@homu r+

homu commented 9 years ago

:pushpin: Commit 1d90394 has been approved by danburkert

homu commented 9 years ago

:hourglass: Testing commit 1d90394 with merge f0dfa4d...

homu commented 9 years ago

:sunny: Test successful - status