fergalmoran/ladybird
1
0
Fork 0
mirror of https://github.com/fergalmoran/ladybird.git synced 2026-03-20 04:17:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

LibJS: Disable Temporal custom time zone test

Browse Source 
This test has been flaky for quite some time. Disable it for now, and
revisit once we've caught up with the Temporal spec.

See also: b7676cc436
This commit is contained in:
Timothy Flynn
2023-10-18 15:07:06 -04:00
committed by Tim Flynn
parent 3c2d820391
commit f8763c16b2
1 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
Userland/Libraries/LibJS/Tests/builtins/Temporal/Now/Now.plainDateTime.js
View File

@@ -29,7 +29,10 @@ describe("correct behavior", () => {
// and negative timezones and skip one if we jump the year. To ensure at least one is
// tested we have the timeZoneTested which is only set to true if one of the tests passed.
test("custom time zone positive", () => {
// FIXME: The custom time zone tests are disabled due to being flaky. See:
// https://github.com/SerenityOS/serenity/issues/20806
test.skip("custom time zone positive", () => {
const calendar = new Temporal.Calendar("iso8601");
const timeZone = {
getOffsetNanosecondsFor() {
@@ -53,7 +56,7 @@ describe("correct behavior", () => {
timeZoneTested = true;
});
test("custom time zone negative", () => {
test.skip("custom time zone negative", () => {
const calendar = new Temporal.Calendar("iso8601");
const timeZone = {
getOffsetNanosecondsFor() {
@@ -77,7 +80,9 @@ describe("correct behavior", () => {
timeZoneTested = true;
});
expect(timeZoneTested).toBeTrue();
test.skip("custom time zone test was executed", () => {
expect(timeZoneTested).toBeTrue();
});
test("cannot have a time zone with more than a day", () => {
[86400000000000, -86400000000000, 86400000000001, 86400000000002].forEach(offset => {