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

Add direct tests for the generic event loop #1745

Open
camshaft opened this issue May 3, 2023 · 0 comments
Open

Add direct tests for the generic event loop #1745

camshaft opened this issue May 3, 2023 · 0 comments

Comments

@camshaft
Copy link
Contributor

camshaft commented May 3, 2023

Problem:

In #1740, we added a generic event loop that replaces the existing ones that have been copy/pasted for each IO provider. This new code is indirectly tested through the io/testing IO provider and all of the associated integration tests.

Solution:

It would be nice to increase coverage by adding some direct tests to the code in order to state all of the properties that it needs to hold to be correct. This should include at least some unit tests to show the event loop waking up and emitting the correct events to the publisher. Ideally, it would also include a single iteration bolero/kani test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant