Open Bug 1312699 Opened 5 years ago Updated 4 years ago
Controller .test Reset()/reset() behave correctly
Right now, we do something that looks odd in racy in in the code  that we use to reset Telemetry between XPCSHELL tests. Ideally, we should change that this way: 1) Call the shutdown() function of all our subsystems (TelemetryEnvironment, TelemetrySession, ...) 2) Reset any state/variable held by these modules. We change change the testReset function of other modules to do that. 3) Then call the setup() functions again for each subsystem. This could potentially cause some fallout in our tests if they were relying on the old, unclear behaviour to work.  - https://dxr.mozilla.org/mozilla-central/rev/c845bfd0accb7e0c29b41713255963b08006e701/toolkit/components/telemetry/TelemetryController.jsm#947
5 years ago
Points: --- → 2
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.