[Pkg-javascript-commits] [node-stream-http] 07/10: Use not well known port for test

Bastien Roucariès rouca at moszumanska.debian.org
Sun Aug 20 13:54:26 UTC 2017


This is an automated email from the git hooks/post-receive script.

rouca pushed a commit to branch master
in repository node-stream-http.

commit 7f8923aea62b90f9e19bd7d3ec5ff6c2542209aa
Author: Bastien ROUCARIÈS <roucaries.bastien at gmail.com>
Date:   Mon Jun 12 22:46:45 2017 +0200

    Use not well known port for test
    
    Avoid problem on buildd
    
    Forwarded: no
---
 test/node/http-browserify.js | 32 ++++++++++++++++++--------------
 1 file changed, 18 insertions(+), 14 deletions(-)

diff --git a/test/node/http-browserify.js b/test/node/http-browserify.js
index fda8512..dc4244a 100644
--- a/test/node/http-browserify.js
+++ b/test/node/http-browserify.js
@@ -3,7 +3,10 @@
 var test = require('tape')
 var url = require('url')
 
-var location = 'http://localhost:8081/foo/123'
+var port1 = 30801;
+var port2 = 30882;
+
+var location = 'http://localhost:'+port1+'/foo/123'
 
 var noop = function() {}
 global.location = url.parse(location)
@@ -22,20 +25,21 @@ test('Test simple url string', function(t) {
 	var request = http.get(testUrl, noop)
 
 	var resolved = url.resolve(location, request._opts.url)
-	t.equal(resolved, 'http://localhost:8081/api/foo', 'Url should be correct')
+	var url1 = 'http://localhost:'+port1.toString()+'/api/foo';
+	t.equal(resolved, url1, 'Url1 should be correct')
 	t.end()
 
 })
 
 test('Test full url object', function(t) {
 	var testUrl = {
-		host: "localhost:8081",
+		host: "localhost:"+port1,
 		hostname: "localhost",
-		href: "http://localhost:8081/api/foo?bar=baz",
+		href: "http://localhost:"+port1.toString()+"/api/foo?bar=baz",
 		method: "GET",
 		path: "/api/foo?bar=baz",
 		pathname: "/api/foo",
-		port: "8081",
+		port: port1.toString(),
 		protocol: "http:",
 		query: "bar=baz",
 		search: "?bar=baz",
@@ -45,7 +49,7 @@ test('Test full url object', function(t) {
 	var request = http.get(testUrl, noop)
 
 	var resolved = url.resolve(location, request._opts.url)
-	t.equal(resolved, 'http://localhost:8081/api/foo?bar=baz', 'Url should be correct')
+	t.equal(resolved, 'http://localhost:'+port1.toString()+'/api/foo?bar=baz', 'Url should be correct')
 	t.end()
 })
 
@@ -53,21 +57,21 @@ test('Test alt protocol', function(t) {
 	var params = {
 		protocol: "foo:",
 		hostname: "localhost",
-		port: "3000",
+		port: port2.toString(),
 		path: "/bar"
 	}
 
 	var request = http.get(params, noop)
 
 	var resolved = url.resolve(location, request._opts.url)
-	t.equal(resolved, 'foo://localhost:3000/bar', 'Url should be correct')
+	t.equal(resolved, 'foo://localhost:'+port2.toString()+'/bar', 'Url should be correct')
 	t.end()
 })
 
 test('Test page with \'file:\' protocol', function (t) {
 	var params = {
 		hostname: 'localhost',
-		port: 3000,
+		port: port2,
 		path: '/bar'
 	}
 
@@ -79,7 +83,7 @@ test('Test page with \'file:\' protocol', function (t) {
 	global.location = normalLocation // Reset the location
 
 	var resolved = url.resolve(fileLocation, request._opts.url)
-	t.equal(resolved, 'http://localhost:3000/bar', 'Url should be correct')
+	t.equal(resolved, 'http://localhost:'+port2+'/bar', 'Url should be correct')
 	t.end()
 })
 
@@ -88,7 +92,7 @@ test('Test string as parameters', function(t) {
 	var request = http.get(testUrl, noop)
 
 	var resolved = url.resolve(location, request._opts.url)
-	t.equal(resolved, 'http://localhost:8081/api/foo', 'Url should be correct')
+	t.equal(resolved, 'http://localhost:'+port1+'/api/foo', 'Url should be correct')
 	t.end()
 })
 
@@ -108,11 +112,11 @@ test('Test withCredentials param', function(t) {
 })
 
 test('Test ipv6 address', function(t) {
-	var testUrl = 'http://[::1]:80/foo'
+	var testUrl = 'http://[::1]:'+port1+'/foo'
 	var request = http.get(testUrl, noop)
 
 	var resolved = url.resolve(location, request._opts.url)
-	t.equal(resolved, 'http://[::1]:80/foo', 'Url should be correct')
+	t.equal(resolved, 'http://[::1]:'+port1+'/foo', 'Url should be correct')
 	t.end()
 })
 
@@ -121,7 +125,7 @@ test('Test relative path in url', function(t) {
 	var request = http.get(params, noop)
 
 	var resolved = url.resolve(location, request._opts.url)
-	t.equal(resolved, 'http://localhost:8081/foo/bar', 'Url should be correct')
+	t.equal(resolved, 'http://localhost:'+port1+'/foo/bar', 'Url should be correct')
 	t.end()
 })
 

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/node-stream-http.git



More information about the Pkg-javascript-commits mailing list