Remove Lock class dependency from Election class

[vbmade2000]

There is an instantiation of Lock class inside Election class. This makes Election class dependent on Lock class. You can see instantiation at https://github.com/metaparticle-io/sync/blob/master/python/metaparticle_sync/election.py#L8:21.

IMHO, instance of Lock class should be supplied from outside. It removes dependency of Election class on Lock class. Also I believe it makes Election class more unit test friendly.

Let me know your thoughts. If you agree, I can send a PR.

[vbmade2000]

@brendandburns Any thoughts on this issue ?

[brendandburns]

Sure, that works for me, happy to see a PR.

Many thanks!

[brendandburns]

(and sorry for the delay)

[vbmade2000]

@brendandburns I am confused about lock_callback and lock_lost_callback constructor arguments as their default value are from Lock class itself. If We supply Lock instance from outside of Election class, I am not sure how to handle these two arguments.

self.lock = Lock(name, lock_callback=self._lock, lock_lost_callback=self._lost_lock)