mozilla-mobile/firefox-ios

[Glean] Migrate TermsOfUseTelemetry to use the GleanWrapper instead

已关闭

#29,805 创建于 2025年10月3日

在 GitHub 查看贡献者指南
 (4 条评论) (0 个反应) (0 位负责人)Swift (3,203 个派生)batch import
Contributor OKGood first issue

仓库指标

星标
 (12,950 个星标)
PR 合并指标
 (平均合并 3天 2小时) (30 天内合并 221 个 PR)

描述

: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
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
半天
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
SwiftXCTest
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
65