Context struct for raft test helper functions

moby / swarmkit

A toolkit for orchestrating distributed systems at any scale. It includes primitives for node discovery, raft-based consensus, task scheduling and more.

Apache License 2.0

3.34k stars 611 forks source link

Context struct for raft test helper functions #377

Open aaronlehmann opened 8 years ago

aaronlehmann commented 8 years ago

The argument list of several raft utility functions has grown over time. They used to only take a nodes map, but now they also take a clockSource and securityConfig. It would probably clean things up to have some kind of raft cluster test context that gets passed to these functions or used as a method receiver for them.

cc @abronan

aluzzardi commented 8 years ago

Bump.

Should we leave this as is (no milestone)?

aaronlehmann commented 8 years ago

Yes, no urgency. It's a test cleanup issue.