Revert "module: fix inconsistency between load and _findPath"

This reverts commit 1b92214 from PR #22382. See the discussion at nodejs/citgm#604 Refs: #22382 Fixes: #4778
nodejs · jdalton · Sep 26, 2018 · Oct 3, 2018 · b954c9be59e7a00bb8c08a3e7aa9486675612e04
commit b954c9be59e7a00bb8c08a3e7aa9486675612e04
diff --git a/lib/internal/modules/cjs/loader.js b/lib/internal/modules/cjs/loader.js
@@ -217,16 +217,6 @@ function tryExtensions(p, exts, isMain) {
   return false;
 }
 
-function readExtensions() {
-  const exts = Object.keys(Module._extensions);
-  for (var i = 0, j = 0; i < exts.length; ++i) {
-    if (path.extname(exts[i]) === '')
-      exts[j++] = exts[i];
-  }
-  exts.length = j;
-  return exts;
-}
-
 var warned = false;
 Module._findPath = function(request, paths, isMain) {
   if (path.isAbsolute(request)) {
@@ -283,15 +273,15 @@ Module._findPath = function(request, paths, isMain) {
       if (!filename) {
         // try it with each of the extensions
         if (exts === undefined)
-          exts = readExtensions();
+          exts = Object.keys(Module._extensions);
         filename = tryExtensions(basePath, exts, isMain);
       }
     }
 
     if (!filename && rc === 1) {  // Directory.
       // try it with each of the extensions at "index"
       if (exts === undefined)
-        exts = readExtensions();
+        exts = Object.keys(Module._extensions);
       filename = tryPackage(basePath, exts, isMain);
       if (!filename) {
         filename = tryExtensions(path.resolve(basePath, 'index'), exts, isMain);

diff --git a/test/known_issues/test-module-deleted-extensions.js b/test/known_issues/test-module-deleted-extensions.js
@@ -0,0 +1,18 @@
+'use strict';
+// Refs: https://github.com/nodejs/node/issues/4778
+const common = require('../common');
+const assert = require('assert');
+const fs = require('fs');
+const path = require('path');
+const tmpdir = require('../common/tmpdir');
+const file = path.join(tmpdir.path, 'test-extensions.foo.bar');
+
+tmpdir.refresh();
+fs.writeFileSync(file, '', 'utf8');
+require.extensions['.foo.bar'] = (module, path) => {};
+delete require.extensions['.foo.bar'];
+require.extensions['.bar'] = common.mustCall((module, path) => {
+  assert.strictEqual(module.id, file);
+  assert.strictEqual(path, file);
+});
+require(path.join(tmpdir.path, 'test-extensions'));
diff --git a/test/parallel/test-module-deleted-extensions.js b/test/parallel/test-module-deleted-extensions.js