Add synctest in storage incentives

[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 in storage incentives.
Remove parallel execution from tests (synctest).

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

#5232

Screenshots (if appropriate):

[Copilot]

Pull Request Overview

This PR migrates two test functions (TestSocMine and TestClose/TestPhaseCancel) in the storage incentives package to use Go 1.23's testing/synctest package for better control over concurrent test execution. Additionally, the Makefile's CI test targets have been temporarily narrowed to only run tests in the pkg/storageincentives package.

• Wraps test logic in synctest.Test() and removes t.Parallel() calls
• Adds testing/synctest import to affected test files
• Restricts CI test execution to ./pkg/storageincentives instead of ./...

Reviewed Changes

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

File	Description
pkg/storageincentives/soc_mine_test.go	Wraps TestSocMine with synctest.Test and removes t.Parallel()
pkg/storageincentives/events_test.go	Wraps TestClose and TestPhaseCancel with synctest.Test and removes t.Parallel()
Makefile	Restricts test-ci and test-ci-race targets to only test pkg/storageincentives

Comments suppressed due to low confidence (1)

pkg/storageincentives/soc_mine_test.go:27

• Corrected spelling of 'incestives' to 'incentives'.

// TestSocMine dumps a sample made out SOCs to upload for storage incestives

---

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

[Copilot]

The test-ci and test-ci-race targets have been changed from testing all packages (./...) to only testing ./pkg/storageincentives. This significantly reduces test coverage in CI and means tests in other packages (e.g., pkg/accounting, pkg/api, pkg/storer, etc.) will not run during CI. If this is intentional for testing the synctest migration, consider adding a comment explaining this is temporary, or restore the original ./... pattern if all migrations are complete.

[Copilot]

The test-ci-race target has been changed from testing all packages (./...) to only testing ./pkg/storageincentives. This significantly reduces test coverage in CI and means race detection will not run for tests in other packages. If this is intentional for testing the synctest migration, consider adding a comment explaining this is temporary, or restore the original ./... pattern if all migrations are complete.