Test: replace context.TODO and context.Background to t.Context in uni…

[richie-king]

Proposed changes

Problem: Unit tests currently use context.TODO() and context.Background(), which can potentially lead to goroutine leaks and resource cleanup issues because these contexts are not automatically canceled when a test completes.

Solution: Replaced usage of context.TODO() and context.Background() with t.Context() in unit tests. This ensures that contexts are automatically canceled when tests complete and aligns the test suite with modern Go 1.24+ testing practices.

Testing: Ran all unit tests to ensure they pass with the new context handling.
Closes #3157

Checklist

Before creating a PR, run through this checklist and mark each as complete.

• I have read the contributing guidelines.
• I have added tests that prove my fix is effective or that my feature works
• I have checked that all unit tests pass after adding my changes
• I have updated necessary documentation
• I have rebased my branch onto main
• I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes

If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

Refactored unit tests to use t.Context() instead of context.TODO() or context.Background() to prevent potential goroutine leaks.

[nginx-bot]

Hi @richie-king! Welcome to the project! 🎉

Thanks for opening this pull request!
Be sure to check out our Contributing Guidelines while you wait for someone on the team to review this.

[github-actions]

✅ All required contributors have signed the F5 CLA for this PR. Thank you!
_{Posted by the CLA Assistant Lite bot.}

[richie-king]

🎉 Thank you for your contribution! It appears you have not yet signed the F5 Contributor License Agreement (CLA), which is required for your changes to be incorporated into an F5 Open Source Software (OSS) project. Please kindly read the F5 CLA and reply on a new comment with the following text to agree:

I have hereby read the F5 CLA and agree to its terms

You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

I have hereby read the F5 CLA and agree to its terms

[richie-king]

recheck

[richie-king]

I have hereby read the F5 CLA and agree to its terms

[richie-king]

recheck

[bjee19]

Hey @richie-king, thanks for this contribution! The changes are looking pretty good, however it looks like the PR is failing a pipeline regarding some lint issues, do you think you could address those?

[sjberman]

I'm surprised there are only 17 files that needed updating.

[richie-king]

Hey @richie-king, thanks for this contribution! The changes are looking pretty good, however it looks like the PR is failing a pipeline regarding some lint issues, do you think you could address those?

Sure, I will. I feel good about making CI pass

[richie-king]

I'm surprised there are only 17 files that needed updating.

I've checked all the test files. All the unit tests are included, except for the BDD tests, which do not use the T object.

[sjberman]

Some pipeline failures we're seeing on forks, not related to the PR. I'm looking into it.

[bjee19]

We're still in progress on fixing our issue related to the pipeline issues. Once that gets resolved this should be close to merging, thanks for your patience.

[sjberman]

@richie-king I think we've fixed the pipeline issue at this point. If you wouldn't mind rebasing and fixing the conflicts, we'll get this merged soon!

[richie-king]

@richie-king I think we've fixed the pipeline issue at this point. If you wouldn't mind rebasing and fixing the conflicts, we'll get this merged soon!

sure, I will make it happen ASAP