Skip to content

Commit

Permalink
test: remove the legacy url parser function
Browse files Browse the repository at this point in the history
PR-URL: nodejs#42656
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Akhil Marsonya <akhil.marsonya27@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
cola119 committed Apr 14, 2022
1 parent 1285ac1 commit 24adba6
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions test/common/inspector-helper.js
Expand Up @@ -5,8 +5,7 @@ const fs = require('fs');
const http = require('http');
const fixtures = require('../common/fixtures');
const { spawn } = require('child_process');
const { parse: parseURL } = require('url');
const { pathToFileURL } = require('url');
const { URL, pathToFileURL } = require('url');
const { EventEmitter } = require('events');

const _MAINSCRIPT = fixtures.path('loop.js');
Expand Down Expand Up @@ -418,7 +417,7 @@ class NodeInstance extends EventEmitter {
return http.get({
port,
family: 4,
path: parseURL(devtoolsUrl).path,
path: new URL(devtoolsUrl).pathname,
headers: {
'Connection': 'Upgrade',
'Upgrade': 'websocket',
Expand Down

0 comments on commit 24adba6

Please sign in to comment.