add_task(async function invalid_input_throws() {
  Assert.throws(() => PlacesUtils.bookmarks.getRecent(),
                /numberOfItems argument is required/);
  Assert.throws(() => PlacesUtils.bookmarks.getRecent("abc"),
                /numberOfItems argument must be an integer/);
  Assert.throws(() => PlacesUtils.bookmarks.getRecent(1.2),
                /numberOfItems argument must be an integer/);
  Assert.throws(() => PlacesUtils.bookmarks.getRecent(0),
                /numberOfItems argument must be greater than zero/);
  Assert.throws(() => PlacesUtils.bookmarks.getRecent(-1),
                /numberOfItems argument must be greater than zero/);
});

add_task(async function getRecent_returns_recent_bookmarks() {
  await PlacesUtils.bookmarks.eraseEverything();

  let bm1 = await PlacesUtils.bookmarks.insert({ parentGuid: PlacesUtils.bookmarks.unfiledGuid,
                                                 url: "http://example.com/",
                                                 title: "a bookmark" });
  let bm2 = await PlacesUtils.bookmarks.insert({ parentGuid: PlacesUtils.bookmarks.unfiledGuid,
                                                 url: "http://example.org/path",
                                                 title: "another bookmark" });
  let bm3 = await PlacesUtils.bookmarks.insert({ parentGuid: PlacesUtils.bookmarks.unfiledGuid,
                                                 url: "http://example.net/",
                                                 title: "another bookmark" });
  let bm4 = await PlacesUtils.bookmarks.insert({ parentGuid: PlacesUtils.bookmarks.unfiledGuid,
                                                 url: "http://example.net/path",
                                                 title: "yet another bookmark" });
  checkBookmarkObject(bm1);
  checkBookmarkObject(bm2);
  checkBookmarkObject(bm3);
  checkBookmarkObject(bm4);

  // Add a tag to the most recent url to prove it doesn't get returned.
  PlacesUtils.tagging.tagURI(uri(bm4.url), ["Test Tag"]);

  // Add a separator.
  await PlacesUtils.bookmarks.insert({
    parentGuid: PlacesUtils.bookmarks.unfiledGuid,
    type: PlacesUtils.bookmarks.TYPE_SEPARATOR
  });

  // Add a query bookmark.
  let queryURL = `place:folder=${PlacesUtils.bookmarksMenuFolderId}&queryType=1`;
  let bm5 = await PlacesUtils.bookmarks.insert({ parentGuid: PlacesUtils.bookmarks.unfiledGuid,
                                                 url: queryURL,
                                                 title: "a test query" });
  checkBookmarkObject(bm5);

  // Verify that getRecent only returns actual bookmarks.
  let results = await PlacesUtils.bookmarks.getRecent(100);
  Assert.equal(results.length, 4, "The expected number of bookmarks was returned.");
  checkBookmarkObject(results[0]);
  Assert.deepEqual(bm4, results[0], "The first result is the expected bookmark.");
  checkBookmarkObject(results[1]);
  Assert.deepEqual(bm3, results[1], "The second result is the expected bookmark.");
  checkBookmarkObject(results[2]);
  Assert.deepEqual(bm2, results[2], "The third result is the expected bookmark.");
  checkBookmarkObject(results[3]);
  Assert.deepEqual(bm1, results[3], "The fourth result is the expected bookmark.");

  // Verify that getRecent utilizes the numberOfItems argument.
  results = await PlacesUtils.bookmarks.getRecent(1);
  Assert.equal(results.length, 1, "The expected number of bookmarks was returned.");
  checkBookmarkObject(results[0]);
  Assert.deepEqual(bm4, results[0], "The first result is the expected bookmark.");

  await PlacesUtils.bookmarks.eraseEverything();
});
