Summary: | REGRESSION: LayoutTest js/intl-datetimeformat.html is failing | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryan Haddad <ryanhaddad> | ||||
Component: | JavaScriptCore | Assignee: | Andy VanWagoner <andy> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | andy, ap, cdumez, commit-queue, darin, fpizlo, ggaren, keith_miller, mark.lam, saam, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Ryan Haddad
2017-05-30 08:41:11 PDT
Marked test as failing / skipped for JSC in http://trac.webkit.org/projects/webkit/changeset/217560 Is this a regression? The difference is "Greenwich Mean Time" vs. "GMT". (In reply to Alexey Proskuryakov from comment #3) > Is this a regression? > > The difference is "Greenwich Mean Time" vs. "GMT". It started after the Sierra bots were updated to 10.12.5 last Friday. It looks to me like it's actually more correct now than it was before, since timeZoneName:'long' means written out, but the expectation needs to be updated. This is likely due to updated ICU data. How should we handle situations like this where some bots have different locale data? Is there a way to branch inside the test? I could update the test to use array.includes to check if the result is one of an acceptable list. Supporting multiple successful results seems best. We can have per platform results (including different macOS versions), but that's harder to maintain. Created attachment 311675 [details]
Patch
ping review? Comment on attachment 311675 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=311675&action=review > LayoutTests/js/intl-datetimeformat-expected.txt:380 > +PASS ['00:00 GMT','00:00 Horário do Meridiano de Greenwich'].includes(Intl.DateTimeFormat('pt-BR', { minute:'2-digit', hour:'numeric', timeZoneName:'long', timeZone: 'UTC' }).format(0)) is true Why these two specific strings? What about all the other languages? Comment on attachment 311675 [details] Patch Rejecting attachment 311675 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-01', 'apply-attachment', '--no-update', '--non-interactive', 311675, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: ngeLog Hunk #1 succeeded at 1 with fuzz 3. patching file LayoutTests/js/intl-datetimeformat-expected.txt patching file LayoutTests/js/script-tests/intl-datetimeformat.js patching file LayoutTests/platform/mac/TestExpectations Hunk #1 FAILED at 1541. 1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/mac/TestExpectations.rej Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Darin Adler']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.webkit.org/results/3882769 Comment on attachment 311675 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=311675&action=review >> LayoutTests/js/intl-datetimeformat-expected.txt:380 >> +PASS ['00:00 GMT','00:00 Horário do Meridiano de Greenwich'].includes(Intl.DateTimeFormat('pt-BR', { minute:'2-digit', hour:'numeric', timeZoneName:'long', timeZone: 'UTC' }).format(0)) is true > > Why these two specific strings? What about all the other languages? This specific test was for Brazilian Portuguese. Depending on the mac version (or rather ICU data) it may have a long name for the timezone, or may fallback on the short abbreviation. Committed r218025: <http://trac.webkit.org/changeset/218025> |