[Merged by Bors] - Fix some Date tests

[jedel1043]

Found some tests on the Date builtin that were failing for incorrect length attributes and missing checks.

[github-actions]

Test262 conformance changes

Test result	main count	PR count	difference
Total	93,815	93,815	0
Passed	69,540	69,562	+22
Ignored	18,422	18,422	0
Failed	5,853	5,831	-22
Panics	0	0	0
Conformance	74.12%	74.15%	+0.02%

Fixed tests (22):

test/intl402/Date/prototype/toLocaleTimeString/length.js [strict mode] (previously Failed)
test/intl402/Date/prototype/toLocaleTimeString/length.js (previously Failed)
test/intl402/Date/prototype/toLocaleDateString/length.js [strict mode] (previously Failed)
test/intl402/Date/prototype/toLocaleDateString/length.js (previously Failed)
test/intl402/Date/prototype/toLocaleString/length.js [strict mode] (previously Failed)
test/intl402/Date/prototype/toLocaleString/length.js (previously Failed)
test/built-ins/Date/prototype/setTime/this-value-valid-date.js [strict mode] (previously Failed)
test/built-ins/Date/prototype/setTime/this-value-valid-date.js (previously Failed)
test/built-ins/Date/prototype/setTime/this-value-invalid-date.js [strict mode] (previously Failed)
test/built-ins/Date/prototype/setTime/this-value-invalid-date.js (previously Failed)
test/built-ins/Date/prototype/setTime/new-value-time-clip.js [strict mode] (previously Failed)
test/built-ins/Date/prototype/setTime/new-value-time-clip.js (previously Failed)
test/built-ins/Date/prototype/setTime/arg-to-number.js [strict mode] (previously Failed)
test/built-ins/Date/prototype/setTime/arg-to-number.js (previously Failed)
test/built-ins/Date/prototype/toLocaleTimeString/S15.9.5.7_A2_T1.js [strict mode] (previously Failed)
test/built-ins/Date/prototype/toLocaleTimeString/S15.9.5.7_A2_T1.js (previously Failed)
test/built-ins/Date/prototype/toUTCString/invalid-date.js [strict mode] (previously Failed)
test/built-ins/Date/prototype/toUTCString/invalid-date.js (previously Failed)
test/built-ins/Date/prototype/toLocaleDateString/S15.9.5.6_A2_T1.js [strict mode] (previously Failed)
test/built-ins/Date/prototype/toLocaleDateString/S15.9.5.6_A2_T1.js (previously Failed)
test/built-ins/Date/prototype/toLocaleString/S15.9.5.5_A2_T1.js [strict mode] (previously Failed)
test/built-ins/Date/prototype/toLocaleString/S15.9.5.5_A2_T1.js (previously Failed)

[codecov]

Codecov Report

Merging #2431 (fa700c7) into main (2362f73) will decrease coverage by 0.00%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main    #2431      +/-   ##
==========================================
- Coverage   38.86%   38.85%   -0.01%     
==========================================
  Files         316      316              
  Lines       24131    24123       -8     
==========================================
- Hits         9378     9374       -4     
+ Misses      14753    14749       -4     

Impacted Files	Coverage Δ
boa_engine/src/value/mod.rs	50.36% <0.00%> (-0.17%)	⬇️
boa_engine/src/builtins/date/mod.rs	86.39% <66.66%> (+0.12%)	⬆️
boa_engine/src/value/integer.rs	69.23% <80.00%> (+6.73%)	⬆️
boa_engine/src/builtins/error/range.rs	100.00% <0.00%> (ø)
boa_engine/src/context/mod.rs	35.00% <0.00%> (+0.21%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[Razican]

Looks good :) thank you! We should probably add more inline comments with the statements from the spec, but they were not there before most of the times, so we can merge it as-is.

I have marked one of the functions, but there are more.

[HalidOdat]

Looks perfect to me :)

[HalidOdat]

bors r+

[bors]

Pull request successfully merged into main.

Build succeeded:

• Build and Test (macos-latest)
• Build and Test (windows-latest)
• Coverage
• Misc