cybertec-postgresql / vip-manager / 40355181934

92%

Build Type

push

github

Committed by web-flow

Commit Message

[!] name the fields of the etcd leader checker (#447)

Embedding both the configuration and the etcd client put their methods and
fields into one namespace: elc.Get() and elc.Close() silently belonged to the
client, elc.Logger to the configuration, and any name the two come to share in
a future release of either would break the build far away from the cause. The
tests reached through the same promotion.

No behaviour change. The consul and the patroni checker embed their client the
same way and can follow if this direction is welcome.

Coverage Stats

11 of 12 new or added lines in 1 file covered. (91.67%)

2 existing lines in 2 files now uncovered.

858 of 940 relevant lines covered (91.28%)

1.35 hits per line