test: remove common.hasSmallICU

common.hasSmallICU is used in only one test and is a one-liner. Move
into the test where it is used to chip away at the `common` monolith.

PR-URL: nodejs#22937
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

test/common/README.md

@@ -158,11 +158,6 @@ Indicates `hasCrypto` and `crypto` with fips.
 
 Indicates if [internationalization] is supported.
 
-### hasSmallICU
-* [<boolean>]
-
-Indicates `hasIntl` and `small-icu` are supported.
-
 ### hasIPv6
 * [<boolean>]
 

test/common/index.js

@@ -32,8 +32,7 @@ const { fixturesDir } = require('./fixtures');
 const tmpdir = require('./tmpdir');
 const {
   bits,
-  hasIntl,
-  hasSmallICU
+  hasIntl
 } = process.binding('config');
 
 const noop = () => {};
@@ -715,7 +714,6 @@ module.exports = {
   hasIntl,
   hasCrypto,
   hasIPv6,
-  hasSmallICU,
   hasMultiLocalhost,
   isAIX,
   isAlive,

test/parallel/test-icu-data-dir.js

@@ -1,7 +1,9 @@
 'use strict';
 const common = require('../common');
 const os = require('os');
-if (!(common.hasIntl && common.hasSmallICU))
+
+const { hasSmallICU } = process.binding('config');
+if (!(common.hasIntl && hasSmallICU))
   common.skip('missing Intl');
 
 const assert = require('assert');