v3.1.0

CHANGELOG.md

@@ -11,6 +11,11 @@
 
 _Note: Gaps between patch versions are faulty/broken releases._
 
+## 3.1.0
+
+ * **Breaking Change**
+  * [Make class methods unenumerable](/Users/sebastian/Projects/6to5/6to5/test/fixtures/esnext/es6-classes/getter-setter.js).
+
 ## 3.0.16
 
  * **Bug Fix**

lib/6to5/transformation/transformers/es6/classes.js

@@ -231,6 +231,7 @@ Class.prototype.pushMethod = function (node) {
   }
 
   util.pushMutatorMap(mutatorMap, methodName, kind, node.computed, node);
+  util.pushMutatorMap(mutatorMap, methodName, "enumerable", node.computed, t.literal(false));
 };
 
 /**

lib/6to5/util.js

@@ -81,7 +81,7 @@ exports.sourceMapToComment = function (map) {
   return "//# sourceMappingURL=data:application/json;base64," + base64;
 };
 
-exports.pushMutatorMap = function (mutatorMap, key, kind, computed, method) {
+exports.pushMutatorMap = function (mutatorMap, key, kind, computed, value) {
   var alias;
 
   if (t.isIdentifier(key)) {
@@ -106,7 +106,7 @@ exports.pushMutatorMap = function (mutatorMap, key, kind, computed, method) {
     map._computed = true;
   }
 
-  map[kind] = method;
+  map[kind] = value;
 };
 
 exports.buildDefineProperties = function (mutatorMap) {
@@ -121,7 +121,7 @@ exports.buildDefineProperties = function (mutatorMap) {
       map.writable = t.literal(true);
     }
 
-    map.enumerable = t.literal(true);
+    map.enumerable = map.enumerable || t.literal(true);
     map.configurable = t.literal(true);
 
     each(map, function (node, key) {

package.json

@@ -1,7 +1,7 @@
 {
   "name": "6to5",
   "description": "Turn ES6 code into readable vanilla ES5 with source maps",
-  "version": "3.0.16",
+  "version": "3.1.0",
   "author": "Sebastian McKenzie <sebmck@gmail.com>",
   "homepage": "https://6to5.org/",
   "repository": "6to5/6to5",

packages/6to5-runtime/package.json

@@ -1,7 +1,7 @@
 {
   "name": "6to5-runtime",
   "description": "6to5 selfContained runtime",
-  "version": "3.0.15",
+  "version": "3.0.16",
   "repository": "6to5/6to5",
   "author": "Sebastian McKenzie <sebmck@gmail.com>"
 }

test/fixtures/esnext/es6-classes/enumerable.js

@@ -17,4 +17,4 @@ for (var key in point) {
 }
 
 assert.equal(point.toString(), '(1, 2)');
-assert.deepEqual(keys.sort(), ['toString', 'x', 'y']);
+assert.deepEqual(keys.sort(), ['x', 'y']);

test/fixtures/esnext/es6-classes/getter-setter.js

@@ -25,4 +25,4 @@ var forLoopProperties = [];
 for (var key in mazer) {
   forLoopProperties.push(key);
 }
-assert.ok(forLoopProperties.indexOf('name') >= 0, 'getters/setters are enumerable');
+assert.ok(forLoopProperties.indexOf('name') === -1, 'getters/setters should be unenumerable');

test/fixtures/transformation/async-to-generator/async/expected.js

@@ -12,11 +12,11 @@ var Foo = (function () {
       value: _asyncToGenerator(function* () {
         var wat = yield bar();
       }),
+      enumerable: false,
       writable: true,
-      enumerable: true,
       configurable: true
     }
   });
 
   return Foo;
-})();
+})();

test/fixtures/transformation/bluebird-coroutines/class/expected.js

@@ -12,11 +12,11 @@ var Foo = (function () {
       value: _bluebird.coroutine(function* () {
         var wat = yield bar();
       }),
+      enumerable: false,
       writable: true,
-      enumerable: true,
       configurable: true
     }
   });
 
   return Foo;
-})();
+})();

test/fixtures/transformation/es6-classes/accessing-super-class/expected.js

@@ -31,8 +31,8 @@ var Test = (function (Foo) {
         _get(Object.getPrototypeOf(Test), "foo", this).apply(this, arguments);
         (_get2 = _get(Object.getPrototypeOf(Test), "foo", this)).call.apply(_get2, [this, "test"].concat(_slice.call(arguments)));
       },
+      enumerable: false,
       writable: true,
-      enumerable: true,
       configurable: true
     }
   }, {
@@ -43,8 +43,8 @@ var Test = (function (Foo) {
         _get(Object.getPrototypeOf(Test.prototype), "test", this).apply(this, arguments);
         (_get2 = _get(Object.getPrototypeOf(Test.prototype), "test", this)).call.apply(_get2, [this, "test"].concat(_slice.call(arguments)));
       },
+      enumerable: false,
       writable: true,
-      enumerable: true,
       configurable: true
     }
   });

test/fixtures/transformation/es6-classes/calling-super-properties/expected.js

@@ -19,8 +19,8 @@ var Test = (function (Foo) {
       value: function test() {
         return _get(Object.getPrototypeOf(Test), "wow", this).call(this);
       },
+      enumerable: false,
       writable: true,
-      enumerable: true,
       configurable: true
     }
   });

test/fixtures/transformation/es6-classes/instance-getter-and-setter/expected.js

@@ -10,10 +10,10 @@ var Test = (function () {
       get: function () {
         return 5 + 5;
       },
+      enumerable: false,
       set: function (val) {
         this._test = val;
       },
-      enumerable: true,
       configurable: true
     }
   });

test/fixtures/transformation/es6-classes/instance-getter/expected.js

@@ -10,7 +10,7 @@ var Test = (function () {
       get: function () {
         return 5 + 5;
       },
-      enumerable: true,
+      enumerable: false,
       configurable: true
     }
   });

test/fixtures/transformation/es6-classes/instance-method/expected.js

@@ -10,8 +10,8 @@ var Test = (function () {
       value: function test() {
         return 5 + 5;
       },
+      enumerable: false,
       writable: true,
-      enumerable: true,
       configurable: true
     }
   });

test/fixtures/transformation/es6-classes/instance-setter/expected.js

@@ -10,7 +10,7 @@ var Test = (function () {
       set: function (val) {
         this._test = val;
       },
-      enumerable: true,
+      enumerable: false,
       configurable: true
     }
   });

test/fixtures/transformation/es6-classes/statement/expected.js

@@ -18,8 +18,8 @@ var BaseView = (function () {
       value: function foo() {
         this.autoRender = true;
       },
+      enumerable: false,
       writable: true,
-      enumerable: true,
       configurable: true
     }
   });

test/fixtures/transformation/es6-classes/static/expected.js

@@ -8,14 +8,14 @@ var A = (function () {
   _prototypeProperties(A, {
     a: {
       value: function a() {},
+      enumerable: false,
       writable: true,
-      enumerable: true,
       configurable: true
     },
     b: {
       get: function () {},
+      enumerable: false,
       set: function (b) {},
-      enumerable: true,
       configurable: true
     }
   });

test/fixtures/transformation/playground/object-getter-memoization/expected.js

@@ -12,14 +12,14 @@ var Foo = (function () {
       get: function () {
         return _defineProperty(this, "bar", complex()).bar;
       },
-      enumerable: true,
+      enumerable: false,
       configurable: true
     }
   }, bar, {
     get: function () {
       return _defineProperty(this, bar, complex())[bar];
     },
-    enumerable: true,
+    enumerable: false,
     configurable: true
   }));
 

test/fixtures/transformation/source-maps/class/expected.js

@@ -10,7 +10,7 @@ var Test = (function () {
       get: function () {
         throw new Error("wow");
       },
-      enumerable: true,
+      enumerable: false,
       configurable: true
     }
   });