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

Status: RESOLVED FIXED

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

Description

[Mark Banner (:standard8)]

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?

Comment 1

[Marco Bonardo [:mak]]

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.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+2]]

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

Comment 3

[Dão Gottwald [:dao]]

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.

Comment 4

[Dão Gottwald [:dao]]

Attachment: Bug 1522536 - Remove redundant address bar UI tests. r=whimboo

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/59fad017d3c5
Remove redundant address bar UI tests. r=whimboo

Comment 6

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/59fad017d3c5