fix-object-properties

.eslintignore

@@ -1,4 +1,4 @@
 node_modules
 test/binding
 test/import-identifiers
-test/object-keys
+test/object-keys-properties

lib/index.js

@@ -97,8 +97,9 @@ const plugin = function ({ types: t }) {
           // Don't transform import identifiers. This is meant to mimic webpack's
           // DefinePlugin behavior.
           || isImportIdentifier(nodePath)
-          // Do not transform Object keys unless they are computed like {[key]: value}
+          // Do not transform Object keys / properties unless they are computed like {[key]: value}
           || nodePath.key === "key" && nodePath.parent.computed === false
+          || nodePath.key === "property" && nodePath.parent.computed === false
         ) {
           return;
         }

test/index.js

@@ -202,15 +202,15 @@ describe("babel-plugin-transform-define", () => {
         path.join(__dirname, "./binding/expected.js"), babelOpts);
     });
 
-    it("should not transform object keys unless they are computed", () => {
+    it("should not transform object keys / properties unless they are computed", () => {
       const babelOpts = getBabelOps({
         __DEV__: true,
         __DEV2__: "true"
       });
 
       return assertTransform(
-        path.join(__dirname, "./object-keys/actual.js"),
-        path.join(__dirname, "./object-keys/expected.js"), babelOpts);
+        path.join(__dirname, "./object-keys-properties/actual.js"),
+        path.join(__dirname, "./object-keys-properties/expected.js"), babelOpts);
     });
   });
 

test/object-keys-properties/actual.js

@@ -0,0 +1,28 @@
+const obj = {
+    __DEV__
+  };
+const obj1 = {
+  __DEV__: "test"
+};
+const obj2 = {
+  __DEV__: __DEV__
+};
+const obj3 = {
+  "__DEV__": __DEV__
+};
+const obj4 = {
+  ["__DEV__"]: __DEV__
+};
+
+const obj5 = {
+  [__DEV__]: __DEV__
+};
+
+const obj6 = {
+  [__DEV2__]: __DEV2__
+};
+
+
+const access = obj.__DEV__;
+const access1 = obj[__DEV2__];
+const access2 = obj["__DEV__"];

test/object-keys/expected.js → test/object-keys-properties/expected.js

@@ -19,4 +19,8 @@ var obj4 = _defineProperty({}, "__DEV__", true);
 
 var obj5 = _defineProperty({}, true, true);
 
-var obj6 = _defineProperty({}, "true", "true");
+var obj6 = _defineProperty({}, "true", "true");
+
+var access = obj.__DEV__;
+var access1 = obj["true"];
+var access2 = obj["__DEV__"];