From b236bc40093916bf3ba702d89b94bae05403cfc5 Mon Sep 17 00:00:00 2001 From: Dave Townsend Date: Thu, 12 May 2011 20:00:21 -0700 Subject: [PATCH] Bug 623134: Add logging for the browser_updatessl.js intermittent failure. rs=dolske --- toolkit/mozapps/extensions/test/browser/browser_updatessl.js | 4 ++-- toolkit/mozapps/extensions/test/browser/redirect.sjs | 1 + 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/toolkit/mozapps/extensions/test/browser/browser_updatessl.js b/toolkit/mozapps/extensions/test/browser/browser_updatessl.js index 36ed833e68c..eb7899819fb 100644 --- a/toolkit/mozapps/extensions/test/browser/browser_updatessl.js +++ b/toolkit/mozapps/extensions/test/browser/browser_updatessl.js @@ -49,7 +49,7 @@ function add_update_test(mainURL, redirectURL, expectedStatus) { } function run_update_tests(callback) { - function run_next_update_test(pos) { + function run_next_update_test() { if (gTests.length == 0) { callback(); return; @@ -71,7 +71,7 @@ function run_update_tests(callback) { AddonUpdateChecker.checkForUpdates("addon1@tests.mozilla.org", "extension", null, url, { onUpdateCheckComplete: function(updates) { - is(updates.length, 1); + is(updates.length, 1, "Should be the right number of results"); is(SUCCESS, expectedStatus, message); info("Update test ran in " + (Date.now() - gLast) + "ms"); run_next_update_test(); diff --git a/toolkit/mozapps/extensions/test/browser/redirect.sjs b/toolkit/mozapps/extensions/test/browser/redirect.sjs index 43fec90b5aa..8f9d1c08af6 100644 --- a/toolkit/mozapps/extensions/test/browser/redirect.sjs +++ b/toolkit/mozapps/extensions/test/browser/redirect.sjs @@ -1,4 +1,5 @@ function handleRequest(request, response) { + dump("*** Received redirect for " + request.queryString + "\n"); response.setStatusLine(request.httpVersion, 301, "Moved Permanently"); response.setHeader("Location", request.queryString, false); }