If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Correct robocop BaseTest.enterUrl assertion argument order

NEW
Unassigned

Status

()

Firefox for Android
General
4 years ago
4 years ago

People

(Reporter: mcomella, Unassigned)

Tracking

27 Branch
All
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

4 years ago
The expected url and actual url should be switched, otherwise the test output is described incorrectly:

11:48:38     INFO -  4 INFO TEST-UNEXPECTED-FAIL | testShareLink | URL typed properly - got http://mochi.test:8888/tests/robocop/robocop_big_link.html, expected about:homehttp://mochi.test:8888/tests/robocop/robocop_big_link.html

via https://tbpl.mozilla.org/php/getParsedLog.php?id=29324381&tree=Fx-Team#error0
Fixing this will change the error messages associated with existing bugs. I don't think there's much room for confusion, but it might be best to comment on existing "URL typed properly" bugs and/or give the sheriffs a heads-up when landing this fix.
You need to log in before you can comment on or make changes to this bug.