| commit | a2a79250994723769f1523229cbe454948cd6763 | |
| author | Jan-Erik Rediger <jrediger@mozilla.com> | |
| Wed, 24 Aug 2022 10:02:46 +0000 (24 12:02 +0200) | ||
| committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | |
| Thu, 25 Aug 2022 15:16:59 +0000 (25 15:16 +0000) | ||
| tree | 2a58051e9afd6fe24b3c87209788256d1cf16c83 | treesnapshot (tar.gz zip) |
| parent | 8b055dae572e9417d3616535176e5be782fbec38 | commitdiff |
[components] Revert "Use the local server for testing"
This reverts commit cf03ecc73be1988e3aaa90edd353a458cdc2f9d9.
In Glean v50 `testGetValue` methods are actually not gated on test
mode (it's still not recommended though and absolutely IS still a
test-only API for all intents and purposes).
Further it turns out that calling `GleanTestLocalServer` in one test
might influence the other test and we're seeing intermittens where
`BaselinePingTest` is trying to upload to `localhost:0`, which of course
is invalid.
I have not yet figured out how we can properly separate out the state.
Alternatively we could merge the 2 tests into a single one.
Fixes https://github.com/mozilla-mobile/android-components/issues/11340
This reverts commit cf03ecc73be1988e3aaa90edd353a458cdc2f9d9.
In Glean v50 `testGetValue` methods are actually not gated on test
mode (it's still not recommended though and absolutely IS still a
test-only API for all intents and purposes).
Further it turns out that calling `GleanTestLocalServer` in one test
might influence the other test and we're seeing intermittens where
`BaselinePingTest` is trying to upload to `localhost:0`, which of course
is invalid.
I have not yet figured out how we can properly separate out the state.
Alternatively we could merge the 2 tests into a single one.
Fixes https://github.com/mozilla-mobile/android-components/issues/11340