Regression wait-on

[akoidan]

This worked perfectly on version start-server-and-test@1.10.8. After updating to version start-server-and-test@1.11.4 this package stopped working.

Platform MacOS@10.15.3
Node v14.12.0

Reproduce

yarn init
yarn add servor@4.0.2
yarn add start-server-and-test@1.11.4
mkdir dist; echo "<div>Hello</div>" > dist/index.html
yarn run start-server-and-test 'servor dist index.html 8080' http://localhost:8080 'echo done'

expected behavior:
done on stdout and process' exit.

actual behavior:

yarn run v1.22.5
$ /private/tmp/lol/node_modules/.bin/start-server-and-test 'servor dist index.html 8080' http://localhost:8080 'echo done'
1: starting server using command "servor dist index.html 8080"
and when url "[ 'http//localhost:8080' ]" is responding with HTTP status code 200
running tests using command "echo done"


  🗂  Serving:	/private/tmp/lol/dist

  🏡 Local:	http://localhost:8080
  📡 Network:	http://192.168.1.144:8080
  📡 Network:	http://169.254.35.22:8080

Error: Timed out waiting for: http//localhost:8080
    at MergeMapSubscriber.project (/private/tmp/lol/node_modules/wait-on/lib/wait-on.js:130:25)
    at MergeMapSubscriber._tryNext (/private/tmp/lol/node_modules/rxjs/internal/operators/mergeMap.js:67:27)
    at MergeMapSubscriber._next (/private/tmp/lol/node_modules/rxjs/internal/operators/mergeMap.js:57:18)
    at MergeMapSubscriber.Subscriber.next (/private/tmp/lol/node_modules/rxjs/internal/Subscriber.js:66:18)
    at AsyncAction.dispatch [as work] (/private/tmp/lol/node_modules/rxjs/internal/observable/timer.js:31:16)
    at AsyncAction._execute (/private/tmp/lol/node_modules/rxjs/internal/scheduler/AsyncAction.js:71:18)
    at AsyncAction.execute (/private/tmp/lol/node_modules/rxjs/internal/scheduler/AsyncAction.js:59:26)
    at AsyncScheduler.flush (/private/tmp/lol/node_modules/rxjs/internal/scheduler/AsyncScheduler.js:52:32)
    at listOnTimeout (internal/timers.js:554:17)
    at processTimers (internal/timers.js:497:7)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

The server meanwhile was up and running

[carhartl]

I'm seeing the same with Node Node.js v17.0.1. and a Vite application..

[herrmannplatz]

Can confirm what @carhartl said. Works as expected on Node v16, but times out on v17

[rtrembecky]

I can double-confirm here, having issues with node 17, where 16 works. Have you guys figured out some workaround or something?

[aarowman]

Any updates on this?

We are seeing an issue too, using version 1.14 here, saw it with Node v16.

After start-server-and-test times out, the process does not exit, just hangs there indefinitely.