mozilla-mobile/firefox-ios

[Glean] Migrate TermsOfUseTelemetry to use the GleanWrapper instead

Chiusa

#29.805 aperta il 3 ott 2025

Vedi su GitHubGuida contributor
 (4 commenti) (0 reazioni) (0 assegnatari)Swift (3203 fork)batch import
Contributor OKGood first issue

Metriche repository

Star
 (12.950 stelle)
Metriche merge PR
 (Merge medio 3g 2h) (221 PR mergiate in 30 g)

Descrizione

: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

Guida contributor

Direzione di ricerca
Esamina i file esistenti come `DefaultGleanWrapper` e `MockGleanWrapper` per comprendere l'API. Sostituisci le chiamate dirette a `GleanMetrics.` in `TermsOfUseTelemetry` con metodi equivalenti di `GleanWrapper.shared.*`. Aggiorna `TermsOfUseTelemetryTests` per iniettare `MockGleanWrapper` e verificare le chiamate.
Tech stack
swift
Dominio
mobile
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
SwiftXCTest
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65