Bustage fix part 1

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);