test: increase coverage for ModuleMap

Add test for ModuleMap set with ModuleJob but bad url. PR-URL: #16045 Reviewed-By: Stephen Belanger <[email protected]> Reviewed-By: Bradley Farias <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Joyee Cheung <[email protected]>

Author: robtpaton

test/es-module/test-esm-loader-modulemap.js

@@ -0,0 +1,56 @@
+'use strict';
+// Flags: --expose-internals
+
+// This test ensures that the type checking of ModuleMap throws
+// errors appropriately
+
+constcommon=require('../common');
+
+const{URL}=require('url');
+constLoader=require('internal/loader/Loader');
+constModuleMap=require('internal/loader/ModuleMap');
+constModuleJob=require('internal/loader/ModuleJob');
+const{ createDynamicModule }=require('internal/loader/ModuleWrap');
+
+conststubModuleUrl=newURL('file://tmp/test');
+conststubModule=createDynamicModule(['default'],stubModuleUrl);
+constloader=newLoader();
+constmoduleMap=newModuleMap();
+constmoduleJob=newModuleJob(loader,stubModule.module,
+()=>newPromise(()=>{}));
+
+common.expectsError(
+()=>moduleMap.get(1),
+{
+code: 'ERR_INVALID_ARG_TYPE',
+type: TypeError,
+message: 'The "url" argument must be of type string'
+}
+);
+
+common.expectsError(
+()=>moduleMap.set(1,moduleJob),
+{
+code: 'ERR_INVALID_ARG_TYPE',
+type: TypeError,
+message: 'The "url" argument must be of type string'
+}
+);
+
+common.expectsError(
+()=>moduleMap.set('somestring','notamodulejob'),
+{
+code: 'ERR_INVALID_ARG_TYPE',
+type: TypeError,
+message: 'The "job" argument must be of type ModuleJob'
+}
+);
+
+common.expectsError(
+()=>moduleMap.has(1),
+{
+code: 'ERR_INVALID_ARG_TYPE',
+type: TypeError,
+message: 'The "url" argument must be of type string'
+}
+);