firebase / angularfire / 831

93%

master: 94%

push

travis-ci

Fixes #617 - $loaded() triggered too soon

Because of promisifying the $$added method, the child_added events were actually being processed internally after the loaded trigger. This occurred because $q.when() actually uses nextTick() internally as well, meaning the child_added events occurred immediately, but then $$process was not called until the second tick occurred; a difficult use case to catch.

The $loaded() event, meanwhile, had already triggered on the first tick. Solution was to promisify all the events and then remove the batch() wrapper since they trigger on nextTick() anyway now. So all events are back to firing in order on the next tick.

18 of 18 new or added lines in 2 files covered. (100.0%)

622 of 671 relevant lines covered (92.7%)

119.5 hits per line