Merge pull request #5 from ssutar/decorators

Handle decorators

Author: pzuraq

package-lock.json

@@ -10,7 +10,7 @@
     "codemod-cli"
   ],
   "dependencies": {
-    "codemod-cli": "^0.1.0"
+    "codemod-cli": "ssutar/codemod-cli"
   },
   "devDependencies": {
     "jest": "^23.1.0"

package.json

@@ -1,7 +1,7 @@
 /**
  * Program comments
  */
-const Foo = EmberObject.extend({
+const Foo = Test.extend({
   /**
    * Property comments
    */

transforms/ember-object/__testfixtures__/basic.input.js

@@ -1,7 +1,7 @@
 /**
  * Program comments
  */
-class Foo extends EmberObject {
+class Foo extends Test {
   /**
    * Property comments
    */
@@ -31,4 +31,4 @@ class Foo extends EmberObject {
   anotherMethod() {
     super.anotherMethod(...arguments);
   }
-}
+}

transforms/ember-object/__testfixtures__/basic.output.js

@@ -0,0 +1,85 @@
+import { alias, sum as add } from "@ember/object/computed";
+import { get, set, observer, computed } from "@ember/object";
+import { inject as controller } from "@ember/controller";
+import { inject as service } from "@ember/service";
+import { on } from "@ember/object/evented";
+import layout from "components/templates/foo";
+
+const Foo = EmberObject.extend({
+  tagName: "div",
+  classNames: ["test-class", "custom-class"],
+  a: "",
+  b: service("store"),
+  myController: controller("abc"),
+  observedProp: observer("xyz"),
+  event: on("click"),
+
+  actions: {
+    /**
+    Comments
+    */
+    bar(temp1) {},
+    baz() {
+      this._super(...arguments);
+    },
+    biz() {}
+  }
+});
+
+var comp = EmberObject.extend({
+  classNameBindings: ["isEnabled:enabled:disabled", "a:b:c", "c:d"],
+  isEnabled: false,
+  a: true,
+  c: "",
+  attributeBindings: ["customHref:href"],
+
+  customHref: "http://emberjs.com"
+});
+
+const Foo = EmberObject.extend({
+  firstName: null,
+  lastName: null,
+
+  /**
+  Computed fullname
+  */
+  fullName: computed("firstName", "lastName", {
+    get(key) {
+      return `${this.get("firstName")} ${this.get("lastName")}`;
+    },
+    set(key, value) {
+      let [firstName, lastName] = value.split(/\s+/);
+      this.set("firstName", firstName);
+      this.set("lastName", lastName);
+      return value;
+    }
+  }),
+
+  /**
+  Computed description
+  */
+  description: computed("fullName", "age", "country", function() {
+    return `${this.get(
+      "fullName"
+    )}; Age: ${this.get("age")}; Country: ${this.get("country")}`;
+  }),
+
+  /**
+   * Fname
+   */
+  fName: alias("firstName"),
+
+  /**
+   * Lname
+   */
+  lName: alias("firstName", "lastName").readOnly(),
+
+  /**
+   * Lname1
+   */
+  lName1: add("description", "lastName").volatile()
+});
+
+const Foo = EmberObject.extend({
+  layout
+});

transforms/ember-object/__testfixtures__/decorators.input.js

@@ -0,0 +1,15 @@
+// Do not transform
+const Foo = EmberObject.extend({
+  statefulObject: {},
+  statefulArray: []
+});
+
+// Do not transform if not a primitive value
+const Foo = EmberObject.extend({
+  macroValue: macro()
+});
+
+// Do not transform as extends Mixin
+const Foo = EmberObject.extend(MyMixin, {
+  biz: "div"
+});

transforms/ember-object/__testfixtures__/decorators.invalid.input.js

@@ -0,0 +1,3 @@
+{
+  "decorators": true
+}

transforms/ember-object/__testfixtures__/decorators.invalid.options.json

@@ -0,0 +1,15 @@
+// Do not transform
+const Foo = EmberObject.extend({
+  statefulObject: {},
+  statefulArray: []
+});
+
+// Do not transform if not a primitive value
+const Foo = EmberObject.extend({
+  macroValue: macro()
+});
+
+// Do not transform as extends Mixin
+const Foo = EmberObject.extend(MyMixin, {
+  biz: "div"
+});

transforms/ember-object/__testfixtures__/decorators.invalid.output.js

@@ -0,0 +1,3 @@
+{
+  "decorators": true
+}

transforms/ember-object/__testfixtures__/decorators.options.json

@@ -0,0 +1,103 @@
+import { alias, sum as add } from "@ember/object/computed";
+import { get, set, observer, computed } from "@ember/object";
+import { inject as controller } from "@ember/controller";
+import { inject as service } from "@ember/service";
+import { on } from "@ember/object/evented";
+import layout from "components/templates/foo";
+
+@tagName("div")
+@classNames(["test-class", "custom-class"])
+class Foo extends EmberObject {
+  a = "";
+
+  @service("store")
+  b;
+
+  @controller("abc")
+  myController;
+
+  @observes("xyz")
+  observedProp;
+
+  @on("click")
+  event;
+
+  /**
+  Comments
+  */
+  @action
+  bar(temp1) {}
+
+  @action
+  baz() {
+    super.baz(...arguments);
+  }
+
+  @action
+  biz() {}
+}
+
+class Comp extends EmberObject {
+  @className("enabled", "disabled")
+  isEnabled = false;
+
+  @className("b", "c")
+  a = true;
+
+  @className("d")
+  c = "";
+
+  @attribute("href")
+  customHref = "http://emberjs.com";
+}
+
+class Foo extends EmberObject {
+  firstName = null;
+  lastName = null;
+
+  /**
+  Computed fullname
+  */
+  @computed("firstName", "lastName")
+  get fullName(key) {
+    return `${this.get("firstName")} ${this.get("lastName")}`;
+  }
+
+  set fullName(key, value) {
+    let [firstName, lastName] = value.split(/\s+/);
+    this.set("firstName", firstName);
+    this.set("lastName", lastName);
+    return value;
+  }
+
+  /**
+  Computed description
+  */
+  @computed("fullName", "age", "country")
+  description() {
+    return `${this.get(
+      "fullName"
+    )}; Age: ${this.get("age")}; Country: ${this.get("country")}`;
+  }
+
+  /**
+   * Fname
+   */
+  @alias("firstName")
+  fName;
+
+  /**
+   * Lname
+   */
+  @alias.readOnly("firstName", "lastName")
+  lName;
+
+  /**
+   * Lname1
+   */
+  @add.volatile("description", "lastName")
+  lName1;
+}
+
+@layout(layout)
+class Foo extends EmberObject {}