1121067 - Write client-side test to make sure we didn't break Telemetry submissions.

Status: RESOLVED FIXED

(Toolkit :: Telemetry, defect, P4)

Description

[Blake Winton (:bwinton) (:☕️)]

As the guilty party, it would have been awesome if we had a client-side test that would have prevented us landing the Telemetry breakage reported in bug 1119327.

In this case, it would have been enough to navigate to about:telemetry and ensure that the text beside Simple Measurements and System Information was "Click to toggle section", and not "(No data collected)", which seems to me like a decent candidate for a mochitest…

Comment 1

[Georg Fritzsche [:gfritzsche]]

Test coverage for these scenarios requires doing a "proper" Firefox launch and verifying ping generation (e.g. from shutdown).
The Telemetry marionette tests that landed should cover this kind of scenario, but we should double check if they do and if that shows up as CI errors anywhere.

Comment 2

[Jan-Erik Rediger [:janerik]]

We now have marionette tests that will test on a real Firefox instance and checks stuff is actually received (on a fake server), e.g. [1]

[1]: https://searchfox.org/mozilla-central/rev/13788edbabb04d004e4a1ceff41d4de68a8320a2/toolkit/components/telemetry/tests/marionette/tests/client/test_search_counts_across_sessions.py#14