Merge pull request #14 from ssutar/class_fields

Add options for class fields

Author: pzuraq

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

@@ -1,7 +1,7 @@
 /**
  * Program comments
  */
-const Foo = Test.extend({
+const Foo = Test.extend(MyMixin, {
   /**
    * Property comments
    */
@@ -31,3 +31,5 @@ const Foo = Test.extend({
     this._super(...arguments);
   }
 });
+
+const Foo = EmberObject.extend(MixinA, MixinB);

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

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

transforms/ember-object/__testfixtures__/class-fields.invalid.input.js

@@ -0,0 +1,33 @@
+/**
+ * Program comments
+ */
+const Foo = Test.extend({
+  /**
+   * Property comments
+   */
+  prop: "defaultValue",
+  boolProp: true,
+  numProp: 123,
+  [MY_VAL]: "val",
+
+  /**
+   * Method comments
+   */
+  method() {
+    // do things
+  },
+
+  otherMethod: function() {},
+
+  get accessor() {
+    return this._value;
+  },
+
+  set accessor(value) {
+    this._value = value;
+  },
+
+  anotherMethod() {
+    this._super(...arguments);
+  }
+});

transforms/ember-object/__testfixtures__/class-fields.invalid.options.json

@@ -0,0 +1,3 @@
+{
+  "classFields": false
+}

transforms/ember-object/__testfixtures__/class-fields.invalid.output.js

@@ -0,0 +1,33 @@
+/**
+ * Program comments
+ */
+const Foo = Test.extend({
+  /**
+   * Property comments
+   */
+  prop: "defaultValue",
+  boolProp: true,
+  numProp: 123,
+  [MY_VAL]: "val",
+
+  /**
+   * Method comments
+   */
+  method() {
+    // do things
+  },
+
+  otherMethod: function() {},
+
+  get accessor() {
+    return this._value;
+  },
+
+  set accessor(value) {
+    this._value = value;
+  },
+
+  anotherMethod() {
+    this._super(...arguments);
+  }
+});

transforms/ember-object/__testfixtures__/class-fields.valid.input.js

@@ -0,0 +1,25 @@
+/**
+ * Program comments
+ */
+const Foo = Test.extend({
+  /**
+   * Method comments
+   */
+  method() {
+    // do things
+  },
+
+  otherMethod: function() {},
+
+  get accessor() {
+    return this._value;
+  },
+
+  set accessor(value) {
+    this._value = value;
+  },
+
+  anotherMethod() {
+    this._super(...arguments);
+  }
+});

transforms/ember-object/__testfixtures__/class-fields.valid.options.json

@@ -0,0 +1,3 @@
+{
+  "classFields": false
+}

transforms/ember-object/__testfixtures__/class-fields.valid.output.js

@@ -0,0 +1,25 @@
+/**
+ * Program comments
+ */
+class Foo extends Test {
+  /**
+   * Method comments
+   */
+  method() {
+    // do things
+  }
+
+  otherMethod() {}
+
+  get accessor() {
+    return this._value;
+  }
+
+  set accessor(value) {
+    this._value = value;
+  }
+
+  anotherMethod() {
+    super.anotherMethod(...arguments);
+  }
+}

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

@@ -8,8 +8,3 @@ const Foo = EmberObject.extend({
 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

@@ -8,8 +8,3 @@ const Foo = EmberObject.extend({
 const Foo = EmberObject.extend({
   macroValue: macro()
 });
-
-// Do not transform as extends Mixin
-const Foo = EmberObject.extend(MyMixin, {
-  biz: "div"
-});