We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
With the latest 0.20.0-RC1 a combination of takeWhile() and toList() on an Observable fails, sometimes.
Here's a unit test that fails:
@Test public void testTakeWhile() { int[] nums = {1, 2, 3}; final AtomicInteger count = new AtomicInteger(); for(final int n: nums) { Observable .from(Boolean.TRUE, Boolean.FALSE) .takeWhile(new Func1<Boolean, Boolean>() { @Override public Boolean call(Boolean value) { return value; } }) .toList() .doOnNext(new Action1<List<Boolean>>() { @Override public void call(List<Boolean> booleans) { count.incrementAndGet(); } }) .subscribe(); } assertEquals(nums.length, count.get()); }
Here's the same unit test that passes with slight modification:
@Test public void testTakeWhile() { int[] nums = {1, 2, 3}; final AtomicInteger count = new AtomicInteger(); for(final int n: nums) { Observable .from(Boolean.TRUE, Boolean.FALSE, Boolean.FALSE) .takeWhile(new Func1<Boolean, Boolean>() { @Override public Boolean call(Boolean value) { return value; } }) .toList() .doOnNext(new Action1<List<Boolean>>() { @Override public void call(List<Boolean> booleans) { count.incrementAndGet(); } }) .subscribe(); } assertEquals(nums.length, count.get()); }
The text was updated successfully, but these errors were encountered:
Fix issue ReactiveX#1451
53b4659
Fix related classes for issue ReactiveX#1451
b90b6a3
6bbd921
Merge pull request #1454 from zsxwing/issue1451
877ee89
Fix issue #1451
This should be fixed in 0.20.0-RC3
Sorry, something went wrong.
No branches or pull requests
With the latest 0.20.0-RC1 a combination of takeWhile() and toList() on an Observable fails, sometimes.
Here's a unit test that fails:
Here's the same unit test that passes with slight modification:
The text was updated successfully, but these errors were encountered: