2528
90%
master: %

Ran 21 Nov 2016 10:13PM UTC

Jobs 2

Files 110

Run time 52s

Badge

Embed ▾

pending completion

Build # 2528

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

[PeerList][Part 11] Differentiating between Available/Unavailable Peers in RoundRobin (#449)

Summary: In order to have a more powerful PeerList in the round robin,
we need to be able to distinguish between a Peer that cannot accept
requests and one that can. This PR adds a distinction in the RoundRobin
PeerList that will only return Available Peers from the ChoosePeer
method.

In the process of doing this, I realized it would be a bad idea to trust
the Peer we pass in the NotifyStatusChanged function, so I changed it
from a Peer to a PeerIdentifier so that the lists need to look up their
own object so we don't get an issue of duplicate peers getting passed
around with different state.

Test Plan: I've upgraded the PeerListAction tests to have a
NotifyStatusChangeAction which will update a Peer's Status and run the
NotifyStatusChange function on the PeerList with the provided Peer.
This required weaving a list of available MockPeers in the Apply
function.
Additionally, the gomock MockPeer was not very flexible with changing
the returned result from an Expectation, so I ended up creating a thin
mock of Peer instead.

Run Details

5311 of 5927 relevant lines covered (89.61%)

1.9 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
3	2528.3 (TYPE=test)	21 Nov 2016 10:13PM UTC	0	89.56	Travis Job 2528.3
4	2528.4 (TYPE=test)	21 Nov 2016 10:14PM UTC	0	89.61	Travis Job 2528.4

Source Files on build 2528

Detailed source file information is not available for this build.

yarpc / yarpc-go / 2528
90%
master: %

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2528

yarpc / yarpc-go / 2528 90% master: %

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2528

yarpc / yarpc-go / 2528
90%
master: %

README BADGES
x