1524657 - TEST-UNEXPECTED-FAIL | Z:\task_1549018891\build\tests\mozmill\testAlarmDefaultValue.js | testAlarmDefaultValue.js::testDefaultAlarms

Status: RESOLVED FIXED

(Calendar :: General, defect)

Description

[Jorg K (CEST = GMT+2)]

Perma red since
https://treeherder.mozilla.org/#/jobs?repo=comm-central&revision=56f6fda8a27435070537edc3dcb27d79e82dca06

Log says:
INFO - SUMMARY-UNEXPECTED-FAIL | Z:\task_1549018891\build\tests\mozmill\testAlarmDefaultValue.js | testAlarmDefaultValue.js::testDefaultAlarms
INFO - EXCEPTION: Controller.assertDOMProperty(ID: item-alarm) : custom == none
INFO - at: controller.js line 1076
INFO - MozMillController.prototype.assertDOMProperty controller.js:1076 11
INFO - testDefaultAlarms/< testAlarmDefaultValue.js:63 9
INFO - invokeEventDialog test-calendar-utils.js:341 5
INFO - testDefaultAlarms testAlarmDefaultValue.js:59 5

Comment 1

[Geoff Lankow (:darktrojan)]

This test was disabled until that change, so I suspect it's something to do with the menulist bug, but why does it only happen on Windows?

Comment 2

[Geoff Lankow (:darktrojan)]

Jorg, can you run this test manually and watch what happens as the preferences tab opens? It should switch to the calendar pane, reminders tab, and change the two alarm length values to 43 days.

Comment 3

[Jorg K (CEST = GMT+2)]

Attachment: 1524657.png

mozmake -C comm/calendar/test/mozmill SOLO_TEST=testAlarmDefaultValue.js mozmill-one

Yes, it does all that. Switches the values to 43 and then creates an event. That the last thing I see.

Oops, I forgot to disable the test. Well, next time.

Comment 4

[Jorg K (CEST = GMT+2)]

Attachment: 1524657-disable-test.patch

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/20a933e91aed
disable testDefaultAlarms in testAlarmDefaultValue.js on Windows. rs=bustage-fix

Comment 6

[Jorg K (CEST = GMT+2)]

TB 66 beta:
https://hg.mozilla.org/releases/comm-beta/rev/77e45ce80f01ebb5d70a8dd292add67c00f613fa

I don't think we need the oranges on the beta tree.

Comment 7

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/comm-central/rev/8a2403d2797c
Backed out changeset 20a933e91aed to enable test fixed by bug 1526418 DONTBUILD

Comment 8

[Geoff Lankow (:darktrojan)]

This should also be backed out of beta once bug 1526418 is there. It appears to be the only test that fails if our preferences tab is not working right, but I'm going to change that.

Comment 9

[Jorg K (CEST = GMT+2)]

OK, I'll do that now.

Comment 10

[Jorg K (CEST = GMT+2)]

https://hg.mozilla.org/releases/comm-beta/rev/f0d8075173240f2ebf16e56679a03e6d31a95901