Skip to content
This repository was archived by the owner on Jan 10, 2025. It is now read-only.

afterObserve.invoke() only if latch is satisfied #1068

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dgochev
Copy link

@dgochev dgochev commented Dec 8, 2022

getOrAwaitValue in LiveDataTestUtil should invoke afterObserve only if the latch is satisfied (i.e. data was changed)

`getOrAwaitValue` in `LiveDataTestUtil` should invoke `afterObserve` only if the latch is satisfied (i.e. data was changed)
@dgochev
Copy link
Author

dgochev commented Dec 8, 2022

Seems like the latch isn't entirely serving its purpose, and afterObserve isn't actually being done after observing. Maybe the name just isn't clear -- is the purpose of afterObserve to do something right after the .observeForever?

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.

1 participant