diff options
| author | Vse Mozhet Byt <vsemozhetbyt@gmail.com> | 2017-07-01 02:29:09 +0300 |
|---|---|---|
| committer | Vse Mozhet Byt <vsemozhetbyt@gmail.com> | 2017-07-04 12:41:49 +0300 |
| commit | 2d2986ae72f2f5c63d95a94f05fa996d9f0609f1 (patch) | |
| tree | 7a92ba45eb1ec041a2f9e40b0e63ac5c0be409c0 /test/parallel/test-intl.js | |
| parent | cc1a47dc6b94b2feb4de9e46be9333d81d537783 (diff) | |
| download | android-node-v8-2d2986ae72f2f5c63d95a94f05fa996d9f0609f1.tar.gz android-node-v8-2d2986ae72f2f5c63d95a94f05fa996d9f0609f1.tar.bz2 android-node-v8-2d2986ae72f2f5c63d95a94f05fa996d9f0609f1.zip | |
test: simplify test skipping
* Make common.skip() exit.
Also add common.printSkipMessage() for partial skips.
* Don't make needless things before skip
PR-URL: https://github.com/nodejs/node/pull/14021
Fixes: https://github.com/nodejs/node/issues/14016
Reviewed-By: Refael Ackermann <refack@gmail.com>
Diffstat (limited to 'test/parallel/test-intl.js')
| -rw-r--r-- | test/parallel/test-intl.js | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/test/parallel/test-intl.js b/test/parallel/test-intl.js index 1e061d8f51..ff9569774d 100644 --- a/test/parallel/test-intl.js +++ b/test/parallel/test-intl.js @@ -52,7 +52,6 @@ if (!common.hasIntl) { `"Intl" object is NOT present but v8_enable_i18n_support is ${enablei18n}`; assert.strictEqual(enablei18n, 0, erMsg); common.skip('Intl tests because Intl object not present.'); - } else { const erMsg = `"Intl" object is present but v8_enable_i18n_support is ${ @@ -72,7 +71,7 @@ if (!common.hasIntl) { // If list is specified and doesn't contain 'en' then return. if (process.config.variables.icu_locales && !haveLocale('en')) { - common.skip( + common.printSkipMessage( 'detailed Intl tests because English is not listed as supported.'); // Smoke test. Does it format anything, or fail? console.log(`Date(0) formatted to: ${dtf.format(date0)}`); |