2
97%
master: 97%

Ran 14 Jun 2017 09:09PM UTC

Files 345

Run time 15s

Badge

Committed 14 Jun 2017 08:53PM UTC coverage: 97.734% (-0.002%) from 97.736%

Job # NODE_VER=6 FULL_VALIDATE=true alias grunt=./node_modules/grunt-cli/bin/grunt danger=./node_modules/danger/distribution/danger

Build Type

push

travis-ci

Committed by

benlesh

Commit Message

fix(groupBy): group duration notifiers will now properly unsubscribe and clean up (#2662)

* test(groupBy): Test that GroupBy-durationSelectors are disposed of

Duration selectors where not disposed when the GroupDurationSubscriber's completed

* fix(groupBy): unsubscribe GroupDurationSubscriber after completing the group

The Groups are disposed by the GroupDurationSelector, however the
GroupDurationSubscriber can be subscribed to a different observable
than the group itself. To prevent any unwanted subscriptions to
accumulate over time we need to explicitly unsubscribe after the
first event in GroupDurationSubscriber closes the group.

Fixes #2660

* fix(groupBy): ensures durationSelector subscriptions are cleaned

The subscriptions to the durationSelector would pile up in the
internal subscription list of the GroupBySubscriber. By removing
the GroupDurationSubscriber explicitly from the GroupBySubscriber
we prevent potential OOM exceptions.

Fixes #2661

* refactor(groupBy): Refactor GroupDurationSubscriber to pass-through to underlying group Subject

* refactor(groupBy): simplify durationSelector test

Coverage Stats

6642 of 6796 relevant lines covered (97.73%)

684.55 hits per line

Source Files on job 4592.2 (NODE_VER=6 FULL_VALIDATE=true alias grunt=./node_modules/grunt-cli/bin/grunt danger=./node_modules/danger/distribution/danger)

ReactiveX / rxjs / 4592 / 2
97%
master: 97%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4592.2 (NODE_VER=6 FULL_VALIDATE=true alias grunt=./node_modules/grunt-cli/bin/grunt danger=./node_modules/danger/distribution/danger)

ReactiveX / rxjs / 4592 / 2 97% master: 97%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4592.2 (NODE_VER=6 FULL_VALIDATE=true alias grunt=./node_modules/grunt-cli/bin/grunt danger=./node_modules/danger/distribution/danger)

ReactiveX / rxjs / 4592 / 2
97%
master: 97%

README BADGES
x