Skip to content

Conversation

@jlami
Copy link
Contributor

@jlami jlami commented Oct 26, 2021

This should fail for indexeddb on the following test:
test.bulk_docs.js-local
2839 implement revs_limit

@jlami
Copy link
Contributor Author

jlami commented Oct 27, 2021

Hmmm, I guess this only fails on Windows Chrome.
After checking Firefox I see that it has no problems on Windows.

I'm not sure if reporting this as a bug on Chrome is worth it, but here is the bug tracker: https://bugs.chromium.org/p/chromium/issues/detail?id=1263612

@jlami
Copy link
Contributor Author

jlami commented Oct 28, 2021

The Chromium issue got tagged Wontfix, so we might have to decide if we want to do something about this on our side. I'll make a separate issue for that.

@jlami jlami closed this Oct 28, 2021
Linux does not have a problem with 1000 revisions, let's see if 4900 will work
@jlami
Copy link
Contributor Author

jlami commented Oct 28, 2021

Test if higher revision count will trigger this.

@jlami jlami reopened this Oct 28, 2021
@jlami
Copy link
Contributor Author

jlami commented Oct 28, 2021

The pouchdb-server failure is unexpected, but the change to use 4900 as revs_limit reproduces the RangeError now on both Firefox and Chrome

@janl
Copy link
Member

janl commented Jan 12, 2026

also todo for us: look into running tests on windows

@alxndrsn
Copy link
Contributor

I don't understand the rationale for the suggested test change.

Should there be an additional test which deliberately generates RangeError cross-platform and ensures that it's handled as expected?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants