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.

Sign up for GitHub

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

Jump to bottom

flaky test.parallel/test-http-client-response-timeout #43680

Closed
mhdawson opened this issue Jul 4, 2022 · 1 comment · Fixed by #44145
Closed

flaky test.parallel/test-http-client-response-timeout #43680

mhdawson opened this issue Jul 4, 2022 · 1 comment · Fixed by #44145
Labels
flaky-test Issues and PRs related to the tests with unstable failures on the CI.

Comments

@mhdawson
Copy link
Member

mhdawson commented Jul 4, 2022

Test

test-http-client-response-timeout

Platform

osx11

Console output

Mismatched <anonymous> function calls. Expected exactly 1, actual 0.
    at Proxy.mustCall (/Users/iojs/build/workspace/node-test-commit-osx-arm/nodes/osx11/test/common/index.js:396:10)
    at ClientRequest.<anonymous> (/Users/iojs/build/workspace/node-test-commit-osx-arm/nodes/osx11/test/parallel/test-http-client-response-timeout.js:10:32)
    at ClientRequest.<anonymous> (/Users/iojs/build/workspace/node-test-commit-osx-arm/nodes/osx11/test/common/index.js:438:15)
    at Object.onceWrapper (node:events:652:26)
    at ClientRequest.emit (node:events:537:28)
    at HTTPParser.parserOnIncomingClient [as onIncoming] (node:_http_client:659:27)
    at HTTPParser.parserOnHeadersComplete (node:_http_common:117:17)
    at Socket.socketOnData (node:_http_client:512:22)
    at Socket.emit (node:events:537:28)

Build links

Additional information

No response
@mhdawson mhdawson added the flaky-test Issues and PRs related to the tests with unstable failures on the CI. label Jul 4, 2022
@tniessen
Copy link
Member

Another failure in #43756

Mismatched <anonymous> function calls. Expected exactly 1, actual 0.
    at Proxy.mustCall (/Users/iojs/build/workspace/node-test-commit-osx-arm/nodes/osx11/test/common/index.js:396:10)
    at ClientRequest.<anonymous> (/Users/iojs/build/workspace/node-test-commit-osx-arm/nodes/osx11/test/parallel/test-http-client-response-timeout.js:10:32)
    at ClientRequest.<anonymous> (/Users/iojs/build/workspace/node-test-commit-osx-arm/nodes/osx11/test/common/index.js:438:15)
    at Object.onceWrapper (node:events:628:26)
    at ClientRequest.emit (node:events:513:28)
    at HTTPParser.parserOnIncomingClient [as onIncoming] (node:_http_client:674:27)
    at HTTPParser.parserOnHeadersComplete (node:_http_common:117:17)
    at Socket.socketOnData (node:_http_client:521:22)
    at Socket.emit (node:events:513:28)

https://ci.nodejs.org/job/node-test-commit-osx-arm/6520/nodes=osx11/testReport/junit/(root)/test/parallel_test_http_client_response_timeout/

tniessen added a commit to tniessen/node that referenced this issue Jul 12, 2022
@tniessen
test: mark test-http-client-response-timeout flaky
93f3372 
Refs: nodejs#43680
@tniessen tniessen mentioned this issue Jul 12, 2022
Merged
nodejs-github-bot pushed a commit that referenced this issue Jul 12, 2022
@tniessen
test: mark test-http-client-response-timeout flaky
a6e3d3b 
Refs: #43680

PR-URL: #43792
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Beth Griggs <bgriggs@redhat.com>
danielleadams pushed a commit that referenced this issue Jul 26, 2022
@tniessen @danielleadams
test: mark test-http-client-response-timeout flaky
a9ecba2 
Refs: #43680

PR-URL: #43792
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Beth Griggs <bgriggs@redhat.com>
mmomtchev added a commit to mmomtchev/node that referenced this issue Jul 29, 2022
@mmomtchev
test: do not race server.close with timeout
3b3cd1f 
Refs: nodejs#43680
targos pushed a commit that referenced this issue Jul 31, 2022
@tniessen @targos
test: mark test-http-client-response-timeout flaky
087fc8d 
Refs: #43680

