Skip to content

Commit b3465be

Browse files
committed
[Temporal] Remove v8 staging test plain-date-time-with
The tests in `test/staging/Temporal/v8/plain-date-time-with.js` are covered by tests in - `test/built-ins/Temporal/PlainDateTime/prototype/with/branding.js` - `test/built-ins/Temporal/PlainDateTime/prototype/with/argument-not-object.js` --> add NaN - `test/built-ins/Temporal/PlainDateTime/prototype/with/infinity-throws-rangeerror.js` - `test/built-ins/Temporal/PlainDateTime/prototype/with/calendar-temporal-object-throws.js` --> add Now constructs as values - `test/built-ins/Temporal/PlainDateTime/prototype/with/options-wrong-type.js` - `test/built-ins/Temporal/PlainDateTime/prototype/with/calendar-throws.js` - `test/built-ins/Temporal/PlainDateTime/prototype/with/timezone-throws.js` - `test/built-ins/Temporal/PlainDateTime/prototype/with/basic.js`
1 parent 14ae1ee commit b3465be

File tree

3 files changed

+4
-56
lines changed

3 files changed

+4
-56
lines changed

test/built-ins/Temporal/PlainDateTime/prototype/with/argument-not-object.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ const args = [
1616
Symbol(),
1717
2020,
1818
2020n,
19+
NaN,
1920
];
2021
for (const argument of args) {
2122
assert.throws(TypeError, () => instance.with(argument), `Does not support ${typeof argument}`);

test/built-ins/Temporal/PlainDateTime/prototype/with/calendar-temporal-object-throws.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,9 @@ const values = [
1616
Temporal.PlainTime.from("15:19:45"),
1717
Temporal.PlainYearMonth.from("2022-04"),
1818
Temporal.ZonedDateTime.from("2022-04-12T15:19:45[UTC]"),
19+
Temporal.Now.plainDateTimeISO(),
20+
Temporal.Now.plainDateISO(),
21+
Temporal.Now.plainTimeISO(),
1922
];
2023

2124
for (const value of values) {

test/staging/Temporal/v8/plain-date-time-with.js

Lines changed: 0 additions & 56 deletions
This file was deleted.

0 commit comments

Comments
 (0)