Closed Bug 1442153 Opened 3 years ago Closed 3 years ago
_inspector _expand-collapse .js fails when converted to async/await instead of Task
59 bytes, text/x-review-board-request
$ ./mach mochitest devtools/client/inspector/test/browser_inspector_expand-collapse.js fails with this error: TEST-UNEXPECTED-FAIL | devtools/client/inspector/test/browser_inspector_expand-collapse.js | node has been successfully expanded
Hum, that's fascinating, the error from comment 0 happens when you are based on top of my WIP patch for bug 1440321 (converting the whole client folder to async/await) On top of m-c tip, you get this error instead: TEST-UNEXPECTED-FAIL | devtools/client/inspector/test/browser_inspector_expand-collapse.js | Uncaught exception - Protocol error (unknownError): '' is not a valid selector Note that is may be related to bug 1441531 as this test also uses waitForMultipleChildrenUpdates.
This is related to the changes in bug 1441531. We will need to wait for those changes to land before we can merge the fix for this. I'm running try now to make sure nothing else broke.
No longer blocks: 1440321
Generators have some funny behavior with async/await it looks like... It was returning a generator object rather than the return value. Since no other tests were relying on the function in question i updated it to be async/await as well
Comment on attachment 8959623 [details] Bug 1442153 - Convert browser_inspector_expand-collaps.js to async await and getNodeFrontForSelector in test/head. https://reviewboard.mozilla.org/r/228440/#review234564 Thanks Yulia, do not hesitate to squash my change into yours.
Attachment #8959623 - Flags: review?(poirot.alex) → review+
Comment on attachment 8962358 [details] Bug 1442153 - Convert browser_inspector_expand-collapse.js to async/await. https://reviewboard.mozilla.org/r/231220/#review236614
Attachment #8962358 - Flags: review?(poirot.alex) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/0a61a8fd3866 Convert browser_inspector_expand-collapse.js to async/await. r=ochameau
You need to log in before you can comment on or make changes to this bug.