Add automated test for "Searching for bookmarks from the Bookmarks Sidebar works properly"

RESOLVED FIXED in Firefox 59

Status

()

enhancement
P3
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: icrisan, Assigned: icrisan)

Tracking

(Blocks 1 bug)

unspecified
Firefox 59
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox59 fixed)

Details

Attachments

(1 attachment, 2 obsolete attachments)

testrail link: https://testrail.stage.mozaws.net/index.php?/cases/view/4104

Preconditions: A dirty profile with reasonable amount of browsing history and bookmarks.

Steps:
1. Launch Firefox
2. Enable the Bookmarks Sidebar
3. Search for the name of a bookmarked website, using the search bar available in the sidebar
4. Search for a bookmark by tag, using the search bar available in the sidebar

Expected results:
Step 1: Firefox is successfully launched.
Step 2: The Bookmarks Sidebar is enabled and displayed at the left side.
Step 3: The bookmark searched for is found and displayed in the sidebar.
Step 4: The bookmark(s) having that tag or/and the URL associated is found and displayed in the sidebar.
Posted patch search_bookmarks_sidebar (obsolete) — Splinter Review
Did you mean to assign this to me Ioana? As you've attached a patch I suspect it should be assigned to you instead?
Flags: needinfo?(icrisan)
Hi Dave! 
We assigned it to you just temporary. I don't know a person with rights to review the "bookmarks" tests. We intended to discuss about this in today's meeting but seems that the meeting was postponed until tomorrow. Can you help me with a name?
Flags: needinfo?(icrisan)
The assignee is the individual working on the bug, not the reviewer. If you view the patch and set the review flag you'll see suggested reviewers for this component. They appear to be Drew Willcoxon :adw, Justin Dolske [:Dolske], Marco Bonardo [::mak], and Mark Banner (:standard8).
Assignee: dave.hunt → icrisan
Attachment #8930467 - Flags: review?(adw)
(In reply to Dave Hunt (:davehunt) from comment #4)
> The assignee is the individual working on the bug, not the reviewer. If you
> view the patch and set the review flag you'll see suggested reviewers for
> this component. They appear to be Drew Willcoxon :adw, Justin Dolske
> [:Dolske], Marco Bonardo [::mak], and Mark Banner (:standard8).

Thank you!
Comment on attachment 8930467 [details] [diff] [review]
search_bookmarks_sidebar

Review of attachment 8930467 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/components/places/tests/browser/browser_bookmarks_sidebar_search.js
@@ +1,2 @@
> +/**
> + *  Test searching for bookmarks(by title and by tag) from the Bookmarks sidebar.

Nit: Need a space between "bookmarks" and "("

@@ +55,5 @@
> +  });
> +
> +  // Cleanup.
> +  await PlacesUtils.bookmarks.eraseEverything();
> +  await PlacesTestUtils.clearHistory();

Please use PlacesUtils.bookmarks.eraseEverything() instead.  (The PlacesTestUtils shortcut is deprecated.)
Attachment #8930467 - Flags: review?(adw) → review+
Posted patch search_bookmarks_sidebar_1 (obsolete) — Splinter Review
Performed the changes that Drew requested.
Priority: -- → P3
Keywords: checkin-needed
Pushed by nerli@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4bc8cdd5e716
Searching for bookmarks from the Bookmarks Sidebar works properly r=adw
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/4bc8cdd5e716
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 59
You need to log in before you can comment on or make changes to this bug.