Merge pull request #117 from mydea/disallowlist

Author: Robert Jackson

__tests__/index-test.js

@@ -179,7 +179,7 @@ describe(`ember-modules-api-polyfill-reexport`, () => {
 
 // Ensure unknown exports are not removed
 describe(`unknown imports from known module`, () => {
-  it(`allows blacklisting import paths`, () => {
+  it(`allows disallowing import paths`, () => {
     let input = `import { derp } from '@ember/object/computed';`;
 
     expect(() => {
@@ -210,22 +210,20 @@ export { capitalize };`
 });
 
 describe('options', () => {
-  describe('blacklist', () => {
-    it(`allows blacklisting import paths`, () => {
+  describe('ignore', () => {
+    it(`allows ignoring import paths`, () => {
       let input = `import { assert } from '@ember/debug';`;
-      let actual = transform(input, [
-        [Plugin, { blacklist: ['@ember/debug'] }],
-      ]);
+      let actual = transform(input, [[Plugin, { ignore: ['@ember/debug'] }]]);
 
       expect(actual).toEqual(input);
     });
 
-    it(`allows blacklisting specific named imports`, () => {
+    it(`allows ignoring specific named imports`, () => {
       let input = `import { assert, inspect } from '@ember/debug';var _x = inspect`;
       let actual = transform(input, [
         [
           Plugin,
-          { blacklist: { '@ember/debug': ['assert', 'warn', 'deprecate'] } },
+          { ignore: { '@ember/debug': ['assert', 'warn', 'deprecate'] } },
         ],
       ]);
 
@@ -234,9 +232,9 @@ describe('options', () => {
       );
     });
 
-    it('does not error when a blacklist is not present', () => {
+    it('does not error when ignore is not present', () => {
       let input = `import { assert, inspect } from '@ember/debug';var _x = assert; var _y = inspect;`;
-      let actual = transform(input, [[Plugin, { blacklist: {} }]]);
+      let actual = transform(input, [[Plugin, { ignore: {} }]]);
 
       expect(actual).toEqual(`var _x = Ember.assert;var _y = Ember.inspect;`);
     });

src/index.js

@@ -3,13 +3,13 @@
 const path = require('path');
 const mapping = require('ember-rfc176-data');
 
-function isBlacklisted(blacklist, importPath, exportName) {
-  if (Array.isArray(blacklist)) {
-    return blacklist.indexOf(importPath) > -1;
+function isIgnored(ignore, importPath, exportName) {
+  if (Array.isArray(ignore)) {
+    return ignore.indexOf(importPath) > -1;
   } else {
-    let blacklistedExports = blacklist[importPath];
+    let ignoredExports = ignore[importPath];
 
-    return blacklistedExports && blacklistedExports.indexOf(exportName) > -1;
+    return ignoredExports && ignoredExports.indexOf(exportName) > -1;
   }
 }
 
@@ -67,7 +67,7 @@ module.exports = function (babel) {
     name: 'ember-modules-api-polyfill',
     visitor: {
       ImportDeclaration(path, state) {
-        let blacklist = (state.opts && state.opts.blacklist) || [];
+        let ignore = (state.opts && state.opts.ignore) || [];
         let node = path.node;
         let declarations = [];
         let removals = [];
@@ -134,7 +134,7 @@ module.exports = function (babel) {
               importName = imported.name;
             }
 
-            if (isBlacklisted(blacklist, importPath, importName)) {
+            if (isIgnored(ignore, importPath, importName)) {
               return;
             }
 
@@ -188,7 +188,7 @@ module.exports = function (babel) {
       },
 
       ExportNamedDeclaration(path, state) {
-        let blacklist = (state.opts && state.opts.blacklist) || [];
+        let ignore = (state.opts && state.opts.ignore) || [];
         let node = path.node;
         if (!node.source) {
           return;
@@ -224,7 +224,7 @@ module.exports = function (babel) {
             // Determine the import name, either default or named
             let importName = local.name;
 
-            if (isBlacklisted(blacklist, importPath, importName)) {
+            if (isIgnored(ignore, importPath, importName)) {
               return;
             }