resolvers/node: [New] add .node extension

Fixes #802
import-js · Feb 18, 2020 · a1af591 · a1af591
1 parent 12971f5
commit a1af591
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 2 deletions.
diff --git a/resolvers/node/index.js b/resolvers/node/index.js
@@ -28,7 +28,7 @@ function opts(file, config) {
   return Object.assign({
       // more closely matches Node (#333)
       // plus 'mjs' for native modules! (#939)
-      extensions: ['.mjs', '.js', '.json'],
+      extensions: ['.mjs', '.js', '.json', '.node'],
     },
     config,
     {

diff --git a/resolvers/node/test/dot-node.node b/resolvers/node/test/dot-node.node
diff --git a/resolvers/node/test/native.node b/resolvers/node/test/native.node
diff --git a/resolvers/node/test/paths.js b/resolvers/node/test/paths.js
@@ -40,10 +40,21 @@ describe("default options", function () {
       .equal(path.resolve(__dirname, './native.mjs'))
   })
 
+  it("finds .node modules, with lowest precedence", function () {
+    expect(node.resolve('./native.node', './test/file.js'))
+      .to.have.property('path')
+      .equal(path.resolve(__dirname, './native.node'))
+  })
+
+  it("finds .node modules", function () {
+    expect(node.resolve('./dot-node', './test/file.js'))
+      .to.have.property('path')
+      .equal(path.resolve(__dirname, './dot-node.node'))
+  })
+
   it("still finds .js if explicit", function () {
     expect(node.resolve('./native.js', './test/file.js'))
       .to.have.property('path')
       .equal(path.resolve(__dirname, './native.js'))
   })
-
 })