Open
Bug 1956816
Opened 3 months ago
Update test_geckoview_translations.html to use GleanTest.js
Categories
(GeckoView :: Translations, task)
Tracking
(Not tracked)
NEW
People
(Reporter: nordzilla, Unassigned)
Details
Description
The test_geckoview_translations file uses its own implementation of GleanTest
, which predates the now more widely available implementation in GleanTest.js.
In Bug 1952119 I tried to migrate this over, but I ran into several intermittent test failures. Since there was a time limit to remove some expiring telemetry, I opted to simply remove the telemetry and file this follow-up bug to do the migration.
When we migrate this, we should also introduce an assertion for the labeled counter, which would be invoked by:
GleanTest.translations.requestCount.full_page.testGetValue()
The current GleanTest
implementation used by the test file only supports a depth of exactly two properties, but the new GleanTest
implementation can support the above call which is 3 properties deep.
Steps to Implement
- Migrate existing test to use
GleanTest.js
. - Introduce new assertions for the labeled
requestCount
counter as described above. - Determine if the intermittent test failures that I encountered when trying this earlier are fixed. If not, dig into them and fix them.
You need to log in
before you can comment on or make changes to this bug.
Description
•