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

[try_put_and_wait] Test coverage should be significantly extended #1481

Open
kboyarinov opened this issue Aug 15, 2024 · 0 comments
Open

[try_put_and_wait] Test coverage should be significantly extended #1481

kboyarinov opened this issue Aug 15, 2024 · 0 comments
Assignees
@kboyarinov
Labels
tests

Comments

@kboyarinov
Copy link
Contributor 

          I see that all of the tests are ultimately completely serial. The function_node is serial but also the flow graph executes in an arena with a single slot. I suppose this is done to make the results deterministic since most of the tests are whitebox tests that know the exact processing order. Is there risk that serial execution could hide some bugs caused by concurrency?

Originally posted by @vossmjp in #1412 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kboyarinov kboyarinov

Labels
tests
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@omalyshe @kboyarinov