Random test failures in test/parallel/test-dgram-send-callback-recursive.js

[evanlucas]

Introduced in PR #1313.

Seen on OS X.

Sometimes it will fail repeatedly and other times it will fail 1 out of 5.

/cc @yosuke-furukawa

[Fishrock123]

x-post #1313 (comment)

https://jenkins-iojs.nodesource.com/view/iojs/job/iojs+any-pr+multi/657/nodes=osx1010/tapTestReport/test.tap-153/

https://jenkins-iojs.nodesource.com/view/iojs/job/iojs+any-pr+multi/657/nodes=win2012r2/tapTestReport/test.tap-153/

https://jenkins-iojs.nodesource.com/view/iojs/job/iojs+any-pr+multi/657/nodes=win2008r2/tapTestReport/test.tap-153/

not ok 153 - test-dgram-send-callback-recursive.js
# c:\workspace\iojs+any-pr+multi\nodes\win2012r2\test\parallel\test-dgram-send-callback-recursive.js:14
# throw new Error('infinite loop detected');
# ^
# Error: infinite loop detected
# at SendWrap.onsend [as callback] (c:\workspace\iojs+any-pr+multi\nodes\win2012r2\test\parallel\test-dgram-send-callback-recursive.js:14:11)
# at Immediate._onImmediate (dgram.js:342:10)
# at processImmediate [as _immediateCallback] (timers.js:368:17)

[yosuke-furukawa]

OK, I will investigate it.

[yosuke-furukawa]

OK I found the reason.
I just mistook the condition.
When io.js CPU load is heavy, the loop counter goes 10 higher like 11, but This counter limit is 10.
I should fix the condition. I will make pull request ASAP .

Thank you for notification.

[Fishrock123]

Should be fixed in 4e2f999

CI: https://jenkins-iojs.nodesource.com/view/iojs/job/iojs+any-pr+multi/676/

[evanlucas]

Looks good to me :]