Add test case to cover non-enumerable keys made enumerable by a previ…

…ous getter.

From https://bugzilla.mozilla.org/show_bug.cgi?id=1208464#c13 and tc39/proposal-object-values-entries#5.

.eslintrc

@@ -7,6 +7,7 @@
 		"max-nested-callbacks": [2, 3],
 		"max-statements": [2, 12],
 		"new-cap": [2, { "capIsNewExceptions": ["RequireObjectCoercible"] }],
-		"no-invalid-this": [1]
+		"no-invalid-this": [1],
+		"no-restricted-syntax": [2, "BreakStatement", "ContinueStatement", "DebuggerStatement", "LabeledStatement", "WithStatement"]
 	}
 }

.jscs.json

@@ -20,7 +20,6 @@
 
 	"disallowNodeTypes": [
 		"DebuggerStatement",
-		"ForInStatement",
 		"LabeledStatement",
 		"SwitchCase",
 		"SwitchStatement",

implementation.js

@@ -1,16 +1,16 @@
 'use strict';
 
 var ES = require('es-abstract/es7');
-var keys = require('object-keys');
 var has = require('has');
+var bind = require('function-bind');
+var isEnumerable = bind.call(Function.call, Object.prototype.propertyIsEnumerable);
 
 module.exports = function entries(O) {
 	var obj = ES.RequireObjectCoercible(O);
-	var objKeys = keys(obj);
 	var entrys = [];
-	for (var i = 0; i < objKeys.length; ++i) {
-		if (has(obj, objKeys[i])) {
-			entrys.push([objKeys[i], obj[objKeys[i]]]);
+	for (var key in obj) {
+		if (has(obj, key) && isEnumerable(obj, key)) {
+			entrys.push([key, obj[key]]);
 		}
 	}
 	return entrys;

package.json

@@ -37,7 +37,7 @@
 		"define-properties": "^1.1.1",
 		"es-abstract": "^1.3.2",
 		"has": "^1.0.1",
-		"object-keys": "^1.0.7"
+		"function-bind": "^1.0.2"
 	},
 	"devDependencies": {
 		"tape": "^4.2.1",

test/tests.js

@@ -67,4 +67,16 @@ module.exports = function (entries, t) {
 		st.deepEqual(entries(o), [['a', 1], ['c', 3]], 'when "b" is deleted prior to being visited, it should not show up');
 		st.end();
 	});
+
+	t.test('not-yet-visited keys made non-enumerable on [[Get]] must not show up in output', { skip: !define.supportsDescriptors }, function (st) {
+		var o = { a: 'A', b: 'B' };
+		Object.defineProperty(o, 'a', {
+			get: function () {
+				Object.defineProperty(o, 'b', { enumerable: false });
+				return 'A';
+			}
+		});
+		st.deepEqual(entries(o), [['a', 'A']], 'when "b" is made non-enumerable prior to being visited, it should not show up');
+		st.end();
+	});
 };