1721703 - Fix incorrect usage of assertPausedLocation in Debugger mochitests

Status: RESOLVED FIXED

(DevTools :: Debugger, task, P3)

Description

[Julian Descottes [:jdescottes]]

The test helper assertPausedLocation has the following jsdoc:

/**
 * Assert that the debugger is paused at the correct location.
 *
 * @memberof mochitest/asserts
 * @param {Object} dbg
 * @param {String} source
 * @param {Number} line
 * @static
 */

However, it only accepts a single dbg parameter, source & line are not used/supported.
We should update the JSDoc and fix existing callsites passing 2nd & 3rd arguments to this method. Instead assertPausedAtSourceAndLine can be used. Or we can merge the two methods.

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1721703 - [devtools] Fix incorrect usage of assertPausedLocation in Debugger mochitests

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/30c583a13c33
[devtools] Fix incorrect usage of assertPausedLocation in Debugger mochitests r=nchevobbe

Comment 3

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/30c583a13c33