Skip to content

Commit 4cc1780

Browse files
committed
Fix asserts in TimeZone transition tests
equal() was forgotten here, leading to these tests always passing.
1 parent 9a18c32 commit 4cc1780

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

polyfill/test/timezone.mjs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -177,8 +177,8 @@ describe('TimeZone', () => {
177177
it(`(${zone}).getPlainDateTimeFor(${inst})`, () =>
178178
assert(zone.getPlainDateTimeFor(inst) instanceof Temporal.PlainDateTime));
179179
it(`(${zone}).getInstantFor(${dtm})`, () => assert(zone.getInstantFor(dtm) instanceof Temporal.Instant));
180-
it(`(${zone}).getNextTransition(${inst})`, () => zone.getNextTransition(inst), null);
181-
it(`(${zone}).getPreviousTransition(${inst})`, () => zone.getPreviousTransition(inst), null);
180+
it(`(${zone}).getNextTransition(${inst})`, () => equal(zone.getNextTransition(inst), null));
181+
it(`(${zone}).getPreviousTransition(${inst})`, () => equal(zone.getPreviousTransition(inst), null));
182182
it('wraps around to the next day', () =>
183183
equal(`${zone.getPlainDateTimeFor(Temporal.Instant.from('2020-02-06T23:59Z'))}`, '2020-02-07T00:59:00'));
184184
});
@@ -192,8 +192,8 @@ describe('TimeZone', () => {
192192
it(`(${zone}).getPlainDateTimeFor(${inst})`, () =>
193193
assert(zone.getPlainDateTimeFor(inst) instanceof Temporal.PlainDateTime));
194194
it(`(${zone}).getInstantFor(${dtm})`, () => assert(zone.getInstantFor(dtm) instanceof Temporal.Instant));
195-
it(`(${zone}).getNextTransition(${inst})`, () => zone.getNextTransition(inst), null);
196-
it(`(${zone}).getPreviousTransition(${inst})`, () => zone.getPreviousTransition(inst), null);
195+
it(`(${zone}).getNextTransition(${inst})`, () => equal(zone.getNextTransition(inst), null));
196+
it(`(${zone}).getPreviousTransition(${inst})`, () => equal(zone.getPreviousTransition(inst), null));
197197
});
198198
describe('America/Los_Angeles', () => {
199199
const zone = new Temporal.TimeZone('America/Los_Angeles');
@@ -228,8 +228,8 @@ describe('TimeZone', () => {
228228
it(`(${zone}).getPlainDateTimeFor(${inst})`, () =>
229229
equal(`${zone.getPlainDateTimeFor(inst)}`, '1900-01-01T12:19:32'));
230230
it(`(${zone}).getInstantFor(${dtm})`, () => equal(`${zone.getInstantFor(dtm)}`, '1900-01-01T11:40:28Z'));
231-
it(`(${zone}).getNextTransition(${inst})`, () => zone.getNextTransition(inst), null);
232-
it(`(${zone}).getPreviousTransition(${inst})`, () => zone.getPreviousTransition(inst), null);
231+
it(`(${zone}).getNextTransition(${inst})`, () => equal(`${zone.getNextTransition(inst)}`, '1916-04-30T23:40:28Z'));
232+
it(`(${zone}).getPreviousTransition(${inst})`, () => equal(zone.getPreviousTransition(inst), null));
233233
});
234234
describe('with DST change', () => {
235235
it('clock moving forward', () => {

0 commit comments

Comments
 (0)