Closed Bug 858772 Opened 12 years ago Closed 6 years ago

Intermittent dom/apps/tests/test_app_update.html | Test timed out.

Categories

(Core Graveyard :: DOM: Apps, defect, P3)

Product:
Core Graveyard
Component:
DOM: Apps
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: RyanVM, Assigned: marco)

References

Details

(Keywords: intermittent-failure, Whiteboard: [apps-automation:P1])

Attachments

(2 files)

Fail if the checkForUpdate request fails instead of timing out
(deleted), patch
fabrice
: review+
Details | Diff | Splinter Review
logs.txt
(deleted), text/plain
Details 
https://tbpl.mozilla.org/php/getParsedLog.php?id=21478871&tree=Mozilla-Inbound

Android Armv6 Tegra 250 mozilla-inbound opt test mochitest-3 on 2013-04-05 03:10:37 PDT for push 8262a337d5be
slave: tegra-173

17277 INFO TEST-START | /tests/dom/apps/tests/test_app_update.html
17278 INFO TEST-PASS | /tests/dom/apps/tests/test_app_update.html | setVersion OK
17279 INFO TEST-PASS | /tests/dom/apps/tests/test_app_update.html | Checking uninstalled app
17280 INFO TEST-PASS | /tests/dom/apps/tests/test_app_update.html | Message from app: OK: Launched app
17281 INFO TEST-PASS | /tests/dom/apps/tests/test_app_update.html | App is not installed
17282 INFO TEST-PASS | /tests/dom/apps/tests/test_app_update.html | Version should be correct
17283 INFO TEST-PASS | /tests/dom/apps/tests/test_app_update.html | Messaging from app complete
17284 INFO TEST-PASS | /tests/dom/apps/tests/test_app_update.html | setVersion OK
17285 INFO TEST-PASS | /tests/dom/apps/tests/test_app_update.html | App is non-null
17286 INFO TEST-PASS | /tests/dom/apps/tests/test_app_update.html | Checking installed app
17287 INFO TEST-PASS | /tests/dom/apps/tests/test_app_update.html | Message from app: OK: Launched app
17288 INFO TEST-PASS | /tests/dom/apps/tests/test_app_update.html | App is installed
17289 ERROR TEST-UNEXPECTED-FAIL | /tests/dom/apps/tests/test_app_update.html | Test timed out.
17290 INFO TEST-END | /tests/dom/apps/tests/test_app_update.html | finished in 324123ms
Whiteboard: [apps-automation]
Whiteboard: [apps-automation] → [apps-automation:P1]
Is this Android only?
(In reply to Marco Castelluccio [:marco] from comment #53)
> Is this Android only?

Yes, it appears so.
This is the point where the test stops, let's see if the test times out because the checkForUpdate request fails or because checkForUpdate never finishes.
Attachment #8406286 - Flags: review?(fabrice)
Comment on attachment 8406286 [details] [diff] [review]
Fail if the checkForUpdate request fails instead of timing out

Review of attachment 8406286 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/apps/tests/test_app_update.html
@@ +130,1 @@
>      yield undefined;

do we really need 2 yields?
Attachment #8406286 - Flags: review?(fabrice) → review+
(In reply to Fabrice Desré [:fabrice] from comment #105)
> Comment on attachment 8406286 [details] [diff] [review]
> Fail if the checkForUpdate request fails instead of timing out
> 
> Review of attachment 8406286 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: dom/apps/tests/test_app_update.html
> @@ +130,1 @@
> >      yield undefined;
> 
> do we really need 2 yields?

Yeah, because we're waiting for both app.ondownloadapplied and request.onsuccess.
Keywords: checkin-needed
Whiteboard: [apps-automation:P1] → [apps-automation:P1][leave-open]
Attached file logs.txt (deleted) — 
Comparing the log of a local build with the failure, I've noticed
there's a difference in the stuff the SJS dumps to the console.
Looks like bug 915879 helped to reduce the frequency of failures.
Comment 187 to comment 234 are mis-stars of real bustage caused by bug 1029352, which I've just backed out...
Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
Product: Core → Core Graveyard
I think it is now fixed.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Whiteboard: [apps-automation:P1][leave-open] → [apps-automation:P1]
Assignee: nobody → mcastelluccio
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: