diff --git a/test/built-ins/object/values/exception-during-enumeration.js b/test/built-ins/object/values/exception-during-enumeration.js new file mode 100644 index 0000000000..97efeed491 --- /dev/null +++ b/test/built-ins/object/values/exception-during-enumeration.js @@ -0,0 +1,21 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: Object.values should terminate if getting a value throws an exception +includes: [Test262Error.js] +author: Jordan Harband +---*/ + +var trappedKey = { + get a() { + throw new Test262Error('This error should be re-thrown'); + }, + get b() { + $ERROR('Should not try to get the second element'); + } +}; + +assert.throws(Test262Error, function () { + Object.values(trappedKey); +}); diff --git a/test/built-ins/object/values/exception-not-object-coercible.js b/test/built-ins/object/values/exception-not-object-coercible.js new file mode 100644 index 0000000000..0a45f73f28 --- /dev/null +++ b/test/built-ins/object/values/exception-not-object-coercible.js @@ -0,0 +1,15 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: Object.values should fail if given a null or undefined value +author: Jordan Harband +---*/ + +assert.throws(TypeError, function () { + Object.values(null); +}); + +assert.throws(TypeError, function () { + Object.values(undefined); +}); diff --git a/test/built-ins/object/values/function-length.js b/test/built-ins/object/values/function-length.js new file mode 100644 index 0000000000..9cd36f3463 --- /dev/null +++ b/test/built-ins/object/values/function-length.js @@ -0,0 +1,14 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: Object.values should have length 1 +author: Jordan Harband +includes: [propertyHelper.js] +---*/ + +assert.sameValue(Object.values.length, 1, 'Expected Object.values.length to be 1'); + +verifyNotEnumerable(Object.values, 'length'); +verifyNotWritable(Object.values, 'length'); +verifyConfigurable(Object.values, 'length'); diff --git a/test/built-ins/object/values/function-name.js b/test/built-ins/object/values/function-name.js new file mode 100644 index 0000000000..3bd1d2464b --- /dev/null +++ b/test/built-ins/object/values/function-name.js @@ -0,0 +1,18 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: Object.values should have name property with value 'values' +author: Jordan Harband +includes: [propertyHelper.js] +---*/ + +assert.sameValue( + Object.values.name, + 'values', + 'Expected Object.values.name to be "values"' +); + +verifyNotEnumerable(Object.values, 'name'); +verifyNotWritable(Object.values, 'name'); +verifyConfigurable(Object.values, 'name'); diff --git a/test/built-ins/object/values/function-property-descriptor.js b/test/built-ins/object/values/function-property-descriptor.js new file mode 100644 index 0000000000..228bf62d29 --- /dev/null +++ b/test/built-ins/object/values/function-property-descriptor.js @@ -0,0 +1,12 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: Object.values should be writable, non-enumerable, and configurable +author: Jordan Harband +includes: [propertyHelper.js] +---*/ + +verifyConfigurable(Object, 'values'); +verifyNotEnumerable(Object, 'values'); +verifyWritable(Object, 'values'); diff --git a/test/built-ins/object/values/getter-adding-key.js b/test/built-ins/object/values/getter-adding-key.js new file mode 100644 index 0000000000..5b760ed59b --- /dev/null +++ b/test/built-ins/object/values/getter-adding-key.js @@ -0,0 +1,24 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: Object.values sees a new element added by a getter that is hit during iteration +author: Jordan Harband +---*/ + +var bAddsC = { + a: 'A', + get b() { + this.c = 'C'; + return 'B'; + } +}; + +var result = Object.values(bAddsC); + +assert.sameValue(Array.isArray(result), true, 'result is an array'); +assert.sameValue(result.length, 3, 'result has 3 items'); + +assert.sameValue(result[0], 'A', 'first value is "A"'); +assert.sameValue(result[1], 'B', 'second value is "B"'); +assert.sameValue(result[2], 'C', 'third value is "C"'); diff --git a/test/built-ins/object/values/getter-making-future-key-nonenumerable.js b/test/built-ins/object/values/getter-making-future-key-nonenumerable.js new file mode 100644 index 0000000000..f50d2e77ae --- /dev/null +++ b/test/built-ins/object/values/getter-making-future-key-nonenumerable.js @@ -0,0 +1,26 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: Object.values does not see an element made non-enumerable by a getter that is hit during iteration +author: Jordan Harband +---*/ + +var bDeletesC = { + a: 'A', + get b() { + Object.defineProperty(this, 'c', { + enumerable: false + }); + return 'B'; + }, + c: 'C' +}; + +var result = Object.values(bDeletesC); + +assert.sameValue(Array.isArray(result), true, 'result is an array'); +assert.sameValue(result.length, 2, 'result has 2 items'); + +assert.sameValue(result[0], 'A', 'first value is "A"'); +assert.sameValue(result[1], 'B', 'second value is "B"'); diff --git a/test/built-ins/object/values/getter-removing-future-key.js b/test/built-ins/object/values/getter-removing-future-key.js new file mode 100644 index 0000000000..c0dc869af2 --- /dev/null +++ b/test/built-ins/object/values/getter-removing-future-key.js @@ -0,0 +1,24 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: Object.values does not see an element removed by a getter that is hit during iteration +author: Jordan Harband +---*/ + +var bDeletesC = { + a: 'A', + get b() { + delete this.c; + return 'B'; + }, + c: 'C' +}; + +var result = Object.values(bDeletesC); + +assert.sameValue(Array.isArray(result), true, 'result is an array'); +assert.sameValue(result.length, 2, 'result has 2 items'); + +assert.sameValue(result[0], 'A', 'first value is "A"'); +assert.sameValue(result[1], 'B', 'second value is "B"'); diff --git a/test/built-ins/object/values/inherited-properties-omitted.js b/test/built-ins/object/values/inherited-properties-omitted.js new file mode 100644 index 0000000000..6611852fd0 --- /dev/null +++ b/test/built-ins/object/values/inherited-properties-omitted.js @@ -0,0 +1,23 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: Object.values does not see inherited properties. +author: Jordan Harband +---*/ + +var F = function G() {}; +F.prototype.a = {}; +F.prototype.b = {}; + +var f = new F(); +f.b = {}; // shadow the prototype +f.c = {}; // solely an own property + +var result = Object.values(f); + +assert.sameValue(Array.isArray(result), true, 'result is an array'); +assert.sameValue(result.length, 2, 'result has 2 items'); + +assert.sameValue(result[0], f.b, 'first value is f.b'); +assert.sameValue(result[1], f.c, 'second value is f.c'); diff --git a/test/built-ins/object/values/primitive-booleans.js b/test/built-ins/object/values/primitive-booleans.js new file mode 100644 index 0000000000..84d019d084 --- /dev/null +++ b/test/built-ins/object/values/primitive-booleans.js @@ -0,0 +1,17 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: Object.values accepts boolean primitives. +author: Jordan Harband +---*/ + +var trueResult = Object.values(true); + +assert.sameValue(Array.isArray(trueResult), true, 'trueResult is an array'); +assert.sameValue(trueResult.length, 0, 'trueResult has 0 items'); + +var falseResult = Object.values(false); + +assert.sameValue(Array.isArray(falseResult), true, 'falseResult is an array'); +assert.sameValue(falseResult.length, 0, 'falseResult has 0 items'); diff --git a/test/built-ins/object/values/primitive-numbers.js b/test/built-ins/object/values/primitive-numbers.js new file mode 100644 index 0000000000..64fda45d0c --- /dev/null +++ b/test/built-ins/object/values/primitive-numbers.js @@ -0,0 +1,14 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: Object.values accepts number primitives. +author: Jordan Harband +---*/ + +assert.sameValue(Object.values(0).length, 0, '0 has zero values'); +assert.sameValue(Object.values(-0).length, 0, '-0 has zero values'); +assert.sameValue(Object.values(Infinity).length, 0, 'Infinity has zero values'); +assert.sameValue(Object.values(-Infinity).length, 0, '-Infinity has zero values'); +assert.sameValue(Object.values(NaN).length, 0, 'NaN has zero values'); +assert.sameValue(Object.values(Math.PI).length, 0, 'Math.PI has zero values'); diff --git a/test/built-ins/object/values/primitive-strings.js b/test/built-ins/object/values/primitive-strings.js new file mode 100644 index 0000000000..adb0af1e50 --- /dev/null +++ b/test/built-ins/object/values/primitive-strings.js @@ -0,0 +1,16 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: Object.values accepts string primitives. +author: Jordan Harband +---*/ + +var result = Object.values('abc'); + +assert.sameValue(Array.isArray(result), true, 'result is an array'); +assert.sameValue(result.length, 3, 'result has 3 items'); + +assert.sameValue(result[0], 'a', 'first value is "a"'); +assert.sameValue(result[1], 'b', 'second value is "b"'); +assert.sameValue(result[2], 'c', 'second value is "c"'); diff --git a/test/built-ins/object/values/primitive-symbols.js b/test/built-ins/object/values/primitive-symbols.js new file mode 100644 index 0000000000..053c216d6c --- /dev/null +++ b/test/built-ins/object/values/primitive-symbols.js @@ -0,0 +1,13 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: Object.values accepts Symbol primitives. +author: Jordan Harband +features: [Symbol] +---*/ + +var result = Object.values(Symbol()); + +assert.sameValue(Array.isArray(result), true, 'result is an array'); +assert.sameValue(result.length, 0, 'result has 0 items'); diff --git a/test/built-ins/object/values/symbols-omitted.js b/test/built-ins/object/values/symbols-omitted.js new file mode 100644 index 0000000000..50f5294592 --- /dev/null +++ b/test/built-ins/object/values/symbols-omitted.js @@ -0,0 +1,24 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: Object.values does not include Symbol keys. +author: Jordan Harband +features: [Symbol] +---*/ + +var value = {}; +var enumSym = Symbol('enum'); +var nonEnumSym = Symbol('nonenum'); +var symValue = Symbol('value'); + +var obj = { key: symValue }; +obj[enumSym] = value; +Object.defineProperty(obj, nonEnumSym, { enumerable: false, value: value }); + +var result = Object.values(obj); + +assert.sameValue(Array.isArray(result), true, 'result is an array'); +assert.sameValue(result.length, 1, 'result has 1 item'); + +assert.sameValue(result[0][1], symValue, 'first value is `symValue`'); diff --git a/test/built-ins/object/values/tamper-with-global-object.js b/test/built-ins/object/values/tamper-with-global-object.js new file mode 100644 index 0000000000..8c7644bbea --- /dev/null +++ b/test/built-ins/object/values/tamper-with-global-object.js @@ -0,0 +1,18 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: > + Object.values should not have its behavior impacted by modifications to the global property Object +author: Jordan Harband +---*/ + +function fakeObject() { + $ERROR('The overriden version of Object was called!'); +} + +var global = Function('return this;')(); +global.Object = fakeObject; + +assert.sameValue(Object, fakeObject, 'Sanity check failed: could not modify the global Object'); +assert.sameValue(Object.values(1).length, 0, 'Expected number primitive to have zero values'); diff --git a/test/built-ins/object/values/tamper-with-object-keys.js b/test/built-ins/object/values/tamper-with-object-keys.js new file mode 100644 index 0000000000..12b9bcf81d --- /dev/null +++ b/test/built-ins/object/values/tamper-with-object-keys.js @@ -0,0 +1,17 @@ +// Copyright (C) 2015 Jordan Harband. All rights reserved. +// This code is governed by the BSD license found in the LICENSE file. + +/*--- +description: > + Object.values should not have its behavior impacted by modifications to Object.keys +author: Jordan Harband +---*/ + +function fakeObjectKeys() { + $ERROR('The overriden version of Object.keys was called!'); +} + +Object.keys = fakeObjectKeys; + +assert.sameValue(Object.keys, fakeObjectKeys, 'Sanity check failed: could not modify the global Object.keys'); +assert.sameValue(Object.values({ a: 1 }).length, 1, 'Expected object with 1 key to have 1 value');