<div dir="ltr"><div dir="ltr"><div dir="ltr">We are about to land <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1467712" target="_blank">https://bugzilla.mozilla.org/show_bug.cgi?id=1467712</a> which changes the behavior of `ok()` test helpers provided by Assert.jsm, SimpleTest.js and browser-test.js (as well as some test helpers based on them).</div><div dir="ltr"><br></div><div>`ok()` used to accept up to four arguments: condition, name, exception/diagnostic and stack.</div><div>After bug 1467712 lands, `ok()`will only accept the 2 first arguments and will fail the test if more arguments are provided.</div><div><br></div><div>The reason behind this change is that `ok()` was regularly used by mistake instead of `is()`:</div><div>  ok(value1, value2, "Check value1 == value2"); <br></div><div>  // passes as long as value1 is truthy and won't detect regressions if value1 != value2</div><div><br></div><div> We recently fixed all the incorrect uses of ok() in the code base in <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1499096" target="_blank">https://bugzilla.mozilla.org/show_bug.cgi?id=1499096</a>. `is()` is called with 3 arguments, so restricting the number of arguments for ok() will prevent developers from making similar mistakes in the future.</div><div><br></div><div>If you need the old `ok()` with 4 arguments, it has been moved to the `record()` method and you can use it instead.<br></div></div></div>