Skip to content

Commit

Permalink
Update polyfill and tests
Browse files Browse the repository at this point in the history
  • Loading branch information
justingrant committed Jun 3, 2023
1 parent df93c0e commit 34650bc
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
4 changes: 2 additions & 2 deletions polyfill/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@ The full surface area of this proposal is expected to be covered by Test262 test
- [x] DONE Add lightweight "Demitasse" tests in [test/canonicaltz.mjs](./test/canonicaltz.mjs) that cover this proposal's full surface area
- [x] DONE Fix 15 existing Test262 tests that were broken by this proposal, because they assumed that time zone identifiers are always canonicalized
- [x] DONE Open a Test262 draft PR
- [ ] Migrate `Temporal.TimeZone.p.equals` (the only new API in this proposal) Demitasse tests to Test262
- [ ] Migrate `Temporal.TimeZone` Demitasse tests to Test262
- [x] Migrate `Temporal.TimeZone.p.equals` (the only new API in this proposal) Demitasse tests to Test262
- [x] Migrate `Temporal.TimeZone` Demitasse tests to Test262
- [ ] Migrate `Temporal.ZonedDateTime` Demitasse tests to Test262
- [ ] Migrate `Intl.DateTimeFormat` Demitasse tests to Test262
- [ ] Remove Demitasse tests from this repo and from CI workflows
Expand Down
10 changes: 5 additions & 5 deletions polyfill/polyfill.diff
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,10 @@ index 244067cb..d4b6f741 100644
getOffsetStringFor(instant: Temporal.Instant | string): string;
getPlainDateTimeFor(instant: Temporal.Instant | string, calendar?: CalendarLike): Temporal.PlainDateTime;
diff --git a/polyfill/lib/ecmascript.mjs b/polyfill/lib/ecmascript.mjs
index 6b2a318d..bf86bda9 100644
index adbb9d2e..47803e67 100644
--- a/polyfill/lib/ecmascript.mjs
+++ b/polyfill/lib/ecmascript.mjs
@@ -370,7 +370,7 @@ export function ParseTemporalTimeZone(stringIdent) {
@@ -371,7 +371,7 @@ export function ParseTemporalTimeZone(stringIdent) {
if (IsTimeZoneOffsetString(tzName)) return CanonicalizeTimeZoneOffsetString(tzName);
const record = GetAvailableNamedTimeZoneIdentifier(tzName);
if (!record) throw new RangeError(`Unrecognized time zone ${tzName}`);
Expand All @@ -33,7 +33,7 @@ index 6b2a318d..bf86bda9 100644
}
if (z) return 'UTC';
// if !tzName && !z then offset must be present
@@ -2118,7 +2118,16 @@ export function TimeZoneEquals(one, two) {
@@ -2119,7 +2119,16 @@ export function TimeZoneEquals(one, two) {
if (one === two) return true;
const tz1 = ToTemporalTimeZoneIdentifier(one);
const tz2 = ToTemporalTimeZoneIdentifier(two);
Expand Down Expand Up @@ -103,9 +103,9 @@ index 1a593c7f..06de9d5f 100644

const formatter = new DateTimeFormat(locales, optionsCopy);
diff --git a/polyfill/test262 b/polyfill/test262
index 3e858ef0..be8f7e90 160000
index 3e858ef0..7c1328f0 160000
--- a/polyfill/test262
+++ b/polyfill/test262
@@ -1 +1 @@
-Subproject commit 3e858ef02d2eda1e1e7eeff89ad7deeaf99d2766
+Subproject commit be8f7e90217da86f3949d8ece80dc5ae3d990165
+Subproject commit 7c1328f0b47779154d5fb46117036a5ffb3e4115
2 changes: 1 addition & 1 deletion temporal

0 comments on commit 34650bc

Please sign in to comment.