Merge pull request #69 from kellyselden/remove_blank

remove imports without specifiers

Author: chadhietala

fixtures/removes-imports-without-specifiers/expectation6.js

@@ -0,0 +1,2 @@
+import '@ember/debug-tools';
+import '@glimmer/env';

fixtures/removes-imports-without-specifiers/expectation7.js

@@ -83,7 +83,11 @@ function macros(babel) {
               let debugToolsImport = options.debugTools.debugToolsImport;
 
               if (debugToolsImport && debugToolsImport === importPath) {
-                this.macroBuilder.collectDebugToolsSpecifiers(item.get('specifiers'));
+                if (!item.node.specifiers.length) {
+                  item.remove();
+                } else {
+                  this.macroBuilder.collectDebugToolsSpecifiers(item.get('specifiers'));
+                }
               }
             }
           });

fixtures/removes-imports-without-specifiers/sample.js

@@ -327,6 +327,26 @@ function createTests(options) {
     h.generateTest('does-not-modify-non-imported-flags');
   });
 
+  describe('Removes Imports Without Specifiers', function() {
+    let h = transformTestHelper({
+      presets,
+      plugins: [
+        [
+          DebugToolsPlugin,
+          {
+            debugTools: {
+              isDebug: true,
+              source: '@ember/debug-tools',
+            },
+            flags: [{ source: '@glimmer/env', flags: { DEBUG: true } }],
+          },
+        ],
+      ],
+    });
+
+    h.generateTest('removes-imports-without-specifiers');
+  });
+
   describe('Runtime Feature Flags', function() {
     let h = transformTestHelper({
       presets,