From 03ea7e946d8167ee41dd5f4a457e7c071e0d6f31 Mon Sep 17 00:00:00 2001 From: Justin Dolske Date: Mon, 2 Aug 2010 16:02:19 -0700 Subject: [PATCH] Bug 583885 - Fix FUEL browser_Bookmarks.js to not make bad assumption about annotations. r=sdwilsh --- browser/fuel/test/browser_Bookmarks.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/browser/fuel/test/browser_Bookmarks.js b/browser/fuel/test/browser_Bookmarks.js index 552c9572b7d..0aa0ab28d92 100644 --- a/browser/fuel/test/browser_Bookmarks.js +++ b/browser/fuel/test/browser_Bookmarks.js @@ -135,8 +135,10 @@ function test() { // test names array - NOTE: "bookmarkProperties/description" is an annotation too var names = testBookmark.annotations.names; - is(names[1], "testing/bookmark/string", "Checking contents of annotation names array"); - is(names.length, 4, "Checking the annotation names array after adding 3 annotations"); + ok(names.some(function (f) f == "bookmarkProperties/description"), "Checking for description annotation"); + ok(names.some(function (f) f == "testing/bookmark/string"), "Checking for string test annotation"); + ok(names.some(function (f) f == "testing/bookmark/int"), "Checking for int test annotation"); + ok(names.some(function (f) f == "testing/bookmark/double"), "Checking for double test annotation"); // test adding a separator var testSeparator = testFolder.addSeparator();