diff --git a/browser/components/places/tests/unit/test_placesTxn.js b/browser/components/places/tests/unit/test_placesTxn.js index 653ad5796f6..f37be49f03a 100644 --- a/browser/components/places/tests/unit/test_placesTxn.js +++ b/browser/components/places/tests/unit/test_placesTxn.js @@ -113,14 +113,14 @@ var observer = { }; bmsvc.addObserver(observer, false); -// get bookmarks root index -var root = bmsvc.bookmarksMenuFolder; - // index at which items should begin var bmStartIndex = 0; // main function run_test() { + // get bookmarks root index + var root = bmsvc.bookmarksMenuFolder; + const DESCRIPTION_ANNO = "bookmarkProperties/description"; var testDescription = "this is my test description"; var annotationService = Cc["@mozilla.org/browser/annotation-service;1"]. @@ -159,6 +159,7 @@ function run_test() { // Create to a folder var txn2a = ptSvc.createFolder("Folder", root, bmStartIndex); + ptSvc.doTransaction(txn2a); var fldrId = bmsvc.getChildFolder(root, "Folder"); var txn2b = ptSvc.createItem(uri("http://www.example2.com"), fldrId, bmStartIndex, "Testing1b"); ptSvc.doTransaction(txn2b); @@ -166,7 +167,7 @@ function run_test() { do_check_eq(observer._itemAddedId, b2); do_check_eq(observer._itemAddedIndex, bmStartIndex); do_check_true(bmsvc.isBookmarked(uri("http://www.example2.com"))); - txn2.undoTransaction(); + txn2b.undoTransaction(); do_check_eq(observer._itemRemovedId, b2); do_check_eq(observer._itemRemovedIndex, bmStartIndex); @@ -185,30 +186,30 @@ function run_test() { // Moving items between the same folder do_check_eq(observer._itemMovedId, bkmk1Id); do_check_eq(observer._itemMovedOldParent, root); - do_check_eq(observer._itemMovedOldIndex, 0); + do_check_eq(observer._itemMovedOldIndex, 1); do_check_eq(observer._itemMovedNewParent, root); - do_check_eq(observer._itemMovedNewIndex, 1); + do_check_eq(observer._itemMovedNewIndex, 2); txn3.undoTransaction(); do_check_eq(observer._itemMovedId, bkmk1Id); do_check_eq(observer._itemMovedOldParent, root); - do_check_eq(observer._itemMovedOldIndex, 1); + do_check_eq(observer._itemMovedOldIndex, 2); do_check_eq(observer._itemMovedNewParent, root); - do_check_eq(observer._itemMovedNewIndex, 0); + do_check_eq(observer._itemMovedNewIndex, 1); // Moving items between different folders var txn3b = ptSvc.moveItem(bkmk1Id, fldrId, -1); txn3b.doTransaction(); do_check_eq(observer._itemMovedId, bkmk1Id); do_check_eq(observer._itemMovedOldParent, root); - do_check_eq(observer._itemMovedOldIndex, 0); + do_check_eq(observer._itemMovedOldIndex, 1); do_check_eq(observer._itemMovedNewParent, fldrId); - do_check_eq(observer._itemMovedNewIndex, 2); + do_check_eq(observer._itemMovedNewIndex, 1); txn3.undoTransaction(); do_check_eq(observer._itemMovedId, bkmk1Id); do_check_eq(observer._itemMovedOldParent, fldrId); - do_check_eq(observer._itemMovedOldIndex, 2); + do_check_eq(observer._itemMovedOldIndex, 1); do_check_eq(observer._itemMovedNewParent, root); - do_check_eq(observer._itemMovedNewIndex, 0); + do_check_eq(observer._itemMovedNewIndex, 1); // Test Removing a Folder ptSvc.doTransaction(ptSvc.createFolder("Folder2", root, -1)); @@ -217,22 +218,22 @@ function run_test() { txn4.doTransaction(); do_check_eq(observer._itemRemovedId, fldrId2); do_check_eq(observer._itemRemovedFolder, root); - do_check_eq(observer._itemRemovedIndex, 2); + do_check_eq(observer._itemRemovedIndex, 3); txn4.undoTransaction(); do_check_eq(observer._itemAddedId, fldrId2); do_check_eq(observer._itemAddedParent, root); - do_check_eq(observer._itemAddedIndex, 2); + do_check_eq(observer._itemAddedIndex, 3); // Test removing an item var txn5 = ptSvc.removeItem(bkmk2Id); txn5.doTransaction(); do_check_eq(observer._itemRemovedId, bkmk2Id); do_check_eq(observer._itemRemovedFolder, root); - do_check_eq(observer._itemRemovedIndex, 0); + do_check_eq(observer._itemRemovedIndex, 1); txn5.undoTransaction(); do_check_eq(observer._itemAddedParent, root); - do_check_eq(observer._itemAddedIndex, 0); + do_check_eq(observer._itemAddedIndex, 1); // Test creating a separator var txn6 = ptSvc.createSeparator(root, 1);