Created attachment 634649 [details] [diff] [review] patch v1: stop modifying API call results The webapps mochitests modify the result values of the API calls they make, adding `status` properties that they then use to ensure the correct (success, error) callback was called. That will stop working once we land the fix for bug 702369, which makes those result values immutable, as fabrice notes in bug 763847, comment 9. It's also brittle, as it would break if we added a `status` property to those values in the API. Thus those mochitests should stop modifying those result values and use a different mechanism to ensure the correct callback was called. Here's a patch that does that. I can imagine some further improvements to these tests, but this is the minimal change that addresses the issue. This should block Basecamp/Kilimanjaro because it blocks a Basecamp/Kilimanjaro blocker (bug 702369).
Comment on attachment 634649 [details] [diff] [review] patch v1: stop modifying API call results Erm, Jonas is away, and Felipe did the original review of this code, so switching the review request to Felipe.
Comment on attachment 634649 [details] [diff] [review] patch v1: stop modifying API call results https://hg.mozilla.org/integration/mozilla-inbound/rev/4a7c9600e845