Closed Bug 1261190 Opened 4 years ago Closed 4 years ago

test_bug22310.js fails on Windows 10 in automation

Categories

(Core :: Internationalization, defect)

Unspecified
Windows 10
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: RyanVM, Unassigned)

References

(Depends on 1 open bug)

Details

Windows 10 is available for opt-in on Try (albeit with a whopping one total
slave). I've been toying around with it to see where things stand and this
failure was one that popped up.

I really don't know who the right person is to investigate this, maybe smontagu?

https://treeherder.mozilla.org/logviewer.html#?job_id=18578946&repo=try

13:12:27     INFO -  TEST-START | intl/locale/tests/unit/test_bug22310.js
13:12:27  WARNING -  TEST-UNEXPECTED-FAIL | intl/locale/tests/unit/test_bug22310.js | xpcshell return code: 0
13:12:27     INFO -  TEST-INFO took 103ms
13:12:27     INFO -  >>>>>>>
13:12:27     INFO -  (xpcshell/head.js) | test MAIN run_test pending (1)
13:12:27     INFO -  PROCESS | 8312 | Unexpected date format: month name in no-NO
13:12:27  WARNING -  TEST-UNEXPECTED-FAIL | intl/locale/tests/unit/test_bug22310.js | run_test - [run_test : 51] false == true
13:12:27     INFO -  C:/slave/test/build/tests/xpcshell/tests/intl/locale/tests/unit/test_bug22310.js:run_test:51
13:12:27     INFO -  C:\slave\test\build\tests\xpcshell\head.js:_execute_test:528
13:12:27     INFO -  -e:null:1
13:12:27     INFO -  exiting test
13:12:27     INFO -  <<<<<<<
Flags: needinfo?(smontagu)
Not sure I can help much here -- I don't have a computer with Windows 10 on it. It would be easier if the test displayed the actual output of FormatDateTime when failing. There are apparently issues with date/time format in some locales on Windows 10 -- https://www.reddit.com/r/csharp/comments/3f5z5w/datetime_parsing_fails_in_windows_10/ but I can't tell if that is the issue here.
Flags: needinfo?(smontagu)
Allow me :)
https://treeherder.mozilla.org/logviewer.html#?job_id=18984166&repo=try

15:03:16     INFO -  TEST-START | intl/locale/tests/unit/test_bug22310.js
15:03:16  WARNING -  TEST-UNEXPECTED-FAIL | intl/locale/tests/unit/test_bug22310.js | xpcshell return code: 0
15:03:16     INFO -  TEST-INFO took 104ms
15:03:16     INFO -  >>>>>>>
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for en-US is: Monday, June 30, 2008 1:56:34 PM
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for en-US is: Monday, June 30, 2008 1:56:34 PM
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for da is: 30. juni 2008 13:56:34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for da-DK is: 30. juni 2008 13:56:34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for da is: 30. juni 2008 13:56:34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for en-GB is: 30 June 2008 13:56:34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for en-GB is: 30 June 2008 13:56:34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for en-GB is: 30 June 2008 13:56:34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for en-GB is: 30 June 2008 13:56:34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for en-US is: Monday, June 30, 2008 1:56:34 PM
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for en-US is: Monday, June 30, 2008 1:56:34 PM
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for en-US is: Monday, June 30, 2008 1:56:34 PM
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for en-US is: Monday, June 30, 2008 1:56:34 PM
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for ja-JP is: 2008年6月30日 13:56:34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for ja-JP is: 2008年6月30日 13:56:34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for ja-JP-mac is: 2008年6月30日 13:56:34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for nn-NO is: måndag 30. juni 2008 13:56:34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for nb-NO is: mandag 30. juni 2008 13.56.34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for no-NO is: 2008年6月30日 13:56:34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for sv-SE is: den 30 juni 2008 13:56:34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for kok is: 30 जून 2008 13:56:34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for ta-IN is: 30 ஜூன் 2008 13:56:34
15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for ab-CD is: 2008年6月30日 13:56:34
15:03:16     INFO -  (xpcshell/head.js) | test MAIN run_test pending (1)
15:03:16     INFO -  PROCESS | 3320 | Unexpected date format: month name in no-NO
15:03:16  WARNING -  TEST-UNEXPECTED-FAIL | intl/locale/tests/unit/test_bug22310.js | run_test - [run_test : 53] false == true
15:03:16     INFO -  C:/slave/test/build/tests/xpcshell/tests/intl/locale/tests/unit/test_bug22310.js:run_test:53
15:03:16     INFO -  C:\slave\test\build\tests\xpcshell\head.js:_execute_test:528
15:03:16     INFO -  -e:null:1
15:03:16     INFO -  exiting test
15:03:16     INFO -  <<<<<<<
Flags: needinfo?(smontagu)
(In reply to Ryan VanderMeulen [:RyanVM] from comment #2)
> 15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for nn-NO
> is: måndag 30. juni 2008 13:56:34
> 15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for nb-NO
> is: mandag 30. juni 2008 13.56.34
> 15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for no-NO
> is: 2008年6月30日 13:56:34
...
> 15:03:16     INFO -  PROCESS | 3320 | The FormatDateTime result for ab-CD
> is: 2008年6月30日 13:56:34

So what is happening is that the "no-NO" locale is no longer recognized and uses the fallback locale. It used to give the same result as one of the other Norwegian locales, IIANM "nb-NO". 

I suggest commenting out the "no-NO" line from the test for now, with a reference to bug 1261775.
Flags: needinfo?(smontagu)
Chatted briefly about it with Axel on IRC today:
Pike: RyanVM: surprising yes, but smontagu's comments make sense. 'no' is ambiguous, 'nn' or 'nb' are the real language codes
RyanVM: Pike: so basically win10 started caring more than older OSes
Pike: RyanVM: I am surprised that we use 'no-NO' in a test, though. That's sounds rather useless
I don't have a ton of experience hacking around in xpcshell tests, but I'll try to skip no_NO as minimally as possible.
Depends on: 1261775
https://hg.mozilla.org/mozilla-central/rev/94652a86b48c
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.