search

Atmosphere / wasync

WebSockets with fallback transports client library for Node.js, Android and Java
http://async-io.org
161 stars 47 forks source link

Allow for sub-second reconnect timeouts. #119

Closed thabach closed 9 years ago

thabach commented 9 years ago

Allow for sub-second reconnect timeouts. Seconds are to coarse grained a measure.

jfarcand commented 9 years ago

@thabach I have 6 tests failing locally? Where they failling before your commit? 

Failed tests:
  closeWriteTest(org.atmosphere.tests.LongPollingTest): expected:<OPENPINGCLOSEERROR> but was:<OPENPINGERROR>
  timeoutTest(org.atmosphere.tests.LongPollingTest): expected:<OPENCLOSEERROR> but was:<OPENERROR>
  reconnectFireTest(org.atmosphere.tests.LongPollingTest): expected:<OPENPINGCLOSEERROR> but was:<OPENPINGERROR>
  closeWriteTest(org.atmosphere.tests.StreamingTest): expected:<OPENPINGCLOSEERROR> but was:<OPENPINGERROR>
  timeoutTest(org.atmosphere.tests.StreamingTest): expected:<OPENCLOSEERROR> but was:<OPENERROR>
  reconnectFireTest(org.atmosphere.tests.StreamingTest): expected:<OPENPINGCLOSEERROR> but was:<OPENPINGERROR>

Tests run: 110, Failures: 6, Errors: 0, Skipped: 0
jfarcand commented 9 years ago

OK the regression is from #118 and fixed.