Remove source phase resolution time negative error types

Author: legendecas

test/language/module-code/source-phase-import/import-source-binding-name-2.js test/language/module-code/source-phase-import/import-source-binding-name-2_FIXTURE.js

@@ -3,14 +3,19 @@
 /*---
 description: >
   ImportBinding in ImportDeclaration may be 'source' and 'from'
+  This test is verified with `import-source.js` in the same directory that
+  this file does not raise SyntaxError. Note that a SyntaxError could also
+  be raised when the imported module does not have a source phase
+  representation (see sec-source-text-module-record-initialize-environment, 7.c.ii).
+
 esid: sec-modules
 info: |
   ImportDeclaration:
     import source ImportedBinding FromClause ;
 
 negative:
   phase: resolution
-  type: SyntaxError
+  type: Error
 features: [source-phase-imports]
 flags: [module]
 ---*/

test/language/module-code/source-phase-import/import-source-binding-name.js test/language/module-code/source-phase-import/import-source-binding-name_FIXTURE.js

@@ -3,14 +3,19 @@
 /*---
 description: >
   ImportBinding in ImportDeclaration may be 'source' and 'from'
+  This test is verified with `import-source.js` in the same directory that
+  this file does not raise SyntaxError. Note that a SyntaxError could also
+  be raised when the imported module does not have a source phase
+  representation (see sec-source-text-module-record-initialize-environment, 7.c.ii).
+
 esid: sec-modules
 info: |
   ImportDeclaration:
     import source ImportedBinding FromClause ;
 
 negative:
   phase: resolution
-  type: SyntaxError
+  type: Error
 features: [source-phase-imports]
 flags: [module]
 ---*/

test/language/module-code/source-phase-import/import-source-newlines.js test/language/module-code/source-phase-import/import-source-newlines_FIXTURE.js

@@ -3,6 +3,11 @@
 /*---
 description: >
   import source in ImportDeclaration may include line terminators
+  This test is verified with `import-source.js` in the same directory that
+  this file does not raise SyntaxError. Note that a SyntaxError could also
+  be raised when the imported module does not have a source phase
+  representation (see sec-source-text-module-record-initialize-environment, 7.c.ii).
+
 esid: sec-modules
 info: |
   ImportDeclaration:
@@ -15,7 +20,7 @@ info: |
   Source Text Module Record provides a GetModuleSource implementation that always returns an abrupt completion indicating that a source phase import is not available.
 negative:
   phase: resolution
-  type: SyntaxError
+  type: Error
 features: [source-phase-imports]
 flags: [module]
 ---*/

test/language/module-code/source-phase-import/import-source.js

@@ -0,0 +1,34 @@
+// Copyright (C) 2025 Bloomberg LP. All rights reserved.
+// This code is governed by the BSD license found in the LICENSE file.
+
+/*---
+description: >
+  Verify that ImportDeclaration can be correctly parsed.
+esid: sec-modules
+features: [source-phase-imports]
+flags: [async]
+includes: [asyncHelpers.js]
+---*/
+
+function assertImportSourceResolutionFailure(specifier) {
+  // Import the module and assert that the promise is rejected with a host
+  // defined error during the resolution phase.
+  // Note that this is not a `import.source`.
+  return import(specifier).then(
+    () => {
+      throw new Test262Error(`${specifier}: Promise should be rejected`);
+    },
+    error => {
+      print(error)
+      if (error instanceof SyntaxError) {
+        throw new Test262Error(`${specifier}: Promise should be rejected with a non-SyntaxError`);
+      }
+    }
+  );
+}
+
+asyncTest(async function () {
+  await assertImportSourceResolutionFailure('./import-source-binding-name_FIXTURE.js');
+  await assertImportSourceResolutionFailure('./import-source-binding-name-2_FIXTURE.js');
+  await assertImportSourceResolutionFailure('./import-source-newlines_FIXTURE.js');
+});