1089745 - Fix jstest failures in non-PST timezones

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jan de Mooij [:jandem]]

Attachment: Patch

Outside the Pacific time zone, the following jstests have been perma-fail since forever:

    ecma_3/Date/15.9.5.5.js
    ecma_3/Date/15.9.5.6.js
    ecma_3/Date/15.9.5.7.js
    js1_5/extensions/toLocaleFormat-01.js
    js1_5/Regress/regress-58116.js

This patch fixes them as follows:

(*) It adds a tz-pacific flag to the jsreftest harness. This is similar to the jit-test flag with the same name. Like the jit-test harness, it sets the TZ=PST8PDT environment variable.

(*) For the first two tests, I also had to change toLocaleDateString() to toLocaleDateString("en-US") to get a string that Date.parse can parse correctly.

I only changed tasks_unix.py. tasks_win.py seems to use worker threads so it's more complicated: updating os.environ could race with other threads.

Comment 1

[Terrence Cole [:terrence]]

Comment on attachment 8512098 [details] [diff] [review]
Patch

Review of attachment 8512098 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, but be sure to try-server first. We basically just write the test header verbatim into the jstests.list that reftests parse and I'm not sure how it will deal with random, unknown keywords. If it errors out on the new keyword, I'd also be fine just overriding TZ unconditionally.

Comment 2

[Jan de Mooij [:jandem]]

Attachment: Patch v2

You were right, the jsreftest harness wasn't happy. This patch just sets the environment variable in the main() function. A bit simpler and should hopefully also work on Windows/Cygwin this way.

Comment 3

[Jan de Mooij [:jandem]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/99072302a3f4

Comment 4

[Daniel Holbert [:dholbert]]

Any chance this might fix bug 610183?  If so, that would be great!

(I guess we'll find out this weekend.)

Comment 5

[Daniel Holbert [:dholbert]]

(and bug 934238)

Comment 6

[Jan de Mooij [:jandem]]

(In reply to Daniel Holbert [:dholbert] from comment #4)
> Any chance this might fix bug 610183?  If so, that would be great!
> 
> (I guess we'll find out this weekend.)

I replied on IRC, but for others following along: this patch only affects the shell harness and jsreftests run in the browser on tinderbox.

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/99072302a3f4

Comment 8

[Nicholas Nethercote [inactive]]

/me wonders if he can now safely set the timezone on his Linux desktop machine to something other than Pacific time...

Comment 9

[Terrence Cole [:terrence]]

(In reply to Nicholas Nethercote [:njn] from comment #8)
> /me wonders if he can now safely set the timezone on his Linux desktop
> machine to something other than Pacific time...

Were you able to? In particular, is bug 515254 still relevant?

Comment 10

[Nicholas Nethercote [inactive]]

> > /me wonders if he can now safely set the timezone on his Linux desktop
> > machine to something other than Pacific time...
> 
> Were you able to? In particular, is bug 515254 still relevant?

I tried and got no failures, so I closed bug 515254. Thanks for the reminder.