thejsj / socket.io-rethinkdb

A RethinkDB adapter for Socket.io
https://www.npmjs.com/package/socket.io-rethinkdb
20 stars 2 forks source link

Adapter tests are failing #3

Open sergenikov opened 7 years ago

sergenikov commented 7 years ago

Hi @thejsj,

I wanted to use the module, but saw that none of the tests are passing (both on Travis and my local machine). Since there aren't any pull requests that could've broken it, do you know if this adapter can be out of date with RethinkDB or socket.io-adapter itself? All tests fail because of the exceeded timeout and no messages are received. Only the db and the table are created, but they are always empty. If I use this adapter to store sockets in RethinkDB - nothing is stored there as well. Thanks in advance for any input.

Update: the culprit seems to be the remove_undefined function: right now it's recursive and runs infinitely. Did not have the time to debug further yet.