Add tests with subpath mapped to null

Author: aduh95

test/es-module/test-esm-exports.mjs

@@ -45,6 +45,10 @@ import fromInside from '../fixtures/node_modules/pkgexports/lib/hole.js';
     ['pkgexports/a//dir1/dir1', { default: 'main' }],
     // double slash target
     ['pkgexports/doubleslash', { default: 'asdf' }],
+    // Null target with several slashes
+    ['pkgexports/sub//internal/test.js', { default: 'internal only' }],
+    ['pkgexports/sub//internal//test.js', { default: 'internal only' }],
+    ['pkgexports/sub/////internal/////test.js', { default: 'internal only' }],
     // trailing slash
     ['pkgexports/trailing-pattern-slash/',
      { default: 'trailing-pattern-slash' }],
@@ -79,7 +83,11 @@ import fromInside from '../fixtures/node_modules/pkgexports/lib/hole.js';
     ['pkgexports/invalid1', './invalid1'],
     ['pkgexports/invalid4', './invalid4'],
     // Null mapping
+    ['pkgexports/sub/internal/test.js', './sub/internal/test.js'],
+    ['pkgexports/sub/internal//test.js', './sub/internal//test.js'],
     ['pkgexports/null', './null'],
+    ['pkgexports//null', './/null'],
+    ['pkgexports/////null', './////null'],
     ['pkgexports/null/subpath', './null/subpath'],
     // Empty fallback
     ['pkgexports/nofallback1', './nofallback1'],

test/es-module/test-esm-imports.mjs

@@ -79,10 +79,14 @@ const { requireImport, importImport } = importer;
     '#missing',
     // Explicit null import
     '#null',
+    '#subpath/null',
     // No condition match import
     '#nullcondition',
     // Null subpath shadowing
     '#subpath/nullshadow/x',
+    // Null pattern
+    '#subpath/internal/test',
+    '#subpath/internal//test',
   ]);
 
   for (const specifier of undefinedImports) {
@@ -94,10 +98,20 @@ const { requireImport, importImport } = importer;
   }
 
   // Handle not found for the defined imports target not existing
-  loadFixture('#notfound').catch(mustCall((err) => {
-    strictEqual(err.code,
-                isRequire ? 'MODULE_NOT_FOUND' : 'ERR_MODULE_NOT_FOUND');
-  }));
+  const nonDefinedImports = new Set([
+    '#notfound',
+    '#subpath//null',
+    '#subpath/////null',
+    '#subpath//internal/test',
+    '#subpath//internal//test',
+    '#subpath/////internal/////test',
+  ]);
+  for (const specifier of nonDefinedImports) {
+    loadFixture(specifier).catch(mustCall((err) => {
+      strictEqual(err.code,
+                  isRequire ? 'MODULE_NOT_FOUND' : 'ERR_MODULE_NOT_FOUND');
+    }));
+  }
 });
 
 // CJS resolver must still support #package packages in node_modules

test/fixtures/es-modules/pkgimports/package.json

@@ -6,6 +6,8 @@
       "require": "./requirebranch.js"
     },
     "#subpath/*": "./sub/*",
+    "#subpath/internal/*": null,
+    "#subpath/null": null,
     "#subpath/*.asdf": "./test.js",
     "#external": "pkgexports/valid-cjs",
     "#external/subpath/*": "pkgexports/sub/*",

test/fixtures/es-modules/pkgimports/sub/internal/test.js

@@ -0,0 +1,3 @@
+'use strict';
+
+module.exports = 'internal only';