Closed Bug 1522536 Opened 6 years ago Closed 6 years ago

Figure out what to do with Firefox UI tests relating to the address bar wrt QuantumBar

Categories

(Testing :: Firefox UI Tests, enhancement, P2)

Version 3
enhancement

Tracking

(firefox67 fixed)

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: standard8, Assigned: dao)

References

Details

(Whiteboard: [fxsearch])

Attachments

(1 file)

We are currently rewriting the address bar (bug 1477942). In checking for failures we've identified that some tests in testing/firefox-ui/tests/functional/locationbar are currently failing:

testing/firefox-ui/tests/functional/locationbar/test_escape_autocomplete.py TestEscapeAutocomplete.test_escape_autocomplete | TimeoutException: Timed out after 5.0 seconds
testing/firefox-ui/tests/functional/locationbar/test_suggest_bookmarks.py TestStarInAutocomplete.test_star_in_autocomplete | TimeoutException: Timed out after 5.0 seconds

Should these be re-written as browser mochitests (if they're not already covered - I think they probably are), or do we need to update Marionette?

If we can port to mochitest-browser, I think we should. The only tests actually requiring firerox-ui should be the ones involving a restart of Firefox (or multiple instances of it). I don't think we have anything like that for the Address Bar.

I'm happy to get rid of those if coverage is present or will be present for quantum bar.

I too expect browser chrome tests have us covered here. These tests don't seem to do anything noteworthy, let's get rid of them.

Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/59fad017d3c5 Remove redundant address bar UI tests. r=whimboo
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: