LibJS+LibC: Add tests for Date tuple ctor overflow and make mktime()/timegm() handle month overflow

2 files changed, 20 insertions, 2 deletions

diff --git a/Libraries/LibJS/Runtime/DateConstructor.cpp b/Libraries/LibJS/Runtime/DateConstructor.cpp
index ec91f36897..583230c1b1 100644
--- a/Libraries/LibJS/Runtime/DateConstructor.cpp
+++ b/Libraries/LibJS/Runtime/DateConstructor.cpp
@@ -179,7 +179,6 @@ Value DateConstructor::construct(Interpreter& interpreter, Function&)
             value = parse_simplified_iso8601(value.as_string().string());
         // A timestamp since the epoch, in UTC.
         // FIXME: Date() probably should use a double as internal representation, so that NaN arguments and larger offsets are handled correctly.
-        // FIXME: DateTime::from_timestamp() seems to not support negative offsets.
         double value_as_double = value.to_double(interpreter);
         auto datetime = Core::DateTime::from_timestamp(static_cast<time_t>(value_as_double / 1000));
         auto milliseconds = static_cast<u16>(fmod(value_as_double, 1000));
@@ -187,7 +186,6 @@ Value DateConstructor::construct(Interpreter& interpreter, Function&)
     }
     // A date/time in components, in local time.
     // FIXME: This doesn't construct an "Invalid Date" object if one of the parameters is NaN.
-    // FIXME: This doesn't range-check args and convert months > 12 to year increments etc.
     auto arg_or = [&interpreter](size_t i, i32 fallback) { return interpreter.argument_count() > i ? interpreter.argument(i).to_i32(interpreter) : fallback; };
     int year = interpreter.argument(0).to_i32(interpreter);
     int month_index = interpreter.argument(1).to_i32(interpreter);
diff --git a/Libraries/LibJS/Tests/builtins/Date/Date.UTC.js b/Libraries/LibJS/Tests/builtins/Date/Date.UTC.js
index 8a5c0e09dc..f1194dd495 100644
--- a/Libraries/LibJS/Tests/builtins/Date/Date.UTC.js
+++ b/Libraries/LibJS/Tests/builtins/Date/Date.UTC.js
@@ -25,3 +25,23 @@ test("basic functionality", () => {
 
     expect(Date.UTC(20000, 0)).toBe(568971820800000);
 });
+
+test("out of range", () => {
+    expect(Date.UTC(2020, -20)).toBe(1525132800000);
+    expect(Date.UTC(2020, 20)).toBe(1630454400000);
+
+    expect(Date.UTC(2020, 1, -10)).toBe(1579564800000);
+    expect(Date.UTC(2020, 1, 40)).toBe(1583884800000);
+
+    expect(Date.UTC(2020, 1, 15, -50)).toBe(1581544800000);
+    expect(Date.UTC(2020, 1, 15, 50)).toBe(1581904800000);
+
+    expect(Date.UTC(2020, 1, 15, 12, -123)).toBe(1581760620000);
+    expect(Date.UTC(2020, 1, 15, 12, 123)).toBe(1581775380000);
+
+    expect(Date.UTC(2020, 1, 15, 12, 30, -123)).toBe(1581769677000);
+    expect(Date.UTC(2020, 1, 15, 12, 30, 123)).toBe(1581769923000);
+
+    expect(Date.UTC(2020, 1, 15, 12, 30, 30, -2345)).toBe(1581769827655);
+    expect(Date.UTC(2020, 1, 15, 12, 30, 30, 2345)).toBe(1581769832345);
+});