Deno•3y ago

Leaking operation op_http_close in tests

I tried to update my repositories from Deno 1.38.1 to 1.38.4. After the update my test keeps failing with the error message "1 async operation to op_http_close was started in this test, but never completed.". I tried every version in between but had to revert to 1.38.1.

Failing CI run: https://github.com/sgohlke/deno-web/actions/runs/7092893428/job/19305155074
Revert to Deno 1.38.1 is green: https://github.com/sgohlke/deno-web/actions/runs/7092893428

Failing test: https://github.com/sgohlke/deno-web/blob/main/response/responseFunctions_test.ts#L34
I use AbortController to send an AbortSignal to the server. I tried to add the finished function and console.log like in the example on https://deno.land/api@v1.38.4?s=Deno.serve , it did not help. I also tried to store the server in a const and call "await server.shutdown()", but this results in TS error "TS2339 [ERROR]: Property 'shutdown' does not exist on type 'HttpServer'."

Does anyone have an idea how to fix this issue?

GitHub

Use action workflow · sgohlke/deno-web@59a6c5c

Webserver and helper functions in Deno. Contribute to sgohlke/deno-web development by creating an account on GitHub.

GitHub

Use action workflow · sgohlke/deno-web@59a6c5c

Webserver and helper functions in Deno. Contribute to sgohlke/deno-web development by creating an account on GitHub.

Deno

Deno.serve | Runtime APIs | Deno

Serves HTTP requests with the given handler.

GitHub

deno-web/response/responseFunctions_test.ts at main · sgohlke/deno-...

Webserver and helper functions in Deno. Contribute to sgohlke/deno-web development by creating an account on GitHub.

Deno AI Helper•12/4/23, 11:37 PM

I wasn't able to find the answer to your question in the official Deno documentation. However, based on the information you've provided in your message, I can offer some general advice that might help you address the issue.

From what you've described, it seems like there might be an asynchronous operation related to that is not being properly awaited or closed in your tests. This can happen if the server is still running or if there are pending requests that have not been handled when the test completes.

Regarding the error you're encountering when trying to call , it's possible that the type does not have a method in the version of Deno you are using. This could be due to a change in the API or a discrepancy between the documentation and the actual implementation.

Here are a few things you could check or try:

Ensure that you are properly closing the server after the test. This might involve calling a method to stop the server or ensuring that all connections are closed.
If you're using an to signal the server to stop, make sure that the signal is being handled correctly and that it actually leads to the server shutting down.
Double-check the Deno documentation for the version you are using to see if there have been any changes to the API that might affect how the server should be shut down.

Please note that this advice is not based on the Deno documentation, and I recommend double-checking the accuracy of the advice given. If you continue to experience issues, you might want to consider reaching out to the Deno community for further assistance or looking for recent changes in the Deno release notes that might explain the behavior you're observing.

Helpful Links:
Deno 1.19 Release Notes
Deno 1.20 Release Notes
Deno 1.34: deno compile supports npm packages
Deno 1.37: Modern JavaScript in Jupyter Notebooks

Leokuma•12/5/23, 12:26 PM

I think you can open an issue and paste your test code there. There have been some questions about the new sanitizers, and some of them were actually bugs

bartlomieju•12/5/23, 12:32 PM

CC @mmastrac

mmastrac•12/5/23, 3:44 PM

It looks like you need to get the return value from Deno.serve and . The combination of fetch + serve requires this as there's a cached connection running in the background and we need to wait for it all to get torn down after the abort controller fires.

mmastrac•12/5/23, 3:45 PM

You can also use the asyncDispose functionality via

using

using

, which IIRC is which should trigger and wait on shutdown when the block ends

Mmmastrac It looks like you need to get the return value from Deno.serve and `await server...

SteveOP•12/6/23, 8:52 AM

Thank you, the test is now green again. I also tried calling before , as you stated first calling abort then finished fixed the problem. I'm not familiar with using yet, will try this out later.

SteveOP•12/6/23, 8:55 AM

For sake of completeness here is the commit with the working solution: https://github.com/sgohlke/deno-web/commit/dfffefa8de52bc9ab00933d133cf784f1bef3037

GitHub

Fix server test · sgohlke/deno-web@dfffefa