PR-URL: #43792
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Beth Griggs <bgriggs@redhat.com>
lpinca added a commit to lpinca/node that referenced this issue Aug 5, 2022
@lpinca
test: remove test-http-client-response-timeout flaky designation
474a071 
The issue was fixed by nodejs#43890.

Fixes: nodejs#43680
@lpinca lpinca mentioned this issue Aug 5, 2022
Merged
nodejs-github-bot pushed a commit that referenced this issue Aug 7, 2022
@lpinca
test: remove test-http-client-response-timeout flaky designation
b2f4a55 
The issue was fixed by #43890.

Fixes: #43680
PR-URL: #44145
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: theanarkh <theratliter@gmail.com>
danielleadams pushed a commit that referenced this issue Aug 16, 2022
@lpinca @danielleadams
test: remove test-http-client-response-timeout flaky designation
c75c478 
The issue was fixed by #43890.

Fixes: #43680
PR-URL: #44145
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: theanarkh <theratliter@gmail.com>
ruyadorno pushed a commit that referenced this issue Aug 23, 2022
@lpinca @ruyadorno
test: remove test-http-client-response-timeout flaky designation
e5113fa 
The issue was fixed by #43890.

Fixes: #43680
PR-URL: #44145
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: theanarkh <theratliter@gmail.com>
targos pushed a commit that referenced this issue Sep 5, 2022
@lpinca @targos
test: remove test-http-client-response-timeout flaky designation
67fde45 
The issue was fixed by #43890.

Fixes: #43680
PR-URL: #44145
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: theanarkh <theratliter@gmail.com>
Fyko pushed a commit to Fyko/node that referenced this issue Sep 15, 2022
@lpinca @Fyko
test: remove test-http-client-response-timeout flaky designation
21c9427 
The issue was fixed by nodejs#43890.

Fixes: nodejs#43680
PR-URL: nodejs#44145
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: theanarkh <theratliter@gmail.com>
guangwong pushed a commit to noslate-project/node that referenced this issue Oct 10, 2022
@tniessen @guangwong
test: mark test-http-client-response-timeout flaky
d8df059 
Refs: nodejs/node#43680

PR-URL: nodejs/node#43792
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Beth Griggs <bgriggs@redhat.com>
juanarbol pushed a commit that referenced this issue Oct 10, 2022
@lpinca @juanarbol
test: remove test-http-client-response-timeout flaky designation
3440a42 
The issue was fixed by #43890.

Fixes: #43680
PR-URL: #44145
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: theanarkh <theratliter@gmail.com>
juanarbol pushed a commit that referenced this issue Oct 11, 2022
@lpinca @juanarbol
test: remove test-http-client-response-timeout flaky designation
4cfa9ba 
The issue was fixed by #43890.

Fixes: #43680
PR-URL: #44145
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: theanarkh <theratliter@gmail.com>
guangwong pushed a commit to noslate-project/node that referenced this issue Jan 3, 2023
@lpinca @guangwong
test: remove test-http-client-response-timeout flaky designation
dd8921f 
The issue was fixed by nodejs/node#43890.

Fixes: nodejs/node#43680
PR-URL: nodejs/node#44145
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: theanarkh <theratliter@gmail.com>
guangwong pushed a commit to noslate-project/node that referenced this issue Jan 3, 2023
@lpinca @guangwong
test: remove test-http-client-response-timeout flaky designation
b3ff800 
The issue was fixed by nodejs/node#43890.

Fixes: nodejs/node#43680
PR-URL: nodejs/node#44145
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: theanarkh <theratliter@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
flaky-test Issues and PRs related to the tests with unstable failures on the CI.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test: remove test-http-client-response-timeout flaky designation lpinca/node
2 participants
@tniessen @mhdawson