firefox-ios icon indicating copy to clipboard operation
firefox-ios copied to clipboard
verified publisher icon mozilla-mobile

[Glean] Migrate TermsOfUseTelemetry to use the GleanWrapper instead

Open data-sync-user opened this issue 2 months ago • 4 comments

:rocket: Context

TermsOfUseTelemetry is recording telemetry events with the concrete Glean implementation. We don't want this anymore. More on telemetry tests here.

:memo: Description

  • Use the GleanWrapper inside TermsOfUseTelemetry and replace calls to each GleanMetrics. with the corresponding API.
  • TermsOfUseTelemetryTests should be adjusted so we pass a MockGleanWrapper and assert accordingly.

:information_source: Reference Person

lmarceau

┆Issue is synchronized with this Jira Task

data-sync-user avatar Oct 03 '25 17:10 data-sync-user

Hi @lmarceau , I'd like to work on this issue for Outreachy. Can I be assigned? Thanks!

awaan05 avatar Oct 25 '25 16:10 awaan05

thanks @awaan05, I assigned you to this issue!

cyndichin avatar Oct 27 '25 12:10 cyndichin

@cyndichin the PR is up now, the migration is done. Let me know if anything needs adjusting.

awaan05 avatar Nov 09 '25 20:11 awaan05

The PR went stale since we didn't hear back @awaan05. We can re-open if you would like to continue to work on it, otherwise we will open this ticket for others to work on.

cyndichin avatar Dec 09 '25 16:12 cyndichin