diff --git a/lib/internal/cli_table.js b/lib/internal/cli_table.js index f6c711ece8e0ad..71d322ee9eeed1 100644 --- a/lib/internal/cli_table.js +++ b/lib/internal/cli_table.js @@ -4,6 +4,12 @@ const { Buffer } = require('buffer'); const { removeColors } = require('internal/util'); const HasOwnProperty = Function.call.bind(Object.prototype.hasOwnProperty); +// The use of Unicode characters below is the only non-comment use of non-ASCII +// Unicode characters in Node.js built-in modules. If they are ever removed or +// rewritten with \u escapes, then a test will need to be (re-)added to Node.js +// core to verify that Unicode characters work in built-ins. Otherwise, +// consumers using Unicode in _third_party_main.js will run into problems. +// Refs: https://github.com/nodejs/node/issues/10673 const tableChars = { /* eslint-disable node-core/non-ascii-character */ middleMiddle: '─', diff --git a/lib/internal/test/unicode.js b/lib/internal/test/unicode.js deleted file mode 100644 index 451c3c4737898a..00000000000000 --- a/lib/internal/test/unicode.js +++ /dev/null @@ -1,8 +0,0 @@ -'use strict'; - -// This module exists entirely for regression testing purposes. -// See `test/parallel/test-internal-unicode.js`. - -/* eslint-disable node-core/non-ascii-character */ -module.exports = '✓'; -/* eslint-enable node-core/non-ascii-character */ diff --git a/node.gyp b/node.gyp index fba7d4be81e264..e3fc7db47d6e7f 100644 --- a/node.gyp +++ b/node.gyp @@ -161,7 +161,6 @@ 'lib/internal/socket_list.js', 'lib/internal/test/binding.js', 'lib/internal/test/heap.js', - 'lib/internal/test/unicode.js', 'lib/internal/timers.js', 'lib/internal/tls.js', 'lib/internal/trace_events_async_hooks.js', diff --git a/test/parallel/test-internal-unicode.js b/test/parallel/test-internal-unicode.js deleted file mode 100644 index 0a36309c09f651..00000000000000 --- a/test/parallel/test-internal-unicode.js +++ /dev/null @@ -1,12 +0,0 @@ -'use strict'; -require('../common'); - -// Flags: --expose-internals -// -// This test ensures that UTF-8 characters can be used in core JavaScript -// libraries built into Node's binary. - -const assert = require('assert'); -const character = require('internal/test/unicode'); - -assert.strictEqual(character, '✓');