I tried to understand it but could not figure out why this might have fixed the issue. I also tried adding fireOnFirstErrback=True and fireOnFirstErrback=True, consumeErrors=True to DeferredList, and also adding errbacks to everything, to no avail—nothing was printed and the issue persisted.
Unless you have a reason to believe using inlineCallbacks here is somehow bad or hides a/the real issue, this commit seems to have fixed it. The change also makes the test more readable (as inlineCallbacks always does).
I tried to understand it but could not figure out why this might have fixed the issue. I also tried adding
fireOnFirstErrback=TrueandfireOnFirstErrback=True, consumeErrors=TruetoDeferredList, and also adding errbacks to everything, to no avail—nothing was printed and the issue persisted.Unless you have a reason to believe using
inlineCallbackshere is somehow bad or hides a/the real issue, this commit seems to have fixed it. The change also makes the test more readable (asinlineCallbacksalways does).