Closed Bug 1133310 Opened 5 years ago Closed 1 year ago
Environment: check |active GMPlugins| test coverage .
We need |activeGMP| test coverage in test_TelemetryEnvironment.js. test_openh264.js (soon test_gmpProvider.js?) shows how to fake them.
Per bug 1203159 the GMPProvider will already get instantiated in test_TelemetryEnvironment.js, but the version data may be null. We should figure out whats going on there.
Points: --- → 1
Depends on: 1203159
Priority: -- → P3
Summary: TelemetryEnvironment: add |activeGMPlugins| test coverage. → TelemetryEnvironment: check |activeGMPlugins| test coverage.
Per bug 1136714 we should also see that the test-coverage for this is activated on Android again.
We should also change  from |if (!plugin.isGMPlugin)| to |if (!plugin.isGMPlugin || !plugin.isValid)|.  - https://dxr.mozilla.org/mozilla-central/rev/f029ccdee154bdc2b49a1e0fcd5b0fa0397aa97b/toolkit/components/telemetry/TelemetryEnvironment.jsm#624
Please ignore comment 4, as plugin.isValid is not available from the GMPlugin wrapper.
Component: Client: Desktop → Telemetry
Product: Firefox Health Report → Toolkit
It appears as though we have added test coverage for the structure of activeGMP in test_TelemetryEnvironment.js
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.