Treat sideEffects array as inclusion list (#227)

* Treat sideEffects array as inclusion list
- Fixes #226

* Rename 'true-index' to 'array-index' for clarity

* Inline unnecessary local variable

* Rename test files from "-free" to "-effect"

Author: mikeharder

src/index.js

@@ -145,8 +145,7 @@ export default function nodeResolve ( options = {} ) {
 		if (typeof packageSideEffects === 'boolean') {
 			packageInfo.hasModuleSideEffects = () => packageSideEffects;
 		} else if (Array.isArray(packageSideEffects)) {
-			const filter = createFilter(packageSideEffects, null, {resolve: pkgRoot});
-			packageInfo.hasModuleSideEffects = id => !filter(id);
+			packageInfo.hasModuleSideEffects = createFilter(packageSideEffects, null, {resolve: pkgRoot});
 		}
 
 		packageInfoCache.set(pkgPath, packageInfo);

test/node_modules/side-effects-array/dep3-free.js test/node_modules/side-effects-array/dep3-effect.js

@@ -890,9 +890,10 @@ describe( 'rollup-plugin-node-resolve', function () {
 				'true-dep1',
 				'true-dep2',
 				'true-index',
-				'array-dep2',
-				'array-dep4',
-				'true-index'
+				'array-dep1',
+				'array-dep3',
+				'array-dep5',
+				'array-index'
 			]);
 			delete global.sideEffects;
 		}));