feathersjs-ecosystem / feathers-sync

Synchronize service events between Feathers application instances
MIT License
221 stars 41 forks source link

describe.only causing tests to be skipped #188

Closed jamesholcomb closed 1 year ago

jamesholcomb commented 1 year ago

https://github.com/feathersjs-ecosystem/feathers-sync/blame/622fbe28a13e1d2a29063762f83c377900e5e937/test/adapters/redis.test.js#L5

if I remove .only, then the test 'sends sync-out for custom events' fails:

TypeError: Cannot read properties of undefined (reading 'toJSON')

Updating the dev dep to @feathersjs/feathers@5.0.3 had no effect

P.S. This was after digging into why I could not get sync to work with an ioredis client (my app uses sentinels)

related

186

182

daffl commented 1 year ago

Good catch, thank you. The fix from #187 has been released as v3.0.3 and all tests are running now.