Open Bug 1120641 Opened 5 years ago Updated 4 years ago

Add tests for the default search engine header added to the search suggestions popup

Categories

(Firefox :: Search, defect, P4)

defect
Points:
3

Tracking

()

People

(Reporter: mossop, Unassigned)

Details

(Whiteboard: [ui][fxsearch])

No description provided.
Flags: qe-verify-
Flags: firefox-backlog+
Points: --- → 3
Priority: -- → P4
Whiteboard: [fxsearch][searchui]
I'm not sure if the headers here are the line at the top of the search panel with the current default engine, or the line saying "Search for <keyword> with:" (or maybe both?), but if it's the later, we added tests in bug 1110767.
You defined it in bug 1107178 comment 2 ;)

Sounds like we just need a test for the default search engine header now.
Summary: Add tests for the new headers added to the search suggestions popup → Add tests for the default search engine header added to the search suggestions popup
I'd be interested in taking a look into this, it looks like we need to test that the icon URI matches what is stored in the default engine's icon URI, and that the message is properly displayed as "<Engine Name> Search.

Is it possible to have a search engine that doesn't have an icon?  And if so, then what icon URI is displayed instead?  Also is it possible for a search engine to be an empty string?
(In reply to Chris from comment #3)
> I'd be interested in taking a look into this, it looks like we need to test
> that the icon URI matches what is stored in the default engine's icon URI,
> and that the message is properly displayed as "<Engine Name> Search.

We should probably also check that clicking that header starts a search.

> Is it possible to have a search engine that doesn't have an icon?

Yes. In that case a generic search icon is displayed instead.

> Also is it possible for a search engine to be an empty string?

I hope it's not, but I've never checked.
> > Also is it possible for a search engine to be an empty string?
>
> I hope it's not, but I've never checked.

It might be better to add a test for this case here: http://mxr.mozilla.org/mozilla-central/source/toolkit/components/search/tests/xpcshell/ as part of a possible followup bug if we can verify that adding an engine with an empty name is invalid behavior (It looks like it is).
Rank: 45
Whiteboard: [fxsearch][searchui] → [ui][fxsearch]
You need to log in before you can comment on or make changes to this bug.