www

commit 91b0acac1e495755dc29683d1e37ce9340e3a426
parent 48260e22a9e64b32f4dc650d653572a93bd11366
Author: Dan Stillman <dstillman@zotero.org>
Date:   Tue, 19 Jul 2016 20:31:06 -0400

Fix potential error in libraryTreeView::#_removeRow() test

Diffstat:
M
test/tests/libraryTreeViewTest.js
 | 
8
++++----

1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/test/tests/libraryTreeViewTest.js b/test/tests/libraryTreeViewTest.js
@@ -31,15 +31,15 @@ describe("Zotero.LibraryTreeView", function() {
 		it("should remove the last row", function* () {
 			var collection = yield createDataObject('collection');
 			yield waitForItemsLoad(win);
-			var item1 = yield createDataObject('item', { collections: [collection.id] });
-			var item2 = yield createDataObject('item', { collections: [collection.id] });
+			yield createDataObject('item', { collections: [collection.id] });
+			yield createDataObject('item', { collections: [collection.id] });
 			
 			var view = zp.itemsView;
-			assert.equal(view.getRowIndexByID(item2.id), 1);
+			var treeViewID = view.getRow(1).id;
 			zp.itemsView._removeRow(1);
 			
 			assert.equal(view.rowCount, 1);
-			assert.isFalse(view.getRowIndexByID(item2.id));
+			assert.isFalse(view.getRowIndexByID(treeViewID));
 		});
 	});
 })