Issues with latest version of jest with the jsdom environment about fakeindexeddb HOT 3 CLOSED

Hopefully this is fixed in v3.1.4. Please reopen if not.

from fakeindexeddb.

Switching from setImmediate to process.nexTick does make a few of the tests fail. Not sure why, I haven't had a chance to look into it further. But IIRC it was difficult to get all of the asynchronous stuff working correctly, and I think even now there are some rare edge cases where it doesn't work quite right. Very finicky stuff. So I'm a little scared to change it :)

from fakeindexeddb.

I tried using queueMicrotask but got a few timeout in tests (I guess the same ones you got with process.nextTick). I think the difference is that setImmediate is a macrotask, but process.nextTick and queueMicrotask queue microtasks, which behave differently.

I also tried using setTimeout and with this, all tests are passing. The downside of setTimeout is that in browsers it may be delayed more, especially when nested. But is that a problem for this package, as it will run in a test environment? That would remove the need of the polyfill for setImmediate, reducing the complexity.

But anyway this doesn't solve the problem with jest's fake timers, because they also mock setTimeout obviously. Instead of calling setImmediate or setTimeout, we'd need to call the real functions, probably using some magic with jest functions like outlined above.

For now I'm not blocked anymore in our project, as I removed the fake timers from the tests that were using this library.

from fakeindexeddb.