add utility.removeClass tests and move parentPath setting to setContext - fixes #967

src/babel/traversal/path.js

@@ -4,8 +4,7 @@ import Scope from "./scope";
 import t from "../types";
 
 export default class TraversalPath {
-  constructor(parentPath, parent, container) {
-    this.parentPath = parentPath;
+  constructor(parent, container) {
     this.container = container;
     this.parent    = parent;
     this.data      = {};
@@ -25,11 +24,11 @@ export default class TraversalPath {
     }
 
     if (!path) {
-      path = new TraversalPath(parentPath, parent, container);
+      path = new TraversalPath(parent, container);
       paths.push(path);
     }
 
-    path.setContext(context, key);
+    path.setContext(parentPath, context, key);
 
     return path;
   }
@@ -57,11 +56,12 @@ export default class TraversalPath {
     this.scope = TraversalPath.getScope(this.node, this.parent, this.context.scope);
   }
 
-  setContext(context, key) {
+  setContext(parentPath, context, key) {
     this.shouldRemove = false;
     this.shouldSkip   = false;
     this.shouldStop   = false;
 
+    this.parentPath = parentPath;
     this.context    = context;
     this.state      = context.state;
     this.opts       = context.opts;

test/fixtures/transformation/utility.remove-console/nested/actual.js

@@ -0,0 +1,3 @@
+function foo() {
+  console.log("foo");
+}

test/fixtures/transformation/utility.remove-console/nested/expected.js

@@ -0,0 +1,3 @@
+"use strict";
+
+function foo() {}

test/fixtures/transformation/utility.remove-console/options.json

@@ -0,0 +1,3 @@
+{
+  "optional": ["utility.removeConsole"]
+}

test/fixtures/transformation/utility.remove-console/top-level/actual.js

@@ -0,0 +1 @@
+console.log("foo");

test/fixtures/transformation/utility.remove-console/top-level/expected.js

@@ -0,0 +1 @@
+"use strict";