Skip to content

Commit

Permalink
more correct fix for node v18 tests: the issue was ipv4 vs ipv6 disag…
Browse files Browse the repository at this point in the history
…reement on default from server and client (as discussed earlier here #2380 (comment))
  • Loading branch information
trentm committed Oct 13, 2022
1 parent ce94960 commit 382be82
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/nextjs/a-nextjs-app/next.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -332,7 +332,7 @@ if (process.env.XXX_TEST_FILTER) {
* success.
*/
function waitForServerReady (t, cb) {
let sentinel = 20
let sentinel = 10

const pollForServerReady = () => {
const req = http.get(
Expand Down Expand Up @@ -524,7 +524,7 @@ tape.test('-- prod server tests --', { skip: false /* XXX */ }, suite => {
// XXX warning using `npm run start` directly with Docker.
nextServerProc = spawn(
'./node_modules/.bin/next',
['start'],
['start', '-H', 'localhost'],
{
shell: os.platform() === 'win32',
cwd: __dirname,
Expand Down Expand Up @@ -617,7 +617,7 @@ tape.test('-- dev server tests --', { skip: false /* XXX */ }, suite => {
// XXX warning using `npm run dev` directly with Docker.
nextServerProc = spawn(
'./node_modules/.bin/next',
['dev'],
['dev', '-H', 'localhost'],
{
shell: os.platform() === 'win32',
cwd: __dirname,
Expand Down

0 comments on commit 382be82

Please sign in to comment.