From 94f72b748ed18f62cf033f77cc021beb569e7a6f Mon Sep 17 00:00:00 2001 From: Meghan Denny Date: Mon, 28 Apr 2025 20:24:19 -0700 Subject: [PATCH] test: these work already --- .../test-http-client-response-timeout.js | 14 ++++ .../test/parallel/test-http-unix-socket.js | 77 +++++++++++++++++++ .../test-http-url.parse-https.request.js | 61 +++++++++++++++ 3 files changed, 152 insertions(+) create mode 100644 test/js/node/test/parallel/test-http-client-response-timeout.js create mode 100644 test/js/node/test/parallel/test-http-unix-socket.js create mode 100644 test/js/node/test/parallel/test-http-url.parse-https.request.js diff --git a/test/js/node/test/parallel/test-http-client-response-timeout.js b/test/js/node/test/parallel/test-http-client-response-timeout.js new file mode 100644 index 0000000000..7e44d83a83 --- /dev/null +++ b/test/js/node/test/parallel/test-http-client-response-timeout.js @@ -0,0 +1,14 @@ +'use strict'; +const common = require('../common'); +const http = require('http'); + +const server = http.createServer((req, res) => res.flushHeaders()); + +server.listen(common.mustCall(() => { + const req = + http.get({ port: server.address().port }, common.mustCall((res) => { + res.on('timeout', common.mustCall(() => req.destroy())); + res.setTimeout(1); + server.close(); + })); +})); diff --git a/test/js/node/test/parallel/test-http-unix-socket.js b/test/js/node/test/parallel/test-http-unix-socket.js new file mode 100644 index 0000000000..f8362d6183 --- /dev/null +++ b/test/js/node/test/parallel/test-http-unix-socket.js @@ -0,0 +1,77 @@ +// Copyright Joyent, Inc. and other Node contributors. +// +// Permission is hereby granted, free of charge, to any person obtaining a +// copy of this software and associated documentation files (the +// "Software"), to deal in the Software without restriction, including +// without limitation the rights to use, copy, modify, merge, publish, +// distribute, sublicense, and/or sell copies of the Software, and to permit +// persons to whom the Software is furnished to do so, subject to the +// following conditions: +// +// The above copyright notice and this permission notice shall be included +// in all copies or substantial portions of the Software. +// +// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS +// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF +// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN +// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, +// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR +// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE +// USE OR OTHER DEALINGS IN THE SOFTWARE. + +'use strict'; +const common = require('../common'); +const assert = require('assert'); +const http = require('http'); + +const server = http.createServer(function(req, res) { + res.writeHead(200, { + 'Content-Type': 'text/plain', + 'Connection': 'close' + }); + res.write('hello '); + res.write('world\n'); + res.end(); +}); + +const tmpdir = require('../common/tmpdir'); +tmpdir.refresh(); + +server.listen(common.PIPE, common.mustCall(function() { + + const options = { + socketPath: common.PIPE, + path: '/' + }; + + const req = http.get(options, common.mustCall(function(res) { + assert.strictEqual(res.statusCode, 200); + assert.strictEqual(res.headers['content-type'], 'text/plain'); + + res.body = ''; + res.setEncoding('utf8'); + + res.on('data', function(chunk) { + res.body += chunk; + }); + + res.on('end', common.mustCall(function() { + assert.strictEqual(res.body, 'hello world\n'); + server.close(common.mustCall(function(error) { + assert.strictEqual(error, undefined); + server.close(common.expectsError({ + code: 'ERR_SERVER_NOT_RUNNING', + message: 'Server is not running.', + name: 'Error' + })); + })); + })); + })); + + req.on('error', function(e) { + assert.fail(e); + }); + + req.end(); + +})); diff --git a/test/js/node/test/parallel/test-http-url.parse-https.request.js b/test/js/node/test/parallel/test-http-url.parse-https.request.js new file mode 100644 index 0000000000..2f18899176 --- /dev/null +++ b/test/js/node/test/parallel/test-http-url.parse-https.request.js @@ -0,0 +1,61 @@ +// Copyright Joyent, Inc. and other Node contributors. +// +// Permission is hereby granted, free of charge, to any person obtaining a +// copy of this software and associated documentation files (the +// "Software"), to deal in the Software without restriction, including +// without limitation the rights to use, copy, modify, merge, publish, +// distribute, sublicense, and/or sell copies of the Software, and to permit +// persons to whom the Software is furnished to do so, subject to the +// following conditions: +// +// The above copyright notice and this permission notice shall be included +// in all copies or substantial portions of the Software. +// +// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS +// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF +// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN +// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, +// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR +// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE +// USE OR OTHER DEALINGS IN THE SOFTWARE. + +'use strict'; +const common = require('../common'); +if (!common.hasCrypto) + common.skip('missing crypto'); +const { readKey } = require('../common/fixtures'); + +const assert = require('assert'); +const https = require('https'); +const url = require('url'); + +// https options +const httpsOptions = { + key: readKey('agent1-key.pem'), + cert: readKey('agent1-cert.pem') +}; + +function check(request) { + // Assert that I'm https + assert.ok(request.socket._secureEstablished); +} + +const server = https.createServer(httpsOptions, function(request, response) { + // Run the check function + check(request); + response.writeHead(200, {}); + response.end('ok'); + server.close(); +}); + +server.listen(0, function() { + const testURL = url.parse(`https://localhost:${this.address().port}`); + testURL.rejectUnauthorized = false; + + // make the request + const clientRequest = https.request(testURL); + // Since there is a little magic with the agent + // make sure that the request uses the https.Agent + assert.ok(clientRequest.agent instanceof https.Agent); + clientRequest.end(); +});