Skip to content

test: Use `common.hasIntl` in tests related to ICU

Rodrigo Muino Tomonari requested to merge github/fork/watilde/feature/hasIntl into master

We should use common.hasIntl in tests for test cases which are related to ICU. This way we can easily find the test cases that are Intl dependent. Plus, it will be able to make the tests a little faster if we check hasIntl first.

Refs: https://github.com/nodejs/node/pull/10707#discussion_r96273895

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)
  • test
  • intl

Merge request reports