Some tests had the same test descriptions

nodejs · Aug 11, 2024 · 70ce115 · 70ce115
1 parent 49a9ba4
commit 70ce115
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 4 deletions.
diff --git a/lib/internal/modules/cjs/loader.js b/lib/internal/modules/cjs/loader.js
@@ -1571,6 +1571,11 @@ function getMaybeCachedSource(mod, filename) {
   return content;
 }
 
+/**
+ * Built-in handler for `.cts` files.
+ * @param {Module} module The module to compile
+ * @param {string} filename The file path of the module
+ */
 function loadCTS(module, filename) {
   if (isUnderNodeModules(filename)) {
     throw new ERR_UNSUPPORTED_NODE_MODULES_TYPE_STRIPPING(filename);

diff --git a/test/es-module/test-typescript-eval.mjs b/test/es-module/test-typescript-eval.mjs
@@ -70,7 +70,7 @@ test('expect fail eval TypeScript CommonJS syntax with input-type module', async
   strictEqual(result.code, 1);
 });
 
-test('expect fail eval TypeScript CommonJS syntax with input-type module', async () => {
+test('expect fail eval TypeScript ESM syntax with input-type commonjs', async () => {
   const result = await spawnPromisified(process.execPath, [
     '--experimental-strip-types',
     '--input-type=commonjs',

diff --git a/test/es-module/test-typescript-module.mjs b/test/es-module/test-typescript-module.mjs
@@ -42,7 +42,6 @@ test('execute an .mts file importing a .cts file', async () => {
   const result = await spawnPromisified(process.execPath, [
     '--experimental-strip-types',
     '--no-warnings',
-    '--no-warnings',
     fixtures.path('typescript/mts/test-import-commonjs.mts'),
   ]);
 

diff --git a/test/es-module/test-typescript.mjs b/test/es-module/test-typescript.mjs
@@ -203,7 +203,7 @@ test('execute a TypeScript file with CommonJS syntax requiring .mts', async () =
   strictEqual(result.code, 1);
 });
 
-test('execute a TypeScript file with CommonJS syntax requiring .mts with require-module', async () => {
+test('execute a TypeScript file with CommonJS syntax requiring .mts using require-module', async () => {
   const result = await spawnPromisified(process.execPath, [
     '--experimental-strip-types',
     '--experimental-require-module',
@@ -215,7 +215,7 @@ test('execute a TypeScript file with CommonJS syntax requiring .mts with require
   strictEqual(result.code, 0);
 });
 
-test('execute a TypeScript file with CommonJS syntax requiring .mts with require-module', async () => {
+test('execute a TypeScript file with CommonJS syntax requiring .cts using commonjs', async () => {
   const result = await spawnPromisified(process.execPath, [
     '--experimental-strip-types',
     '--experimental-default-type=commonjs',