Closed Bug 1514164 Opened 2 years ago Closed 2 years ago

Identify tests failing with the Quantum Bar


(Firefox :: Address Bar, enhancement, P1)




Firefox 66
Tracking Status
firefox66 --- fixed


(Reporter: mak, Assigned: standard8)



(Whiteboard: [fxsearch])


(3 files)

We must get a full list of all the tests that will fail when we switch on the quantumbar pref, those tests should be moved into a legacy folder in the urlbar component so that we can switch them on/off based on the quantum bar pref, and then ported to the new codebase.
Priority: P1 → P2
Depends on: 1521043

Note: in bug 1521043, I'm going to move all the existing urlbar tests that we know of (without digging), into the legacy directory.

We can then work through them to work out which needs porting etc.

Changed my mind, lets just do this.

Assignee: nobody → standard8
Priority: P2 → P1

This moves the tests to browser/components/urlbar and sets them up for running both against QuantumBar as well as the legacy address bar.

Depends on D16995

Try build:

For context:

  • .ini files now let you set prefs for the test to run (not sure if its is just per dir or per test but we're using it in the dir scope)
  • the plan is to have tests/browser which runs quantumbar=true and tests/legacy with quantumbar=false
  • Since you can also reference tests in other directories, tests/legacy references all the legacy tests, plus the ones in tests/browser which are compatible with the old urlbar.
  • The minor possible downside is if you run one specific test it’ll run it twice (once with the pref true, once false), but that could also been seen as an advantage.

Currently the files are where they are most compatible with - tests/browser if they run with quantumbar, legacy otherwise. We can move them across as we go.

Additionally, at the moment, tests/browser fails with quantumbar=true. I think that's because we haven't quite fixed something we should have, and I'm looking into it. I might do that toggle in a third changeset or a follow-up bug, depending on when it is ready, reviews etc.

IS this commented out?

# 'tests/legacy/browser.ini',

Bah, yes. Shouldn't have been, I was testing something locally and left it in by mistake.

New try push:

Blocks: 1520494
Blocks: 1520839
Blocks: 1521490

Although these tests are all able to pass individually, some still fail when run together. Bug 1521490 will address those.

Depends on D16996

Blocks: 1521513
Blocks: 1521532
Blocks: 1521534

Last try push that included disabling a few tests due to shutdown leaks:

Pushed by
Change urlbar tests to use the test path rather than hard-coding it. r=mak
Move urlbar tests to the new urlbar directory and split out tests that already pass with QuantumBar. r=mak
Enable running some of the urlbar tests against QuantumBar. r=mak
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66
Blocks: 1521707
You need to log in before you can comment on or make changes to this bug.