share / sharedb / 874 / 2

96%

master: 97%

push

travis-ci

Improve test coverage for `SnapshotRequest.onConnectionStateChanged`

This change improves `SnapshotRequest` test coverage to 100%.

Our current test for checking that a dropped connection works didn't
actually fail properly when removing `this.sent = false` from
`SnapshotRequest._onConnectionStateChanged`, so an improved test has
been written that use middleware to wait until the request has
definitely been sent, before dropping the connection and reconnecting.

It also adds a previously uncovered test case for checking that the
connection cannot send the same request multiple times, by using a
similar setup to the rewritten dropped connection case.

This change also adds some documentation recording why we might want to
reset `this.sent = false`, because just looking at the code it's
unclear why we would ever want to do that.

1077 of 1295 branches covered (83.17%)

2275 of 2382 relevant lines covered (95.51%)

134.62 hits per line