Open
Bug 1350941
Opened 8 years ago
Updated 2 years ago
Changing TelemetryController._onUploadPrefChange to async/await causes xpcshell tests to fail
Categories
(Toolkit :: Telemetry, defect, P4)
Tracking
()
NEW
People
(Reporter: djmdeveloper060796, Unassigned)
References
Details
(Whiteboard: [measurement:client])
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0
Build ID: 20170317213149
Steps to reproduce:
In TelemetryController.jsm, in _onUploadPrefChange function change Task.spawn to async function ,i.e, Task.spawn(function*() {}) to (async function() {})() and change yield to await
run ./mach xpcshell-test toolkit/components/telemetry/tests/unit/test_TelemetryController.js
Actual results:
The test should have passed
Expected results:
The test fails. However in the Task.spawn function if the order of ping removal is changed that is first the pending pings are removed and then the current pings are removed then the test passes
Reporter | ||
Comment 1•8 years ago
|
||
@Alessio I would love to work on this bug, how should I proceed?
Flags: needinfo?(alessio.placitelli)
Comment 2•8 years ago
|
||
I warn you, this bug is potentially involving and troublesome. Feel free to pick another bug if it doesn't sound too interesting! :-)
This is an investigation bug, the expected outcome of it should be:
1) Figure out why the change makes the test fail.
2) Fix the problem.
I'd love to stress how the first point is important: we don't want to change the order of any operation unless we understand exactly what is going on behind the scenes and why that fixes the issue.
With that said, there's no fixed set of steps to fix it. What I would do:
- Run the test without the change, and save the log output to a file.
- Change the code to reproduce the problem.
- Dump the log output of the failing test to a different file.
- Analyse the logs to figure out what's going on, how is that behaving different.
Status: UNCONFIRMED → NEW
Depends on: 1344738
Ever confirmed: true
Flags: needinfo?(alessio.placitelli) → needinfo?(djmdeveloper060796)
OS: Unspecified → All
Priority: -- → P4
Hardware: Unspecified → All
Reporter | ||
Comment 3•8 years ago
|
||
@Alessio I would like to give it a try, I will start working on it ASAP. I will let you know if I am stuck :)
Flags: needinfo?(djmdeveloper060796)
Updated•7 years ago
|
Component: Untriaged → Telemetry
Product: Firefox → Toolkit
Updated•7 years ago
|
Whiteboard: [measurement:client]
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•