From 95331ea86475d2d2131ef42e8bf9f07803425676 Mon Sep 17 00:00:00 2001 From: Linus Groh Date: Sat, 31 Jul 2021 13:54:08 +0100 Subject: [PATCH] LibJS/Tests: Fix Temporal.Now.plainDateTime{,ISO}() epoch calculation Combining month and day like this doesn't always yield correct results. Use dayOfYear multiplied by the seconds per day instead, which does. --- .../LibJS/Tests/builtins/Temporal/Now/Now.plainDateTime.js | 3 +-- .../LibJS/Tests/builtins/Temporal/Now/Now.plainDateTimeISO.js | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/Userland/Libraries/LibJS/Tests/builtins/Temporal/Now/Now.plainDateTime.js b/Userland/Libraries/LibJS/Tests/builtins/Temporal/Now/Now.plainDateTime.js index 0ff22d80ff..7639283295 100644 --- a/Userland/Libraries/LibJS/Tests/builtins/Temporal/Now/Now.plainDateTime.js +++ b/Userland/Libraries/LibJS/Tests/builtins/Temporal/Now/Now.plainDateTime.js @@ -20,8 +20,7 @@ describe("correct behavior", () => { const plainDateTimeToEpochSeconds = plainDateTime => (plainDateTime.year - 1970) * 31_556_952 + - plainDateTime.month * 2_630_000 + - plainDateTime.day * 86_400 + + plainDateTime.dayOfYear * 86_400 + plainDateTime.hour * 3_600 + plainDateTime.minute * 60 + plainDateTime.second + diff --git a/Userland/Libraries/LibJS/Tests/builtins/Temporal/Now/Now.plainDateTimeISO.js b/Userland/Libraries/LibJS/Tests/builtins/Temporal/Now/Now.plainDateTimeISO.js index 9fd61584bb..63eab85628 100644 --- a/Userland/Libraries/LibJS/Tests/builtins/Temporal/Now/Now.plainDateTimeISO.js +++ b/Userland/Libraries/LibJS/Tests/builtins/Temporal/Now/Now.plainDateTimeISO.js @@ -18,8 +18,7 @@ describe("correct behavior", () => { const plainDateTimeToEpochSeconds = plainDateTime => (plainDateTime.year - 1970) * 31_556_952 + - plainDateTime.month * 2_630_000 + - plainDateTime.day * 86_400 + + plainDateTime.dayOfYear * 86_400 + plainDateTime.hour * 3_600 + plainDateTime.minute * 60 + plainDateTime.second +