Skip to content
This repository was archived by the owner on Aug 17, 2020. It is now read-only.

Check for enclosing transaction on actual subscribe, not initial emission.#85

Merged
AlecKazakova merged 1 commit intomasterfrom
jw/check-on-subscribe
Feb 3, 2016
Merged

Check for enclosing transaction on actual subscribe, not initial emission.#85
AlecKazakova merged 1 commit intomasterfrom
jw/check-on-subscribe

Conversation

@JakeWharton
Copy link
Copy Markdown
Collaborator

This ensures that even with backpressure enabled to suppress initial emission an error is still thrown.

…sion.

This ensures that even with backpressure enabled to suppress initial emission an error is still thrown.
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test won't actually fail without the change until just supports backpressure (which is how startWith is implemented)

AlecKazakova pushed a commit that referenced this pull request Feb 3, 2016
Check for enclosing transaction on actual subscribe, not initial emission.
@AlecKazakova AlecKazakova merged commit 8bc1876 into master Feb 3, 2016
@AlecKazakova AlecKazakova deleted the jw/check-on-subscribe branch February 3, 2016 18:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants