share / sharedb / 869 / 3

96%

master: 97%

Build Type

push

travis-ci

Committed by Alec Gibson

Commit Message

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.

Run Details

1089 of 1309 branches covered (83.19%)

2293 of 2400 relevant lines covered (95.54%)

133.85 hits per line