Closed Bug 1197989 Opened 9 years ago Closed 9 years ago

Intermittent apps/clock/test/marionette/hour_format_test.js | Hour format 24 hour format Fire an alarm

Categories

(Firefox OS Graveyard :: Gaia::Clock, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(b2g-master disabled)

RESOLVED FIXED
2.6 S1 - 11/20
Tracking Status
b2g-master --- disabled

People

(Reporter: RyanVM, Assigned: mcav)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

No description provided.
Marcus, this seems to have gone off the deep-end recently. Can you please take a look ASAP before I disable the test? Given the frequency, I'm going to disable today otherwise.
Flags: needinfo?(m)
I don't have an immediate solution, especially since this is a harness failure. Disable it. Aus, any idea what would cause a test like this to crash, without leaving a stack trace, when the test hasn't been changed in a long time?
Flags: needinfo?(m) → needinfo?(aus)
Flags: needinfo?(aus)
Not seen in a long time.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WORKSFORME
... because it was blacklisted in tbpl_manifest.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Assignee: nobody → m
Target Milestone: --- → 2.6 S1 - 11/20
Comment on attachment 8685160 [details] [review] [gaia] mcav:hour-format-test > mozilla-b2g:master Remember when you said that Clock's "onInterrupt" handler was throwing stacktraces?[1] Reenabling this test involved fixing that. Since calling the interrupt handler would throw an exception, (1) the test would racily fail, and (2) we never actually called the interrupt handler in real life anyway. Looking at the docs for "mozinterruptbegin", I don't think we need it anyway (and empirically, we don't, since it never worked). So the solution here isn't to fix the handler; it's to just get rid of it. I've tested locally and the test seems fine now. [1] https://bugzilla.mozilla.org/show_bug.cgi?id=1207044#c21
Attachment #8685160 - Flags: review?(gandalf)
Comment on attachment 8685160 [details] [review] [gaia] mcav:hour-format-test > mozilla-b2g:master You got me with the amount of red in the diff :)
Attachment #8685160 - Flags: review?(gandalf) → review+
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
Whiteboard: [test disabled][leave open]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: