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

flaky: unit test runner sometimes crashes on Windows #4373

Closed
bartlomieju opened this issue Mar 14, 2020 · 8 comments · Fixed by #4395
Closed

flaky: unit test runner sometimes crashes on Windows #4373

bartlomieju opened this issue Mar 14, 2020 · 8 comments · Fixed by #4395
Labels
bug Something isn't working correctly

Comments

@bartlomieju
Copy link
Member

Running tests for: --allow-run --allow-env
running 1 tests
error: Uncaught ConnectionReset: An existing connection was forcibly closed by the remote host. (os error 10054) 
► $deno$/ops/dispatch_minimal.ts:63:11
    at ConnectionReset ($deno$/errors.ts:93:5)
    at unwrapResponse ($deno$/ops/dispatch_minimal.ts:63:11)
    at sendAsyncMinimal ($deno$/ops/dispatch_minimal.ts:106:10)

https://github.com/denoland/deno/pull/4349/checks?check_run_id=507856283

It looks like this happens during running of envCaseInsensitive test, but happens only sometimes. Error suggests that TCP connection between parent and worker process is closed. I'm not sure what cases this failure.

@bartlomieju bartlomieju added the bug Something isn't working correctly label Mar 14, 2020
@bartlomieju
Copy link
Member Author

As a reference each subprocess that is spawned connects to TCP socket at 127.0.0.1:4100

@ry
Copy link
Member

ry commented Mar 16, 2020

I hit this recently (unfortunately I can't find the link)

Let's give this two days - if we can't figure out the cause, then I think we need to revert the test runner changes unfortunately.

@ry
Copy link
Member

ry commented Mar 16, 2020

@ry ry reopened this Mar 16, 2020
@ry
Copy link
Member

ry commented Mar 17, 2020

@dubiousjim
Copy link
Contributor

I've encountered this a bunch of times over the past couple of days.

@ry
Copy link
Member

ry commented Mar 17, 2020

@ry ry changed the title unit test runner sometimes crashes on Windows flaky: unit test runner sometimes crashes on Windows Mar 17, 2020
@ry
Copy link
Member

ry commented Mar 18, 2020

and again https://github.com/denoland/deno/pull/4416/checks?check_run_id=515520636

(i'll stop updating.. it seems to be happening regularly)

@ry
Copy link
Member

ry commented Mar 18, 2020

I caught some debug output in https://github.com/denoland/deno/runs/516691695

msg {"kind":"testStart","name":"runStderrOutput"}
test runStderrOutput ... msg {"kind":"testEnd","result":{"name":"runStderrOutput","duration":31,"status":"passed"}}
ok (31ms)
msg {"kind":"testStart","name":"runEnv"}
test runEnv ... msg {"kind":"testEnd","result":{"name":"runEnv","duration":31,"status":"passed"}}
ok (31ms)
error: Uncaught Error: Worker runner didn't report end
msg {"kind":"testStart","name":"runClose"}
► file:///D:/a/deno/deno/cli/js/tests/unit_test_runner.ts:166:11
test runClose ... msg {"kind":"testEnd","result":{"name":"runClose","duration":0,"status":"passed"}}

ok (0ms)
166     throw new Error("Worker runner didn't report end");
msg {"kind":"testStart","name":"signalPromiseTest"}
              ^
test signalPromiseTest ... msg {"kind":"testEnd","result":{"name":"signalPromiseTest","duration":0,"status":"skipped"}}

SKIPPED (0ms)
    at runTestsForPermissionSet (file:///D:/a/deno/deno/cli/js/tests/unit_test_runner.ts:166:11)
msg {"kind":"testStart","name":"signalShorthandsTest"}
test signalShorthandsTest ... msg {"kind":"testEnd","result":{"name":"signalShorthandsTest","duration":0,"status":"skipped"}}
SKIPPED (0ms)
test runner master ConnectionReset
test js_unit_tests ... FAILED

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working correctly
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants