esm: fix import assertion warning

aduh95 · danielleadams · commit 5b94e2bcdb7e · 2023-04-10T20:55:10.000-04:00
Refs: #46901 (comment) PR-URL: #46971 Reviewed-By: Debadree Chatterjee <debadree333@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Jacob Smith <jacob@frende.me> Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
diff --git a/lib/internal/modules/esm/assert.js b/lib/internal/modules/esm/assert.js
@@ -62,8 +62,8 @@ function validateAssertions(url, format,
   if (!alreadyWarned && ObjectKeys(importAssertions).length !== 0) {
     alreadyWarned = true;
     process.emitWarning(
-      'Import assertions are not a stable feature of the JavaScript language, ' +
-      'avoid relying on their current behavior and syntax as those might change ' +
+      'Import assertions are not a stable feature of the JavaScript language. ' +
+      'Avoid relying on their current behavior and syntax as those might change ' +
       'in a future version of Node.js.',
       'ExperimentalWarning',
     );
diff --git a/test/es-module/test-esm-import-assertion-errors.js b/test/es-module/test-esm-import-assertion-errors.js
@@ -7,8 +7,8 @@ const jsonModuleDataUrl = 'data:application/json,""';
 
 common.expectWarning(
   'ExperimentalWarning',
-  'Import assertions are not a stable feature of the JavaScript language, ' +
-  'avoid relying on their current behavior and syntax as those might change ' +
+  'Import assertions are not a stable feature of the JavaScript language. ' +
+  'Avoid relying on their current behavior and syntax as those might change ' +
   'in a future version of Node.js.'
 );
 
diff --git a/test/es-module/test-esm-import-assertion-errors.mjs b/test/es-module/test-esm-import-assertion-errors.mjs
@@ -6,8 +6,8 @@ const jsonModuleDataUrl = 'data:application/json,""';
 
 expectWarning(
   'ExperimentalWarning',
-  'Import assertions are not a stable feature of the JavaScript language, ' +
-  'avoid relying on their current behavior and syntax as those might change ' +
+  'Import assertions are not a stable feature of the JavaScript language. ' +
+  'Avoid relying on their current behavior and syntax as those might change ' +
   'in a future version of Node.js.'
 );
 
diff --git a/test/es-module/test-esm-import-assertion-validation.js b/test/es-module/test-esm-import-assertion-validation.js
@@ -8,8 +8,8 @@ const { validateAssertions } = require('internal/modules/esm/assert');
 
 common.expectWarning(
   'ExperimentalWarning',
-  'Import assertions are not a stable feature of the JavaScript language, ' +
-  'avoid relying on their current behavior and syntax as those might change ' +
+  'Import assertions are not a stable feature of the JavaScript language. ' +
+  'Avoid relying on their current behavior and syntax as those might change ' +
   'in a future version of Node.js.'
 );
 
diff --git a/test/es-module/test-esm-import-assertion-warning.mjs b/test/es-module/test-esm-import-assertion-warning.mjs
@@ -2,8 +2,8 @@ import { expectWarning } from '../common/index.mjs';
 
 expectWarning(
   'ExperimentalWarning',
-  'Import assertions are not a stable feature of the JavaScript language, ' +
-  'avoid relying on their current behavior and syntax as those might change ' +
+  'Import assertions are not a stable feature of the JavaScript language. ' +
+  'Avoid relying on their current behavior and syntax as those might change ' +
   'in a future version of Node.js.'
 );
 
