search

eemeli / intl-pluralrules

Polyfill for Intl.PluralRules
https://www.npmjs.com/package/intl-pluralrules
ISC License
41 stars 7 forks source link

Add tests & fixes for test262 compliance #27

Closed eemeli closed 2 years ago

eemeli commented 2 years ago

Fixes #8

It seemed like a decent time to take another shot at this, and using the harness was decently easy this time. Also fixing the issues that the tests identified was pretty straightforward.

Before merging this, I should set up some basic browser tests, as even IE 11 should still be supported.

As side effects, jest gets updated to 29, and CI tests no longer target Node.js 12.

The cross-realm tests are skipped for now, as I don't think they can be fixed for a polyfill? Happy to be proven wrong, of course.

eemeli commented 2 years ago

The browser tests will actually need to get added after this, because they'll also depend on applying changes to the tests.