Open Bug 1518756 Opened 6 years ago Updated 2 years ago

[meta] Update more tests to use TelemetryTestUtils.jsm

Categories

(Toolkit :: Telemetry, enhancement, P3)

enhancement

Tracking

()

People

(Reporter: standard8, Unassigned)

References

Details

(Keywords: meta)

User Story

* getParentProcessScalars has a few duplicates:
  * https://searchfox.org/mozilla-central/search?q=getParentProcessScalars&case=false&regexp=false&path=
* Extension telemetry has a few handlers:
  * https://searchfox.org/mozilla-central/search?q=function.*histogram&case=false&regexp=true&path=toolkit%2Fcomponents%2Fextensions%2Ftest
* Other possible cases:
  * https://searchfox.org/mozilla-central/rev/c3ebaf6de2d481c262c04bb9657eaf76bf47e2ac/browser/components/extensions/test/browser/browser_ext_pageAction_telemetry.js#11-17
  * https://searchfox.org/mozilla-central/rev/c3ebaf6de2d481c262c04bb9657eaf76bf47e2ac/browser/components/search/test/browser/browser_searchTelemetry.js#44-84
  * https://searchfox.org/mozilla-central/rev/c3ebaf6de2d481c262c04bb9657eaf76bf47e2ac/dom/webauthn/tests/browser/browser_webauthn_telemetry.js#29-37
  * https://searchfox.org/mozilla-central/rev/c3ebaf6de2d481c262c04bb9657eaf76bf47e2ac/toolkit/components/telemetry/tests/unit/test_TelemetrySend.js#78-80

Probably more, here is the search I used to find some of these:

https://searchfox.org/mozilla-central/search?q=function.*histogram&case=false&regexp=true&path=

In bug 1518152 we created a TelemetryTestUtils.jsm utility file, and used it for a few tests.

I believe there are more places that it should be able to be used for, see the user story for some pointers - it isn't an extensive list.

Depends on: 1518152

Split into a bunch of good first bugs, make this one meta.

Flags: needinfo?(chutten)
Flags: needinfo?(chutten)
Keywords: meta
Priority: -- → P3
Summary: Update more tests to use TelemetryTestUtils.jsm → [meta] Update more tests to use TelemetryTestUtils.jsm
Depends on: 1519476
Depends on: 1519480
Depends on: 1559692
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.