From 3679ce9f843c52676b9b54a813a953e1c1b737b3 Mon Sep 17 00:00:00 2001 From: Mark Hammond Date: Tue, 11 Aug 2015 10:58:54 +1000 Subject: [PATCH] Bug 1185629 - ensure Sync tests arrange for an auto-selected allowed port for its test servers. r=oeger --- services/sync/tests/unit/head_http_server.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/services/sync/tests/unit/head_http_server.js b/services/sync/tests/unit/head_http_server.js index c917c4988c2..82fb89e9034 100644 --- a/services/sync/tests/unit/head_http_server.js +++ b/services/sync/tests/unit/head_http_server.js @@ -545,13 +545,13 @@ SyncServer.prototype = { * Start the SyncServer's underlying HTTP server. * * @param port - * The numeric port on which to start. A falsy value implies the - * default, a randomly chosen port. + * The numeric port on which to start. -1 implies the default, a + * randomly chosen port. * @param cb * A callback function (of no arguments) which is invoked after * startup. */ - start: function start(port, cb) { + start: function start(port = -1, cb) { if (this.started) { this._log.warn("Warning: server already started on " + this.port); return;