From 2313c1930d62f4562d31b7bcc5ca7439ade095c1 Mon Sep 17 00:00:00 2001 From: Dan Stillman Date: Tue, 5 May 2015 02:40:01 -0400 Subject: [PATCH] Get selected item ids directly in collection tests --- test/tests/itemTreeViewTest.js | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/test/tests/itemTreeViewTest.js b/test/tests/itemTreeViewTest.js index ca7acbab0..41f1da7a6 100644 --- a/test/tests/itemTreeViewTest.js +++ b/test/tests/itemTreeViewTest.js @@ -20,13 +20,13 @@ describe("Zotero.ItemTreeView", function() { */ it("should return if item is already selected", function* () { yield itemsView.selectItem(existingItemID); - var selected = itemsView.getSelectedItems(); + var selected = itemsView.getSelectedItems(true); assert.lengthOf(selected, 1); - assert.equal(selected[0].id, existingItemID); + assert.equal(selected[0], existingItemID); yield itemsView.selectItem(existingItemID); - selected = itemsView.getSelectedItems(); + selected = itemsView.getSelectedItems(true); assert.lengthOf(selected, 1); - assert.equal(selected[0].id, existingItemID); + assert.equal(selected[0], existingItemID); }); }) @@ -48,9 +48,9 @@ describe("Zotero.ItemTreeView", function() { it("shouldn't select a new item if skipNotifier is passed", function* () { // Select existing item yield itemsView.selectItem(existingItemID); - var selected = itemsView.getSelectedItems(); + var selected = itemsView.getSelectedItems(true); assert.lengthOf(selected, 1); - assert.equal(selected[0].id, existingItemID); + assert.equal(selected[0], existingItemID); // Create item with skipNotifier flag var item = new Zotero.Item('book'); @@ -59,17 +59,17 @@ describe("Zotero.ItemTreeView", function() { }); // Existing item should still be selected - selected = itemsView.getSelectedItems(); + selected = itemsView.getSelectedItems(true); assert.lengthOf(selected, 1); - assert.equal(selected[0].id, existingItemID); + assert.equal(selected[0], existingItemID); }); it("shouldn't select a new item if skipSelect is passed", function* () { // Select existing item yield itemsView.selectItem(existingItemID); - var selected = itemsView.getSelectedItems(); + var selected = itemsView.getSelectedItems(true); assert.lengthOf(selected, 1); - assert.equal(selected[0].id, existingItemID); + assert.equal(selected[0], existingItemID); // Create item with skipSelect flag var item = new Zotero.Item('book');