mozilla-mobile/firefox-ios

[Glean] Migrate TermsOfUseTelemetry to use the GleanWrapper instead

クローズ

#29,805 opened on 2025/10/03

GitHub で見るコントリビューターガイド
 (4 件のコメント) (0 件のリアクション) (0 人の担当者)Swift (3,203 件のフォーク)batch import
Contributor OKGood first issue

Repository metrics

Stars
 (12,950 個のスター)
PR merge metrics
 (平均マージ 3d 2h) (30d で 221 merged PRs)

説明

: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

コントリビューターガイド

調査方針
既存の `DefaultGleanWrapper` や `MockGleanWrapper` ファイルを調べてAPIを理解してください。`TermsOfUseTelemetry` 内の `GleanMetrics.` 直接呼び出しを `GleanWrapper.shared.*` 相当のメソッドに置き換えてください。`TermsOfUseTelemetryTests` を更新して `MockGleanWrapper` を注入し、呼び出しを検証してください。
技術スタック
swift
領域
mobile
Issue 種別
リファクタリング
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
半日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
SwiftXCTest
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
65