Skip to content
This repository has been archived by the owner on Apr 7, 2021. It is now read-only.

Fixes #57 #61

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

Fixes #57 #61

wants to merge 1 commit into from

Conversation

Iiridayn
Copy link

@Iiridayn Iiridayn commented Jan 6, 2017

cursor.each is called lazily, which means we have a race condition between destroy and _onChange. To resolve that, test if destroy has already run, then give up early processing the change list.

@coveralls
Copy link

coveralls commented Jan 6, 2017

Coverage Status

Coverage increased (+2.1%) to 90.05% when pulling 530fe33 on Iiridayn:fix-#57 into 3f15ade on deepstreamIO:master.

@Iiridayn
Copy link
Author

Iiridayn commented Jan 6, 2017

All tests pass on my local machine - I suspect some degree of non-determinism in the tests therefore.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants