Change Temporal.TimeZone.prototype.toJSON not to call toString

Normative PR: tc39/proposal-temporal#2482
tc39 · Feb 18, 2023 · 87ed3b6 · 87ed3b6
1 parent adf2e1c
commit 87ed3b6
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 45 deletions.
diff --git a/...imeZone/prototype/toJSON/tostring-call.js → ...ototype/toJSON/returns-identifier-slot.js b/...imeZone/prototype/toJSON/tostring-call.js → ...ototype/toJSON/returns-identifier-slot.js
@@ -3,25 +3,21 @@
 
 /*---
 esid: sec-temporal.timezone.prototype.tojson
-description: toJSON() calls toString() and returns its value
+description: toJSON() returns the internal slot value
 includes: [compareArray.js, temporalHelpers.js]
 features: [Temporal]
 ---*/
 
 const actual = [];
-const expected = [
- 'get [Symbol.toPrimitive]',
- 'get toString',
- 'call timeZone.toString',
-];
 
 const timeZone = new Temporal.TimeZone("UTC");
 TemporalHelpers.observeProperty(actual, timeZone, Symbol.toPrimitive, undefined);
+TemporalHelpers.observeProperty(actual, timeZone, "id", "Etc/Bogus");
 TemporalHelpers.observeProperty(actual, timeZone, "toString", function () {
  actual.push("call timeZone.toString");
  return "Etc/TAI";
 });
 
 const result = timeZone.toJSON();
-assert.sameValue(result, 'Etc/TAI', 'toString');
-assert.compareArray(actual, expected);
+assert.sameValue(result, "UTC", "toJSON gets the internal slot value");
+assert.compareArray(actual, [], "should not invoke any observable operations");
diff --git a/test/built-ins/Temporal/TimeZone/prototype/toJSON/tostring-undefined-custom.js b/test/built-ins/Temporal/TimeZone/prototype/toJSON/tostring-undefined-custom.js
diff --git a/test/built-ins/Temporal/TimeZone/prototype/toJSON/tostring-undefined.js b/test/built-ins/Temporal/TimeZone/prototype/toJSON/tostring-undefined.js