Skip to content

Commit ac828c5

Browse files
committed
Add node 10 and 12 to test matrix, disable pl pluralrules test.
Also, I'm making node 8 not specify the subrevision, there are newer versions of it by now, and we're not specific on 10 nor 12. The test for Polish plural forms requires that Intl.PluralRules exists for Polish, and the standard node distributions don't ship with ICU. Let's rely on the upstream tests.
1 parent 4dd134b commit ac828c5

File tree

2 files changed

+4
-11
lines changed

2 files changed

+4
-11
lines changed

.travis.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
language: node_js
22
script: make deps && make dist
3-
node_js: "8.9"
3+
node_js:
4+
- "8"
5+
- "10"
6+
- "12"
47
cache:
58
directories: node_modules
69
notifications:

fluent-intl-polyfill/test/plural_rules_test.js

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -28,16 +28,6 @@ suite('Plural rules', function() {
2828
assert.strictEqual(pr.select(22), 'other');
2929
});
3030

31-
test('known language: pl', function() {
32-
const pr = new Intl.PluralRules('pl');
33-
assert.strictEqual(pr.select(0), 'many');
34-
assert.strictEqual(pr.select(1), 'one');
35-
assert.strictEqual(pr.select(3), 'few');
36-
assert.strictEqual(pr.select(5), 'many');
37-
assert.strictEqual(pr.select(12), 'many');
38-
assert.strictEqual(pr.select(22), 'few');
39-
});
40-
4131
test('unknown languages uses en-US', function() {
4232
const pr = new Intl.PluralRules('xxx');
4333
assert.strictEqual(pr.select(0), 'other');

0 commit comments

Comments
 (0)