TelemetrySend: add test coverage for the submission path.

NEW
Unassigned

Status

()

Toolkit
Telemetry
P4
normal
3 years ago
2 years ago

People

(Reporter: Dexter, Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
Points:
1

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [measurement:client])

(Reporter)

Description

3 years ago
TelemetryPing.jsm builds a submission URL based on some of the information in the ping. We need to add test coverage for this in test_TelemetryPing.js.
(Reporter)

Updated

3 years ago
Blocks: 1120356
Blocks: 1122482
No longer blocks: 1120356
Blocks: 1201022
No longer blocks: 1122482
(Reporter)

Comment 1

2 years ago
This test should be added to test_TelemetrySend.js. We should also consider moving the version check from [1] to the newly added test.

[1] - https://dxr.mozilla.org/mozilla-central/rev/605de27d4e7f530159842649c02075c578d7a4a5/toolkit/components/telemetry/tests/unit/test_TelemetryController.js#140
Points: --- → 1
Priority: -- → P3
Whiteboard: [measurement:client]
(Reporter)

Updated

2 years ago
Blocks: 1137262
No longer blocks: 1201022
Priority: P3 → P4
Summary: TelemetryPing: add test coverage for the submission path. → TelemetrySend: add test coverage for the submission path.
Component: Client: Desktop → Telemetry
Product: Firefox Health Report → Toolkit
You need to log in before you can comment on or make changes to this bug.