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

feat: add receive_events_non_blocking #65

Closed
wants to merge 1 commit into from

Conversation

thalesmg
Copy link
Contributor

@thalesmg thalesmg commented Jun 3, 2024

Sometimes, we want just to collect any events between two points in time, without waiting for any particular number of events or timeout.

Sometimes, we want just to collect any events between two points in time, without waiting
for any particular number of events or timeout.
@ieQu1
Copy link
Contributor

ieQu1 commented Jul 4, 2024

Philosophy behind snabbkaffe is to replace all time relationships in tests with causality relationships. Since causality relationships should be knowable, this should make tests more deterministic. "Collecting all events so far" API is obviously nondeterministic and non-causal. Therefore it should not be there.

P.S. There are a lot of old APIs that violate this rule as well (any API that accepts a timeout...), and they should be gradually removed as well. Ideally, the only API that can behave nondeterministically should be the high level timetrap of check_trace macro.

@ieQu1 ieQu1 closed this Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants