From 0acd00f8cadc160aaf380ddf4de6fb7dc1e7a9db Mon Sep 17 00:00:00 2001 From: Tim Kang Date: Thu, 6 Nov 2014 19:04:17 -0800 Subject: [PATCH] Expose hostname parameter of tiny-lr --- tasks/lib/livereload.js | 13 ++++++++----- test/fixtures/livereload/Gruntfile.js | 3 ++- test/tasks/livereload_test.js | 14 +++++++------- 3 files changed, 17 insertions(+), 13 deletions(-) diff --git a/tasks/lib/livereload.js b/tasks/lib/livereload.js index 5e8ab04..94b40f4 100644 --- a/tasks/lib/livereload.js +++ b/tasks/lib/livereload.js @@ -26,8 +26,11 @@ module.exports = function(grunt) { } else { options = _.defaults(options, defaults); } - if (servers[options.port]) { - this.server = servers[options.port]; + + var host = (options.hostname || '') + ':' + options.port; + + if (servers[host]) { + this.server = servers[host]; } else { this.server = tinylr(options); this.server.server.removeAllListeners('error'); @@ -39,11 +42,11 @@ module.exports = function(grunt) { } process.exit(1); }); - this.server.listen(options.port, function(err) { + this.server.listen(options.port, options.hostname, function (err) { if (err) { return grunt.fatal(err); } - grunt.log.verbose.writeln('Live reload server started on port: ' + options.port); + grunt.log.verbose.writeln('Live reload server started on ' + host); }); - servers[options.port] = this.server; + servers[host] = this.server; } } diff --git a/test/fixtures/livereload/Gruntfile.js b/test/fixtures/livereload/Gruntfile.js index 249f0c2..2e8bbc3 100644 --- a/test/fixtures/livereload/Gruntfile.js +++ b/test/fixtures/livereload/Gruntfile.js @@ -12,11 +12,12 @@ module.exports = function(grunt) { files: ['lib/*.js'], tasks: ['before'], }, - customport: { + customhost: { files: ['lib/*.js'], tasks: ['before'], options: { livereload: { + hostname: 'localhost', port: 8675, }, }, diff --git a/test/tasks/livereload_test.js b/test/tasks/livereload_test.js index 8beb82c..4a0b13f 100644 --- a/test/tasks/livereload_test.js +++ b/test/tasks/livereload_test.js @@ -56,18 +56,18 @@ exports.livereload = { result = helper.unixify(result); helper.verboseLog(result); test.ok(result.indexOf('I ran before livereload.') !== -1, 'task should have ran before live reload.'); - test.ok(result.indexOf('Live reload server started on port: 35729') !== -1, 'live reload server should have been started on port 35729.'); + test.ok(result.indexOf('Live reload server started on :35729') !== -1, 'live reload server should have been started on port 35729.'); test.ok(result.indexOf('Live reloading lib/one.js...') !== -1, 'live reload should have triggered on lib/one.js'); resultData = JSON.parse(resultData); test.equal(resultData.tinylr, 'Welcome', 'tinylr server should have welcomed you.'); test.done(); }); }, - customport: function(test) { + customhost: function(test) { test.expect(4); var resultData = ''; var cwd = path.resolve(fixtures, 'livereload'); - var assertWatch = helper.assertTask(['watch:customport', '-v'], {cwd: cwd}); + var assertWatch = helper.assertTask(['watch:customhost', '-v'], {cwd: cwd}); assertWatch([function() { request(8675, function(data) { resultData += data; @@ -77,7 +77,7 @@ exports.livereload = { result = helper.unixify(result); helper.verboseLog(result); test.ok(result.indexOf('I ran before livereload.') !== -1, 'task should have ran before live reload.'); - test.ok(result.indexOf('Live reload server started on port: 8675') !== -1, 'live reload server should have been started on port 35729.'); + test.ok(result.indexOf('Live reload server started on localhost:8675') !== -1, 'live reload server should have been started on localhost:8675.'); test.ok(result.indexOf('Live reloading lib/one.js...') !== -1, 'live reload should have triggered on lib/one.js'); resultData = JSON.parse(resultData); test.equal(resultData.tinylr, 'Welcome', 'tinylr server should have welcomed you.'); @@ -116,7 +116,7 @@ exports.livereload = { result = helper.unixify(result); helper.verboseLog(result); test.ok(result.indexOf('I ran before livereload.') !== -1, 'task should have ran before live reload.'); - test.ok(result.indexOf('Live reload server started on port: 9876') !== -1, 'live reload server should have been started on port 9876.'); + test.ok(result.indexOf('Live reload server started on :9876') !== -1, 'live reload server should have been started on port 9876.'); test.ok(/Live reloading (lib\/one\.js, lib\/two.js|lib\/two.js, lib\/one.js)\.\.\./.test(result), 'live reload should have triggered on lib/one.js and lib/two.js'); resultData = JSON.parse(resultData); test.equal(resultData.tinylr, 'Welcome', 'tinylr server should have welcomed you.'); @@ -137,7 +137,7 @@ exports.livereload = { result = helper.unixify(result); helper.verboseLog(result); test.ok(result.indexOf('I ran before livereload.') !== -1, 'task should have ran before live reload.'); - test.ok(result.indexOf('Live reload server started on port: 1337') !== -1, 'live reload server should have been started on port 1337.'); + test.ok(result.indexOf('Live reload server started on :1337') !== -1, 'live reload server should have been started on port 1337.'); test.ok(result.indexOf('Live reloading lib/one.js...') !== -1, 'live reload should have triggered on lib/one.js'); resultData = JSON.parse(resultData); test.equal(resultData.tinylr, 'Welcome', 'tinylr server should have welcomed you.'); @@ -157,7 +157,7 @@ exports.livereload = { }], function(result) { result = helper.unixify(result); helper.verboseLog(result); - test.ok(result.indexOf('Live reload server started on port: 35729') !== -1, 'live reload server should have been started on port 35729.'); + test.ok(result.indexOf('Live reload server started on :35729') !== -1, 'live reload server should have been started on port 35729.'); test.ok(result.indexOf('Live reloading lib/one.js...') !== -1, 'live reload should have triggered on lib/one.js'); resultData = JSON.parse(resultData); test.equal(resultData.tinylr, 'Welcome', 'tinylr server should have welcomed you.');