Skip to content
This repository has been archived by the owner on Dec 2, 2024. It is now read-only.

Edge and IE occasionally fail snapshot test(s) #158

Closed
vweevers opened this issue Dec 27, 2018 · 4 comments
Closed

Edge and IE occasionally fail snapshot test(s) #158

vweevers opened this issue Dec 27, 2018 · 4 comments
Labels
bug Something isn't working wontfix This will not be worked on

Comments

@vweevers
Copy link
Member

vweevers commented Dec 27, 2018

Edge failed a test in a Travis/Sauce Labs job. See #157 (comment).

@vweevers vweevers added the bug Something isn't working label Dec 27, 2018
@vweevers
Copy link
Member Author

I can replicate it locally, also in IE, repeating the snapshot tests 1000 times. Side note: they sometimes just stall after opening a few hundred db's, but that's another issue.

I cannot replicate it in Chrome or FF. This suggests there is a bug in IE/Edge's IndexedDB implementation. I tried to find a relevant test in the W3C IndexedDB test suite (which lists many failures for Edge) but came up short.

@vweevers
Copy link
Member Author

vweevers commented Dec 27, 2018

Side note: they sometimes just stall after opening a few hundred db's, but that's another issue.

Correction: they (or at least IE) error on this line - because that test asserts that an iterator can read a key that was deleted after the iterator/snapshot/transaction was created - and IE apparently fails that test (too).

I'm inclined to tag this as wontfix and move on.

Overall, IE failed 4 of 1690 tests. Edge failed 5 of 3000 tests. Chrome, FF and Opera never fail (I ran 6000+ tests on each).

@vweevers vweevers changed the title Look into potential race issue Edge and IE occasionally fail snapshot test(s) Dec 27, 2018
@vweevers vweevers added the wontfix This will not be worked on label Dec 27, 2018
@vweevers
Copy link
Member Author

@ralphtheninja FYI ^^. If you see a snapshot test failing in future builds, restart them. If and when it becomes a nuisance we can consider skipping these tests in IE/Edge.

@vweevers
Copy link
Member Author

Closing due to #86

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant