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

connectivity: Support multiple conditions #2732

Merged
merged 1 commit into from
Jul 30, 2024
Merged

Conversation

michi-covalent
Copy link
Contributor

Save all the condition functions from WithCondition() instead on only remembering the one from the latest invocation. Run the test if and only if it satisfies all the conditions.

Fixes: #2713

Save all the condition functions from WithCondition() instead on only
remembering the one from the latest invocation. Run the test if and only
if it satisfies all the conditions.

Fixes: #2713

Signed-off-by: Michi Mutsuzaki <[email protected]>
@michi-covalent michi-covalent marked this pull request as ready for review July 30, 2024 04:25
@michi-covalent michi-covalent requested a review from a team as a code owner July 30, 2024 04:25
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jul 30, 2024
@michi-covalent michi-covalent merged commit 1c5e05f into main Jul 30, 2024
13 checks passed
@michi-covalent michi-covalent deleted the pr/michi/conditions branch July 30, 2024 04:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

connectivity: check for multiple invocations of WithCondition
2 participants