From b6f6eb5314eef14a5606336531000d7df4054717 Mon Sep 17 00:00:00 2001
From: Nick Woodward <nick@salte.io>
Date: Sat, 6 Apr 2019 16:03:20 -0500
Subject: [PATCH] fix: respect setting the deprecated fields "module", "main",
 and "jsnext" (#204)

---
 src/index.js                     |  2 +-
 test/samples/prefer-main/main.js |  3 +++
 test/test.js                     | 33 ++++++++++++++++++++++++++++++++
 3 files changed, 37 insertions(+), 1 deletion(-)
 create mode 100644 test/samples/prefer-main/main.js

diff --git a/src/index.js b/src/index.js
index 37b9ed6..1a677fa 100644
--- a/src/index.js
+++ b/src/index.js
@@ -51,7 +51,7 @@ function getMainFields (options) {
 				// eslint-disable-next-line no-console
 				console.warn(`node-resolve: setting options.${option} is deprecated, please override options.mainFields instead`);
 				if (options[option] === false) {
-					mainFields = mainFields.filter(mainField => mainField === field);
+					mainFields = mainFields.filter(mainField => mainField !== field);
 				} else if (options[option] === true && mainFields.indexOf(field) === -1) {
 					mainFields.push(field);
 				}
diff --git a/test/samples/prefer-main/main.js b/test/samples/prefer-main/main.js
new file mode 100644
index 0000000..c5a945d
--- /dev/null
+++ b/test/samples/prefer-main/main.js
@@ -0,0 +1,3 @@
+import entry from 'entries';
+
+export default entry;
diff --git a/test/test.js b/test/test.js
index 102ef2b..504094c 100644
--- a/test/test.js
+++ b/test/test.js
@@ -547,6 +547,39 @@ describe( 'rollup-plugin-node-resolve', function () {
 		});
 	});
 
+	it( 'should support disabling "module" field resolution', function () {
+		return rollup.rollup({
+			input: 'samples/prefer-main/main.js',
+			plugins: [
+				nodeResolve({ module: false })
+			]
+		}).then( executeBundle ).then( module => {
+			assert.equal( module.exports, 'MAIN-ENTRY' );
+		});
+	});
+
+	it( 'should support disabling "main" field resolution', function () {
+		return rollup.rollup({
+			input: 'samples/prefer-module/main.js',
+			plugins: [
+				nodeResolve({ main: false })
+			]
+		}).then( executeBundle ).then( module => {
+			assert.equal( module.exports, 'MODULE-ENTRY' );
+		});
+	});
+
+	it( 'should support enabling "jsnext" field resolution', function () {
+		return rollup.rollup({
+			input: 'samples/prefer-module/main.js',
+			plugins: [
+				nodeResolve({ main: false, module: false, jsnext: true })
+			]
+		}).then( executeBundle ).then( module => {
+			assert.equal( module.exports, 'JSNEXT-ENTRY' );
+		});
+	});
+
 	describe( 'symlinks', () => {
 		function createMissingDirectories () {
 			createDirectory( './samples/symlinked/first/node_modules' );