wikimedia / parsoid / 1764

85%

push

travis-ci

<a href="https://github.com/wikimedia/parsoid/commit/8c48d66c5">WTS: Use `yield` instead of directly returning Promises

Direct return of Promises from async functions, although convenient,
can obscure the callstack if an exception is thrown (since the caller
is no longer on the callstack when the returned Promise is resolved),
and may confuse readers which have to keep track of whether the return
value is of Promise type or not in order to figure out control flow
in the case of an exception.

Prefer to explicitly `yield` rather than directly returning a Promise,
at least in the WTS code.

Change-Id: I</a><a class="double-link" href="https://github.com/wikimedia/parsoid/commit/88d5b628d">88d5b628d</a>

8925 of 10960 branches covered (81.43%)

14361 of 16667 relevant lines covered (86.16%)

103346.06 hits per line