Closed Bug 947565 Opened 10 years ago Closed 10 years ago

Intermittent test_packaged_app_install.html | Test timed out.

Categories

(Core Graveyard :: DOM: Apps, defect)

ARM
Android
defect
Not set
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)

RESOLVED FIXED
mozilla32
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)

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 PatchSplinter 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: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
Looks like firefox30 is unaffected too, the custom timeout was introduced in bug 880043.
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.