Closed
Bug 1070615
Opened 10 years ago
Closed 10 years ago
Make changes to Gaia performance tests to support running home screen and system tests
Categories
(Firefox OS Graveyard :: Gaia::PerformanceTest, defect, P1)
Tracking
(tracking-b2g:backlog)
People
(Reporter: Eli, Assigned: Eli)
References
Details
(Whiteboard: [systemsfe])
Attachments
(1 file)
The mechanisms behind Gaia performance tests, `make test-perf`, currently support running tests against individual applications that have clearly defined launch lifecycles, but don't have a well-defined approach for testing against other types of apps like the home screen and system. We need to make changes to `make test-perf` to support running these tests.
Comment 1•10 years ago
|
||
Also activities. See bug 1007421.
Comment 2•10 years ago
|
||
... and keyboard apps, but I don't know if the bug already exists.
Comment 3•10 years ago
|
||
Maybe this should block bug 940164, would be nice to have some bare minimum perf indicators before doing the next wave of refactorings
Updated•10 years ago
|
blocking-b2g: --- → backlog
Priority: -- → P1
Updated•10 years ago
|
Target Milestone: 2.1 S5 (26sep) → 2.1 S6 (10oct)
Assignee | ||
Comment 4•10 years ago
|
||
This patch enables tests to be run against the system app. This introductory patch is meant only for doing manual tests using `make test-perf` and as such, will not report any results to Datazilla. There are a few markers introduced into the System and Vertical Home apps, and is captured by storing a value on an object rather than events as with other performance events, as using events was very buggy with the reboot process.
Usage is:
APP=system TEST_BOOT=2 make test-perf
The JSON reported back currently contains 3 measurements:
mozSystemLoadEnd - When the System app finishes its execution of window.onload
mozOsLogoEnd - When the System app removes the boot logo
mozHomescreenStart - When the Homescreen has first JS initialization
I have reserved TEST_BOOT=1 if we decide that this test should start reporting the Datazilla in the future. A Datazilla-compatible implementation is a larger time effort, and this patch is merely to enable one-off measurements of the System app launch from boot. Since the Flame's timing implementation can be buggy at boot, these timestamps are measured from performance.requestStart. For a closer measurement of how long the boot process took, you can look at the test start, end, and duration, e.g.:
"start": "2014-10-09T18:26:55.874Z",
"end": "2014-10-09T18:27:23.413Z",
"duration": 27539
but this is not a precise measurement of boot, just a close estimate.
Attachment #8502676 -
Flags: review?(kgrandon)
Attachment #8502676 -
Flags: review?(jlal)
Attachment #8502676 -
Flags: feedback?(mhenretty)
Assignee | ||
Comment 5•10 years ago
|
||
Another note, the Datazilla-compatible implementation would include taking advantage of the RUNS variable, so this test will only run once.
Comment 6•10 years ago
|
||
Comment on attachment 8502676 [details]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25007
Not super-happy about the shim, but I think this is fine for now. We should probably plan on making that shim thing real at some point :) Thanks for the patch!
Attachment #8502676 -
Flags: review?(kgrandon) → review+
Assignee | ||
Comment 7•10 years ago
|
||
Yeah, I'm not happy about it either, and I've got some things in the works with Web API to change it. :)
Comment 8•10 years ago
|
||
Comment on attachment 8502676 [details]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25007
I'm not the expert here, but this seems to give us what we want, so fb+ for that!
Attachment #8502676 -
Flags: feedback?(mhenretty) → feedback+
Assignee | ||
Comment 9•10 years ago
|
||
Comment on attachment 8502676 [details]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/25007
James gave a verbal OK on this in IRC.
Attachment #8502676 -
Flags: review?(jlal)
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 10•10 years ago
|
||
This causes the unit tests to fail, probably because window.mozPerformance needs to be shimmed or something. Please fix and make sure we have a green run before landing.
Flags: needinfo?(eperelman)
Keywords: checkin-needed
Assignee | ||
Comment 11•10 years ago
|
||
Hmm, weird, window.mozPerformance IS shimmed in shared/js/performance_testing_helper.js. I'll look into why the unit test is failing.
Flags: needinfo?(eperelman)
Assignee | ||
Comment 12•10 years ago
|
||
Looks like the unit tests for the failing files use their own requires for the performance_testing_helper. I have since updated these files and recommitted. Waiting for a green try run before continuing.
Updated•10 years ago
|
Target Milestone: 2.1 S6 (10oct) → 2.1 S7 (Oct24)
Assignee | ||
Comment 13•10 years ago
|
||
Keywords: checkin-needed
Comment 14•10 years ago
|
||
Assignee | ||
Comment 15•10 years ago
|
||
This test has issues running as-is, see bug 1069863 comment 33. I have a local script running to work around these issues and gather data, but this test should not be used in any official capacity until bug 1069863 is resolved.
Updated•10 years ago
|
blocking-b2g: backlog → ---
tracking-b2g:
--- → backlog
You need to log in
before you can comment on or make changes to this bug.
Description
•