Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

2.x: coverage and fixes 9/03-2 #4469

Merged
merged 1 commit into from
Sep 3, 2016
Merged

Conversation

akarnokd
Copy link
Member

@akarnokd akarnokd commented Sep 3, 2016

  • enable fusion with Observable.map
  • remove impossible branches in some classes
  • add more tests to cover tool classes
  • fix Observable.range() entering regular emission if fusion-drain is interrupted
  • fix FullArbiter and ObservableFullArbiter not cancelling/disposing the incoming Subscription/Disposable in some cases

@codecov-io
Copy link

Current coverage is 78.18% (diff: 94.44%)

Merging #4469 into 2.x will increase coverage by 1.17%

@@                2.x      #4469   diff @@
==========================================
  Files           493        493          
  Lines         33684      33629    -55   
  Methods           0          0          
  Messages          0          0          
  Branches       5315       5290    -25   
==========================================
+ Hits          25939      26293   +354   
+ Misses         5690       5370   -320   
+ Partials       2055       1966    -89   

Powered by Codecov. Last update f59ce00...52dc276

@JakeWharton
Copy link
Contributor

👍

@akarnokd akarnokd merged commit 83a3332 into ReactiveX:2.x Sep 3, 2016
@akarnokd akarnokd deleted the Coverage903_2 branch September 3, 2016 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants