Closed Bug 947565 Opened 11 years ago Closed 11 years ago

Intermittent test_packaged_app_install.html | Test timed out.

Categories

(Core Graveyard :: DOM: Apps, defect)

Product:
Core Graveyard
Component:
DOM: Apps
Platform:
ARM
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox30 unaffected, firefox31 fixed, firefox32 fixed, firefox-esr24 unaffected, b2g-v1.2 wontfix, b2g-v1.3 wontfix, b2g-v1.4 unaffected, b2g-v2.0 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla32
Tracking Flags:
Tracking Status
firefox30 --- unaffected
firefox31 --- fixed
firefox32 --- fixed
firefox-esr24 --- unaffected
b2g-v1.2 --- wontfix
b2g-v1.3 --- wontfix
b2g-v1.4 --- unaffected
b2g-v2.0 --- fixed

People

(Reporter: philor, Assigned: marco)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file, 2 obsolete files)

Patch
1.83 KB, patch
marco
: review+
Details | Diff | Splinter Review
https://tbpl.mozilla.org/php/getParsedLog.php?id=31604435&tree=Mozilla-Central Android 2.2 Tegra mozilla-central opt test mochitest-4 on 2013-12-06 20:07:27 PST for push 42b2a2adda8f slave: tegra-035 38466 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | == TEST == No manifest in the zip file 38467 INFO TEST-PASS | /tests/dom/apps/tests/test_packaged_app_install.html | App installed Etags Match. Sending 304 Etags Match. Sending 304 Etags Match. Sending 304 38468 ERROR TEST-UNEXPECTED-FAIL | /tests/dom/apps/tests/test_packaged_app_install.html | Test timed out. 38469 INFO TEST-INFO | MEMORY STAT vsize after test: 629964800 38470 INFO TEST-INFO | MEMORY STAT residentFast after test: 161660928 38471 INFO TEST-INFO | MEMORY STAT heapAllocated after test: 37032044 38472 INFO TEST-END | /tests/dom/apps/tests/test_packaged_app_install.html | finished in 324484ms
Depends on: 972927
Attached patch Patch (obsolete) — — Splinter Review
With this patch, the test hasn't failed for several runs (13): https://tbpl.mozilla.org/?tree=Try&rev=8650a7467be2
Attachment #8412204 - Flags: review?(fabrice)
Comment on attachment 8412204 [details] [diff] [review] Patch Sorry, wrong bug.
Attachment #8412204 - Attachment is obsolete: true
Attachment #8412204 - Flags: review?(fabrice)
Looks like it is always faling during the "Mini-manifest app name is different from webapp manifest name" step. We're using a custom 30 seconds timeout, is it possible that it is not enough time?
Could be, the B2G emulator is notorious slow. One of reasons timeouts in tests are explicitly discouraged :)
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #172) > Could be, the B2G emulator is notorious slow. One of reasons timeouts in > tests are explicitly discouraged :) Indeed, I don't know why this timeout was added. Anyway, I've written a patch to remove it and sent it to try, let's see if it fixes the problem: https://tbpl.mozilla.org/?tree=Try&rev=cfcd8f8b3365
Attached patch Patch (obsolete) — — Splinter Review
Looks like the removal of the custom timeout fixes the intermittent failure. It should also fix the latest intermittent failure in bug 915879.
Attachment #8413619 - Flags: review?(fabrice)
Comment on attachment 8413619 [details] [diff] [review] Patch Review of attachment 8413619 [details] [diff] [review]: ----------------------------------------------------------------- so much green! wow!
Attachment #8413619 - Flags: review?(fabrice) → review+
Keywords: checkin-needed
Attached patch Patch — — Splinter Review
Rebased on inbound.
Attachment #8413619 - Attachment is obsolete: true
Attachment #8414385 - Flags: review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/a26c468310d5
Assignee: nobody → mar.castelluccio
Flags: in-testsuite+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/a26c468310d5
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
Looks like firefox30 is unaffected too, the custom timeout was introduced in bug 880043.
status-firefox30: wontfix → unaffected
Backed out bug 1036847 for the recent failures in this bug.
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: