Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"docker exec wc -c" test fails with Node.js v16.13.2 #1

Closed
jwhitlock opened this issue Jan 31, 2022 · 2 comments
Closed

"docker exec wc -c" test fails with Node.js v16.13.2 #1

jwhitlock opened this issue Jan 31, 2022 · 2 comments

Comments

@jwhitlock
Copy link
Contributor

This test take 45ms on Node.js v14.17.5, but fails with a timeout on Node.js v16.13.2:

test('docker exec wc -c', async () => {

Test output:

    1) docker exec wc -c
    ✓ cat on server (512ms)
    ✓ exit code (504ms)
    ✓ connection limit
    ✓ automatic pausing (1008ms)
    ✓ session count
    ✓ resize (3513ms)


  6 passing (48s)
  1 failing

  1) trying client
       docker exec wc -c:
     Error: Timeout of 40000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/Users/john/src/docker-exec-websocket-client/test/test.js)
      at listOnTimeout (node:internal/timers:557:17)
      at processTimers (node:internal/timers:500:7)
@jwhitlock
Copy link
Contributor Author

I get similar results on https://github.com/taskcluster/docker-exec-websocket-server - the test docker exec wc -c passes on Node.js 14.17.15 but fails on 16.13.2. And, there are pending updates, including bumping ws from 7.1.2 to 7.4.6.

@jwhitlock
Copy link
Contributor Author

Fixed with the taskcluster/docker-exec-websocket-server update and PR #2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant