mozilla-mobile/firefox-ios

[Glean] Migrate TermsOfUseTelemetry to use the GleanWrapper instead

Geschlossen

#29.805 geöffnet am 03.10.2025

Auf GitHub ansehenContributor Guide
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Swift (3.203 Forks)batch import
Contributor OKGood first issue

Repository-Metriken

Stars
 (12.950 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 3T 2h) (221 gemergte PRs in 30 T)

Beschreibung

: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

Contributor Guide

Research-Richtung
Untersuche vorhandene Dateien wie `DefaultGleanWrapper` und `MockGleanWrapper`, um die API zu verstehen. Ersetze die direkten `GleanMetrics.` Aufrufe in `TermsOfUseTelemetry` durch äquivalente Methoden von `GleanWrapper.shared.*`. Aktualisiere `TermsOfUseTelemetryTests`, um `MockGleanWrapper` zu injizieren und die Aufrufe zu überprüfen.
Tech Stack
swift
Domain
mobile
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
SwiftXCTest
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65