-
Notifications
You must be signed in to change notification settings - Fork 58
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
Make ReadResultSet iterable [API-1315] #1399
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1399 +/- ##
==========================================
- Coverage 93.30% 93.28% -0.02%
==========================================
Files 464 464
Lines 16377 16384 +7
Branches 1331 1332 +1
==========================================
+ Hits 15280 15284 +4
- Misses 799 801 +2
- Partials 298 299 +1
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
src/core/ReadResultSet.ts
Outdated
/** | ||
* Returns an iterator for elements in the list. | ||
* | ||
* @returns the iterator for elements in the list. | ||
*/ | ||
values(): Iterator<T>; |
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.
Why do we need this?
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.
You are right. We can remove it.
@@ -42,4 +42,14 @@ describe('LazyReadResultSetTest', function () { | |||
const set = new LazyReadResultSet(mockSerializationService, 4, [1, 2, 3, 4], [11, 12, 13, 14], 15); | |||
expect(set.get(4)).to.be.undefined; | |||
}); | |||
|
|||
it('object can be iteratable', function () { |
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.
it('object can be iteratable', function () { | |
it('should be iterable', function () { |
"it should be iterable" is fine I think.
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.
I agree you. I will correct it.
* Make ReadResultSet iterable [API-1315] * Changes made according to comments on PR.[API-1315]
Following changes done: