From 741b799286409d51be430a09a5ab8c6d783c4cce Mon Sep 17 00:00:00 2001
From: Mike Pennisi <mike@mikepennisi.com>
Date: Wed, 15 Jul 2015 16:41:40 -0400
Subject: [PATCH] Add tests for String.prototype[Symbol.iterator]

---
 .../prototype/Symbol.iterator/length.js       | 26 +++++++++++++++++
 .../String/prototype/Symbol.iterator/name.js  | 29 +++++++++++++++++++
 .../prototype/Symbol.iterator/prop-desc.js    | 19 ++++++++++++
 .../this-val-non-obj-coercible.js             | 19 ++++++++++++
 .../Symbol.iterator/this-val-to-str-err.js    | 21 ++++++++++++++
 5 files changed, 114 insertions(+)
 create mode 100644 test/built-ins/String/prototype/Symbol.iterator/length.js
 create mode 100644 test/built-ins/String/prototype/Symbol.iterator/name.js
 create mode 100644 test/built-ins/String/prototype/Symbol.iterator/prop-desc.js
 create mode 100644 test/built-ins/String/prototype/Symbol.iterator/this-val-non-obj-coercible.js
 create mode 100644 test/built-ins/String/prototype/Symbol.iterator/this-val-to-str-err.js

diff --git a/test/built-ins/String/prototype/Symbol.iterator/length.js b/test/built-ins/String/prototype/Symbol.iterator/length.js
new file mode 100644
index 0000000000..9ae1c2f20e
--- /dev/null
+++ b/test/built-ins/String/prototype/Symbol.iterator/length.js
@@ -0,0 +1,26 @@
+// Copyright (C) 2015 the V8 project authors. All rights reserved.
+// This code is governed by the BSD license found in the LICENSE file.
+/*---
+es6id: 25.1.3.27
+description: Length of String.prototype[ @@iterator ]
+info: >
+    ES6 Section 17:
+    Every built-in Function object, including constructors, has a length
+    property whose value is an integer. Unless otherwise specified, this value
+    is equal to the largest number of named arguments shown in the subclause
+    headings for the function description, including optional parameters.
+
+    [...]
+
+    Unless otherwise specified, the length property of a built-in Function
+    object has the attributes { [[Writable]]: false, [[Enumerable]]: false,
+    [[Configurable]]: true }.
+features: [Symbol.iterator]
+includes: [propertyHelper.js]
+---*/
+
+assert.sameValue(String.prototype[Symbol.iterator].length, 0);
+
+verifyNotEnumerable(String.prototype[Symbol.iterator], 'length');
+verifyNotWritable(String.prototype[Symbol.iterator], 'length');
+verifyConfigurable(String.prototype[Symbol.iterator], 'length');
diff --git a/test/built-ins/String/prototype/Symbol.iterator/name.js b/test/built-ins/String/prototype/Symbol.iterator/name.js
new file mode 100644
index 0000000000..255c5269ed
--- /dev/null
+++ b/test/built-ins/String/prototype/Symbol.iterator/name.js
@@ -0,0 +1,29 @@
+// Copyright (C) 2015 the V8 project authors. All rights reserved.
+// This code is governed by the BSD license found in the LICENSE file.
+/*---
+es6id: 25.1.3.27
+description: Descriptor for `name` property
+info: >
+  The value of the name property of this function is "[Symbol.iterator]".
+
+  ES6 Section 17: ECMAScript Standard Built-in Objects
+
+  Every built-in Function object, including constructors, that is not
+  identified as an anonymous function has a name property whose value is a
+  String. Unless otherwise specified, this value is the name that is given to
+  the function in this specification.
+
+  [...]
+
+  Unless otherwise specified, the name property of a built-in Function
+  object, if it exists, has the attributes { [[Writable]]: false,
+  [[Enumerable]]: false, [[Configurable]]: true }.
+features: [Symbol.iterator]
+includes: [propertyHelper.js]
+---*/
+
+assert.sameValue(String.prototype[Symbol.iterator].name, '[Symbol.iterator]');
+
+verifyNotEnumerable(String.prototype[Symbol.iterator], 'name');
+verifyNotWritable(String.prototype[Symbol.iterator], 'name');
+verifyConfigurable(String.prototype[Symbol.iterator], 'name');
diff --git a/test/built-ins/String/prototype/Symbol.iterator/prop-desc.js b/test/built-ins/String/prototype/Symbol.iterator/prop-desc.js
new file mode 100644
index 0000000000..6352761e6f
--- /dev/null
+++ b/test/built-ins/String/prototype/Symbol.iterator/prop-desc.js
@@ -0,0 +1,19 @@
+// Copyright (C) 2015 the V8 project authors. All rights reserved.
+// This code is governed by the BSD license found in the LICENSE file.
+/*---
+es6id: 25.1.3.27
+description: Property descriptor
+info: >
+    ES6 Section 17
+
+    Every other data property described in clauses 18 through 26 and in Annex
+    B.2 has the attributes { [[Writable]]: true, [[Enumerable]]: false,
+    [[Configurable]]: true } unless otherwise specified.
+features: [Symbol.iterator]
+includes: [propertyHelper.js]
+---*/
+
+assert.sameValue(typeof String.prototype[Symbol.iterator], 'function');
+verifyNotEnumerable(String.prototype, Symbol.iterator);
+verifyWritable(String.prototype, Symbol.iterator);
+verifyConfigurable(String.prototype, Symbol.iterator);
diff --git a/test/built-ins/String/prototype/Symbol.iterator/this-val-non-obj-coercible.js b/test/built-ins/String/prototype/Symbol.iterator/this-val-non-obj-coercible.js
new file mode 100644
index 0000000000..17bd899b99
--- /dev/null
+++ b/test/built-ins/String/prototype/Symbol.iterator/this-val-non-obj-coercible.js
@@ -0,0 +1,19 @@
+// Copyright (C) 2015 the V8 project authors. All rights reserved.
+// This code is governed by the BSD license found in the LICENSE file.
+/*---
+es6id: 25.1.3.27
+description: The `this` value cannot be coerced into an object
+info: >
+    1. Let O be RequireObjectCoercible(this value).
+    2. Let S be ToString(O).
+    3. ReturnIfAbrupt(S).
+features: [Symbol.iterator]
+---*/
+
+assert.throws(TypeError, function() {
+  String.prototype[Symbol.iterator].call(undefined);
+});
+
+assert.throws(TypeError, function() {
+  String.prototype[Symbol.iterator].call(null);
+});
diff --git a/test/built-ins/String/prototype/Symbol.iterator/this-val-to-str-err.js b/test/built-ins/String/prototype/Symbol.iterator/this-val-to-str-err.js
new file mode 100644
index 0000000000..2d36190b38
--- /dev/null
+++ b/test/built-ins/String/prototype/Symbol.iterator/this-val-to-str-err.js
@@ -0,0 +1,21 @@
+// Copyright (C) 2015 the V8 project authors. All rights reserved.
+// This code is governed by the BSD license found in the LICENSE file.
+/*---
+es6id: 25.1.3.27
+description: Error thrown coercing `this` value to a string
+info: >
+    1. Let O be RequireObjectCoercible(this value).
+    2. Let S be ToString(O).
+    3. ReturnIfAbrupt(S).
+features: [Symbol.iterator]
+---*/
+
+var obj = {
+  toString: function() {
+    throw new Test262Error();
+  }
+};
+
+assert.throws(Test262Error, function() {
+  String.prototype[Symbol.iterator].call(obj);
+});
-- 
GitLab