search

denoland / deno

A modern runtime for JavaScript and TypeScript.
https://deno.com
MIT License
94.66k stars 5.25k forks source link

kv_queue_undelivered_test takes over 100 seconds to complete #20041

Open bartlomieju opened 1 year ago

bartlomieju commented 1 year ago 
test integration::node_unit_tests::process_test ... ok
test integration::node_unit_tests::v8_test ... ok
test integration::node_unit_tests::zlib_test ... ok
test integration::js_unit_tests::serve_test ... ok
test integration::node_unit_tests::worker_threads_test ... ok
test integration::node_unit_tests::scrypt_test ... ok
test integration::node_unit_tests::crypto_key_test ... ok
test integration::js_unit_tests::kv_queue_undelivered_test has been running for over 60 seconds
...

This is probably because of this line: https://github.com/denoland/deno/blob/433ecc9047c1701a6b384fc94593c1f41f9f6bf6/cli/tests/unit/kv_queue_undelivered_test.ts#L45

Most of the unit tests are snappy, so it would be good to shorten the run time of this test.

igorzi commented 1 year ago

Once we expose the retry backoff knob in the enqueue API, we should be able to speed up this test.