test(synctest): add synctest to syncutil test

[akrem-chabchoub]

Checklist

• I have read the coding guide.
• My change requires a documentation update, and I have done it.
• I have added tests to cover my changes.
• I have filled out the description and linked the related issues.

Description

Add synctest package in syncutil tests.

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

#5232

Screenshots (if appropriate):

[Copilot]

Pull Request Overview

This PR updates the test suite for the syncutil package to use Go's testing/synctest package for more deterministic testing of time-dependent synchronization code. The changes narrow CI test scope to focus on the syncutil package.

• Wraps the TestWaitWithTimeout test logic in synctest.Test() to enable deterministic time simulation
• Restricts CI test targets to only run tests in ./pkg/util/syncutil instead of all packages

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File	Description
pkg/util/syncutil/syncutil_test.go	Adds testing/synctest import and wraps test logic in synctest.Test() for deterministic time-based testing
Makefile	Narrows test-ci and test-ci-race targets to only test the syncutil package

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The test-ci target now only tests ./pkg/util/syncutil instead of ./.... This dramatically reduces CI test coverage to a single package. This appears to be a debugging/temporary change rather than an intentional scope reduction. If this is intentional, the change should be documented or the target renamed to reflect its limited scope (e.g., test-ci-syncutil).

[Copilot]

The test-ci target now only tests ./pkg/util/syncutil instead of ./.... This dramatically reduces CI test coverage to a single package. This appears to be a debugging/temporary change rather than an intentional scope reduction. If this is intentional, the change should be documented or the target renamed to reflect its limited scope (e.g., test-ci-syncutil).

[Copilot]

The test-ci-race target now only tests ./pkg/util/syncutil instead of ./.... This dramatically reduces CI test coverage to a single package. This appears to be a debugging/temporary change rather than an intentional scope reduction. If this is intentional, the change should be documented or the target renamed to reflect its limited scope (e.g., test-ci-race-syncutil).

[Copilot]

The test-ci-race target now only tests ./pkg/util/syncutil instead of ./.... This dramatically reduces CI test coverage to a single package. This appears to be a debugging/temporary change rather than an intentional scope reduction. If this is intentional, the change should be documented or the target renamed to reflect its limited scope (e.g., test-ci-race-syncutil).

[acud]

please revert the Makefile changes, otherwise LGTM