Split marDownload.js into individual tests

RESOLVED FIXED in mozilla28

Status

()

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: rstrong, Assigned: rstrong)

Tracking

unspecified
mozilla28
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

This is one of the longer running app update xpcshell tests and now that xpcshell tests are run in parallel by default I'm going to split the tests into separate tests.
This test takes as long as 20 seconds on my system while all the other tests take less than a second.
On Linux x64 Opt 

Slowest test is test_0030_general.js at 10588.650ms

Second slowest test is test_0010_timermanager.js at 6853.222ms which tests a bunch of timer notifications

Third slowest test is test_0113_general.js 3261.155ms which applies a complete mar with update staging
Status: NEW → ASSIGNED
OS: Windows 7 → All
Hardware: x86_64 → All
Summary: Split test_0030_general.js into individual tests → Split marDownload.js into individual tests
Attachment #823169 - Flags: review?(netzen)
Attachment #823169 - Flags: review?(netzen) → review+
Pushed to fx-team
https://hg.mozilla.org/integration/fx-team/rev/ba3ac8b3853c
Flags: in-testsuite+
Target Milestone: --- → mozilla28
Linux x64 Opt
Before:
downloadMar.js 10395.207ms

After:
downloadAndHashCheckMar.js 3731.356ms
downloadInterruptedRecovery.js 8034.864ms

Windows 7 Opt
Before:
downloadMar.js 22056.000ms

After:
downloadAndHashCheckMar.js 18174.000ms
downloadInterruptedRecovery.js 14478.000ms

Times could be further improved upon... perhaps if time permits
https://hg.mozilla.org/mozilla-central/rev/ba3ac8b3853c
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.