-
Notifications
You must be signed in to change notification settings - Fork 4
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
Return same instance instead of wrapping Iterator results when possible #5
Comments
NataliiaPrivezentseva
referenced
this issue
in NataliiaPrivezentseva/vavr
Nov 17, 2017
…ents(), shouldNotRemoveAllNonMatchedElementsFromNonNil(), and shouldRemoveNonExistingElements() methods, added annotation @SuppressWarnings(deprecation) (#1807)
danieldietrich
referenced
this issue
in vavr-io/vavr
Nov 23, 2017
#2170) * Eliminated method useIsEqualToInsteadOfIsSameAs() from AbstractSeqTest class and prepared code for elimination this method from related classes: ArrayTest, HashSetTest, ListTest, QueueTest, StreamTest and VectorTest * Revert "Eliminated method useIsEqualToInsteadOfIsSameAs() from AbstractSeqTest class and prepared code for elimination this method from related classes: ArrayTest, HashSetTest, ListTest, QueueTest, StreamTest and VectorTest" This reverts commit 0b053df in order to delete multiple unnecessary changes. * Eliminated useIsEqualToInsteadOfIsSameAs() from AbstractSeqTest class and updated test code to reflect this change * Used reject(Predicate) instead of removeAll(Predicate) in shouldRemoveExistingElements() and shouldRemoveNonExistingElements() methods * Used reject(Predicate) instead of removeAll(Predicate) in shouldNotRemoveAllNonMatchedElementsFromNonNil() method * Returned back method removeAll(Predicate) in shouldRemoveExistingElements(), shouldNotRemoveAllNonMatchedElementsFromNonNil(), and shouldRemoveNonExistingElements() methods, added annotation @SuppressWarnings(deprecation) (#1807)
NataliiaPrivezentseva
referenced
this issue
in NataliiaPrivezentseva/vavr
Dec 8, 2017
…sableTest class (#1807)
danieldietrich
referenced
this issue
in vavr-io/vavr
Jan 13, 2019
#2170) * Eliminated method useIsEqualToInsteadOfIsSameAs() from AbstractSeqTest class and prepared code for elimination this method from related classes: ArrayTest, HashSetTest, ListTest, QueueTest, StreamTest and VectorTest * Revert "Eliminated method useIsEqualToInsteadOfIsSameAs() from AbstractSeqTest class and prepared code for elimination this method from related classes: ArrayTest, HashSetTest, ListTest, QueueTest, StreamTest and VectorTest" This reverts commit 0b053df in order to delete multiple unnecessary changes. * Eliminated useIsEqualToInsteadOfIsSameAs() from AbstractSeqTest class and updated test code to reflect this change * Used reject(Predicate) instead of removeAll(Predicate) in shouldRemoveExistingElements() and shouldRemoveNonExistingElements() methods * Used reject(Predicate) instead of removeAll(Predicate) in shouldNotRemoveAllNonMatchedElementsFromNonNil() method * Returned back method removeAll(Predicate) in shouldRemoveExistingElements(), shouldNotRemoveAllNonMatchedElementsFromNonNil(), and shouldRemoveNonExistingElements() methods, added annotation @SuppressWarnings(deprecation) (#1807)
danieldietrich
referenced
this issue
in vavr-io/vavr
Jan 13, 2019
#2170) * Eliminated method useIsEqualToInsteadOfIsSameAs() from AbstractSeqTest class and prepared code for elimination this method from related classes: ArrayTest, HashSetTest, ListTest, QueueTest, StreamTest and VectorTest * Revert "Eliminated method useIsEqualToInsteadOfIsSameAs() from AbstractSeqTest class and prepared code for elimination this method from related classes: ArrayTest, HashSetTest, ListTest, QueueTest, StreamTest and VectorTest" This reverts commit 0b053df in order to delete multiple unnecessary changes. * Eliminated useIsEqualToInsteadOfIsSameAs() from AbstractSeqTest class and updated test code to reflect this change * Used reject(Predicate) instead of removeAll(Predicate) in shouldRemoveExistingElements() and shouldRemoveNonExistingElements() methods * Used reject(Predicate) instead of removeAll(Predicate) in shouldNotRemoveAllNonMatchedElementsFromNonNil() method * Returned back method removeAll(Predicate) in shouldRemoveExistingElements(), shouldNotRemoveAllNonMatchedElementsFromNonNil(), and shouldRemoveNonExistingElements() methods, added annotation @SuppressWarnings(deprecation) (#1807)
danieldietrich
referenced
this issue
in vavr-io/vavr
Jan 16, 2019
#2170) * Eliminated method useIsEqualToInsteadOfIsSameAs() from AbstractSeqTest class and prepared code for elimination this method from related classes: ArrayTest, HashSetTest, ListTest, QueueTest, StreamTest and VectorTest * Revert "Eliminated method useIsEqualToInsteadOfIsSameAs() from AbstractSeqTest class and prepared code for elimination this method from related classes: ArrayTest, HashSetTest, ListTest, QueueTest, StreamTest and VectorTest" This reverts commit 0b053df in order to delete multiple unnecessary changes. * Eliminated useIsEqualToInsteadOfIsSameAs() from AbstractSeqTest class and updated test code to reflect this change * Used reject(Predicate) instead of removeAll(Predicate) in shouldRemoveExistingElements() and shouldRemoveNonExistingElements() methods * Used reject(Predicate) instead of removeAll(Predicate) in shouldNotRemoveAllNonMatchedElementsFromNonNil() method * Returned back method removeAll(Predicate) in shouldRemoveExistingElements(), shouldNotRemoveAllNonMatchedElementsFromNonNil(), and shouldRemoveNonExistingElements() methods, added annotation @SuppressWarnings(deprecation) (#1807)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
useIsEqualToInsteadOfIsSameAs()
from unit tests and test forisSameAs()
Example: take(0)
This was only an example. We have many similar occurrences.
The text was updated successfully, but these errors were encountered: