fixup: Add boolean/Symbol() tests for trim(Start/End)

Valerie R Young · Valerie R Young · commit 4587c8136929 · 2017-09-29T16:26:48.000-04:00
diff --git a/test/built-ins/String/prototype/trimEnd/this-value-boolean.js b/test/built-ins/String/prototype/trimEnd/this-value-boolean.js
@@ -2,20 +2,30 @@
 // This code is governed by the BSD license found in the LICENSE file.
 
 /*---
-esid: pending
-description: Behavoir when "this" value is a boolean.
+esid: sec-string.prototype.trimEnd
+description: Behavior when "this" value is a boolean.
+info: |
+  TrimString
+  2. Let S be ? ToString(str).
+
+  ToString
+  Argument Type: Boolean
+  Result:
+    If argument is true, return "true".
+    If argument is false, return "false".
+features: [string-trimming]
 ---*/
 
-var trimStart = String.prototype.trimStart
+var trimEnd = String.prototype.trimEnd
 
 assert.sameValue(
-    trimStart.call(true),
-    'true',
-    'String.prototype.trimStart.call(true)'
+  trimEnd.call(true),
+  'true',
+  'String.prototype.trimEnd.call(true)'
 );
 
 assert.sameValue(
-    String.prototype.trimStart.call(false),
-    'false',
-    'String.prototype.trimStart.call(false)'
+  String.prototype.trimEnd.call(false),
+  'false',
+  'String.prototype.trimEnd.call(false)'
 );
diff --git a/test/built-ins/String/prototype/trimEnd/this-value-symbol-typeerror.js b/test/built-ins/String/prototype/trimEnd/this-value-symbol-typeerror.js
@@ -2,7 +2,7 @@
 // This code is governed by the BSD license found in the LICENSE file.
 
 /*---
-esid: pending
+esid: sec-string.prototype.trimEnd
 description: Type error when "this" value is a Symbol
 info: |
   TrimString
@@ -11,11 +11,14 @@ info: |
   ToString
   Argument Type: Symbol
   Result: Throw a TypeError exception
+features: [string-trimming]
 ---*/
 
 var trimEnd = String.prototype.trimEnd;
-var symbol = Symbol()
+var symbol = Symbol();
 
-assert.throws(TypeError, function() {
-  trimEnd.call(symbol);
-}, 'Symbol()');
+assert.throws(
+  TypeError,
+  function() { trimEnd.call(symbol); },
+  'String.prototype.trimEnd.call(Symbol())'
+);
diff --git a/test/built-ins/String/prototype/trimStart/this-value-boolean.js b/test/built-ins/String/prototype/trimStart/this-value-boolean.js
@@ -2,20 +2,30 @@
 // This code is governed by the BSD license found in the LICENSE file.
 
 /*---
-esid: pending
-description: Behavoir when "this" value is a boolean.
+esid: sec-string.prototype.trimStart
+description: Behavior when "this" value is a boolean.
+info: |
+  TrimString
+  2. Let S be ? ToString(str).
+
+  ToString
+  Argument Type: Boolean
+  Result:
+    If argument is true, return "true".
+    If argument is false, return "false".
+features: [string-trimming]
 ---*/
 
 var trimStart = String.prototype.trimStart
 
 assert.sameValue(
-    trimStart.call(true),
-    'true',
-    'String.prototype.trimStart.call(true)'
+  trimStart.call(true),
+  'true',
+  'String.prototype.trimStart.call(true)'
 );
 
 assert.sameValue(
-    String.prototype.trimStart.call(false),
-    'false',
-    'String.prototype.trimStart.call(false)'
+  String.prototype.trimStart.call(false),
+  'false',
+  'String.prototype.trimStart.call(false)'
 );
diff --git a/test/built-ins/String/prototype/trimStart/this-value-symbol-typeerror.js b/test/built-ins/String/prototype/trimStart/this-value-symbol-typeerror.js
@@ -2,7 +2,7 @@
 // This code is governed by the BSD license found in the LICENSE file.
 
 /*---
-esid: pending
+esid: sec-string.prototype.trimStart
 description: Type error when "this" value is a Symbol
 info: |
   TrimString
@@ -11,11 +11,14 @@ info: |
   ToString
   Argument Type: Symbol
   Result: Throw a TypeError exception
+features: [string-trimming]
 ---*/
 
 var trimStart = String.prototype.trimStart;
-var symbol = Symbol()
+var symbol = Symbol();
 
-assert.throws(TypeError, function() {
-  trimStart.call(symbol);
-}, 'Symbol()');
+assert.throws(
+  TypeError,
+  function() { trimStart.call(symbol); },
+  'String.prototype.trimStart.call(Symbol())'
+);
