1437433 - Add automated test for "Websites can be bookmarked from a private window"

Status: RESOLVED FIXED

(Firefox :: Bookmarks & History, enhancement, P2)

Description

[Paul Silaghi, QA [:pauly]]

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

Steps:
1. Launch Firefox.
2. Open a private window from menu [≡] → New Private Window.
3. Open a website that's currently not in your bookmarks list.
4. Click the star-shaped button to bookmark it.

Expected results:
1. Firefox is successfully launched.
2. A new private window is successfully opened.
3. The website in question is successfully loaded.
4. On click, the star-shaped button shows an animation and changes its color to blue.
The website is successfully bookmarked and no errors are thrown for this action.

Comment 1

[Paul Silaghi, QA [:pauly]]

Attachment: browser_bookmark_private_window

Comment 2

[Mark Banner (:standard8)]

Comment on attachment 8950141 [details] [diff] [review]
browser_bookmark_private_window

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

Looks good, just a couple of comments/queries to address.

::: browser/components/places/tests/browser/browser_bookmark_private_window.js
@@ +16,5 @@
> +
> +add_task(async function test_add_bookmark_from_private_window() {
> +  let win = await BrowserTestUtils.openNewBrowserWindow({private: true});
> +  let tab = win.gBrowser.selectedTab = win.gBrowser.addTab(TEST_URL);
> +  await BrowserTestUtils.browserLoaded(tab.linkedBrowser);

We can simplify these two lines to:

let tab = await BrowserTestUtils.openNewForegroundTab(gBrowser, TEST_URL);

`openNewForegroundTab` automatically waits for the load to complete.

@@ +34,5 @@
> +  Assert.equal(bookmarkStar.getAttribute("starred"), "true", "Bookmark star changed its state correctly.");
> +
> +  // Check if the "Page Bookmarked" panel is open.
> +  let bookmarkPanelTitle = win.document.getElementById("editBookmarkPanelTitle");
> +  Assert.equal(bookmarkPanelTitle.value, "Page Bookmarked", "Bookmark panel title is correct.");

I think this is already checked by the fact that we've done the `await shownPromise`, isn't it?

Comment 3

[Paul Silaghi, QA [:pauly]]

Attachment: browser_bookmark_private_window

Thanks for the review.
Added the requested changes.

Comment 4

[Mark Banner (:standard8)]

Comment on attachment 8951573 [details] [diff] [review]
browser_bookmark_private_window

Great, thanks.

Comment 5

[Paul Silaghi, QA [:pauly]]

Try results:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=eabc730823fc16e8bea73993bd1d6b62ee4c49c0

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/91b807639ce3
Add a test for checking that a website can be bookmarked from a private window. r=standard8

Comment 7

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/91b807639ce3