chore(tests): updating the test strategy to be more parrallel

[Jacobbrewer1]

Describe your changes

This pull request primarily focuses on improving test organization and parallelization across multiple files. It introduces subtests for better readability and structure, removes redundant helper methods, and ensures tests run in parallel where applicable.

Test organization improvements:

• k8s/resource_test.go: Consolidated multiple test functions into subtests using t.Run for better organization and readability. Updated test names to reflect specific scenarios, such as "create configmap error" and "update secret error". [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]

• slices/set_test.go: Reorganized tests into subtests using t.Run for operations like "Difference", "Items", "Each", and "Union". This enhances test clarity and structure. [1] [2] [3] [4] [5]

Parallelization enhancements:

• cache/fixed_hash_bucket_test.go: Added t.Parallel() to enable parallel execution of the Test_FixedHashBucket test.
• logging/logger_test.go: Introduced t.Parallel() for the TestNewLogger and its subtests, ensuring tests run concurrently. [1] [2] [3]
• slices/set_test.go: Enabled parallel execution for all subtests within Test_Set. [1] [2] [3] [4] [5]

Removal of redundant code:

• logging/config_test.go: Removed the setEnv helper method and directly used t.Setenv for setting environment variables in tests. This simplifies the code and reduces unnecessary abstraction. [1] [2]

[Copilot]

Pull Request Overview

This PR restructures existing tests into subtests, enables parallel execution where appropriate, and removes redundant helper code to streamline the test suite.

• Converts standalone test functions into t.Run subtests for clearer organization
• Adds t.Parallel() calls to improve test parallelism
• Removes the custom setEnv helper in favor of using t.Setenv directly

Reviewed Changes

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

Show a summary per file

File	Description
slices/set_test.go	Consolidated multiple Test_Set_* functions into subtests under Test_Set, added t.Parallel()
logging/logger_test.go	Wrapped TestNewLogger cases in subtests with t.Parallel(), added parallelism to TestDuplicateKey
logging/config_test.go	Removed setEnv helper and replaced calls with direct t.Setenv
k8s/resource_test.go	Converted several top-level TestUpsertResource_* functions into subtests with consistent naming and parallelism
health/result_test.go	Added an explicit fatal check after length assertion in compareResult
cache/fixed_hash_bucket_test.go	Enabled parallel execution for Test_FixedHashBucket

Comments suppressed due to low confidence (1)

k8s/resource_test.go:54

• [nitpick] Consider adding t.Parallel() at the start of TestUpsertResource so that its subtests can run in parallel with other tests.

		err := UpsertResource(ctx, kubeClient, secret)

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.