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

chore: Remove SequentialThreadTaskSchedulerTests #18391

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mxtartaglia-sl
Copy link
Contributor

Description:
This PR removes SequentialThreadTaskSchedulerTests given that it does not perform any meaningful assertion.
All behavior is tested in SequentialTaskSchedulerTests.
Related issue(s):

Fixes #

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@mxtartaglia-sl mxtartaglia-sl added this to the v0.61 milestone Mar 18, 2025
@mxtartaglia-sl mxtartaglia-sl self-assigned this Mar 18, 2025
@mxtartaglia-sl mxtartaglia-sl force-pushed the chore-remove-sequentialthreadschedulertest branch 2 times, most recently from 2f1a5f6 to 0d74902 Compare March 18, 2025 22:27
…skSchedulerTests to have a tier down method

Signed-off-by: mxtartaglia <[email protected]>
@mxtartaglia-sl mxtartaglia-sl force-pushed the chore-remove-sequentialthreadschedulertest branch from 869e28b to dec12d9 Compare March 26, 2025 19:33
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.

1 participant