Merge pull request #1385 from monsanto/fix-var-checker

fix reference check for shorthand properties

src/babel/types/validators.js

@@ -26,10 +26,11 @@ export function isReferenced(node: Object, parent: Object): boolean {
       return false;
 
     // yes: { [NODE]: "" }
+    // yes: { NODE }
     // no: { NODE: "" }
     case "Property":
       if (parent.key === node) {
-        return parent.computed;
+        return parent.computed || parent.shorthand;
       }
 
     // no: var NODE = init;

test/core/fixtures/transformation/spec.function-name/assignment/expected.js

@@ -4,9 +4,19 @@ var _i = function i() {
   _i = 5;
 };
 
-var j = function j() {
+var j = (function (_j) {
+  function j() {
+    return _j.apply(this, arguments);
+  }
+
+  j.toString = function () {
+    return _j.toString();
+  };
+
+  return j;
+})(function () {
   var _ = 5;
   j = _.j;
 
   ;
-};
+});

test/core/fixtures/transformation/validation.undeclared-variable-check/undeclared-shorthand-property/exec.js

@@ -0,0 +1 @@
+({foo})

test/core/fixtures/transformation/validation.undeclared-variable-check/undeclared-shorthand-property/options.json

@@ -0,0 +1,3 @@
+{
+  "throws": "Reference to undeclared variable"
+}