-
-
Notifications
You must be signed in to change notification settings - Fork 181
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
Testing rework: .ToObservableChangeSet()
#760
Testing rework: .ToObservableChangeSet()
#760
Conversation
…increasting code coverage from 69% to 100% across the internal implementation classes. Identified existing defects within the `.ToObservableChangeSet()` operators, mostly related to error and completion propagation, and implemented testing for this behavior, with the tests being marked as "Skip" until the defects are fixed. This also includes the defect referred in reactivemarbles#635. Added an `IsCompleted` flag to `ChangeSetAggregator<T>`, to allow for testing of completion propagation.
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #760 +/- ##
==========================================
+ Coverage 64.74% 65.55% +0.80%
==========================================
Files 226 227 +1
Lines 11459 11095 -364
Branches 2334 2294 -40
==========================================
- Hits 7419 7273 -146
+ Misses 3083 2884 -199
+ Partials 957 938 -19 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
|
||
[Fact(Skip = "Outstanding bug, completion is not forwarded")] | ||
[System.Diagnostics.CodeAnalysis.SuppressMessage("Usage", "xUnit1004:Test methods should not be skipped", Justification = "Bug to be fixed")] | ||
public void RemovalsArePending_CompletionWaitsForRemovals() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Rebuilt test fixtures for both
.ToObservableChangeSet()
operators, increasting code coverage from 69% to 100% across the internal implementation classes.Identified existing defects within the
.ToObservableChangeSet()
operators, mostly related to error and completion propagation, and implemented testing for this behavior, with the tests being marked as "Skip" until the defects are fixed. This also includes the defect referred in #635.Added an
IsCompleted
flag toChangeSetAggregator<T>
, to allow for testing of completion propagation.