Open Bug 1184687 Opened 10 years ago Updated 2 years ago

ok() should fail when you pass more than 2 arguments to it

Categories

(Testing :: Mochitest, defect)

defect

Tracking

(firefox42 affected)

Tracking Status
firefox42 --- affected

People

(Reporter: ehsan.akhgari, Unassigned)

Details

In <https://dxr.mozilla.org/mozilla-central/source/dom/tests/mochitest/fetch/test_request.js#217>, now we have: ok(req.url, (new URL("./request", self.location.href)).href, "request.url should be serialized with exclude fragment flag set"); Here the author meant to use is(), but used ok() by mistake, and nothing complained, so this got checked in. ok() should fail if you pass more than two argument to it since that is almost definitely a mistake by the programmer.
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.