Register telemetry events statically
Categories
(Firefox :: Normandy Client, enhancement, P3)
Tracking
()
People
(Reporter: mythmon, Assigned: glasserc)
References
Details
Attachments
(2 files)
Reporter | ||
Updated•7 years ago
|
Updated•7 years ago
|
Assignee | ||
Comment 2•6 years ago
|
||
Reporter | ||
Updated•6 years ago
|
Assignee | ||
Comment 3•6 years ago
|
||
Because the mock no longer checks whether the format of events is
correct, check the events that Telemetry actually records, in order to
make sure we conform to the event schema.
Some uses check that no calls were made to the stub. Although this
is technically still valid (because we don't care about the format of
events in this case), convert even these to check events for
consistency.
TelemetryTestUtils.assertEvents doesn't allow us to pass a message,
but most of these messages don't add anything useful and at least one
is just wrong so it's not a big loss.
Depends on D19540
Assignee | ||
Comment 4•6 years ago
|
||
Assignee | ||
Comment 5•6 years ago
|
||
Just to be sure: https://treeherder.mozilla.org/#/jobs?repo=try&revision=1413fba90820711a634fc018ab40e05ae5b5fe09
Pushed by csabou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ec979e0f73ce
register telemetry events statically r=Gijs
https://hg.mozilla.org/integration/autoland/rev/48d98388c29c
Convert checks to examine snapshotted events r=Gijs
Comment 7•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ec979e0f73ce
https://hg.mozilla.org/mozilla-central/rev/48d98388c29c
Updated•5 years ago
|
Description
•