🛠️ Convert helper ginkgo tests to regular unit tests

[Shweta-Deshpande]

Rollback Plan

N/A

Changes to Security Controls

No

Description

The helpers_tests.go file does not need a ginkgo and gomega framework to execute the tests since they are simple unit tests, testing internal helper functions. This PR will modify them to be regular unit tests which will run in parallel and table driven format. Replaced ginkgo's Expect.To()with testify/assert.

Usage Example

N/A

References

Community Note

• Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.
• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request.
• If you are interested in working on this issue or have submitted a pull request, please leave a comment.

[arybolovlev]

Thanks for making this change!

I think it would make sense to update it right after #641 is merged, since that PR changes the name of the function covered by one of these tests.

Another point: these tests are currently not being executed. You will need to add a new make target -- my suggestion would be test-unit -- and also update the corresponding GitHub Action to ensure this new target runs when needed.

[Shweta-Deshpande]

Thanks for making this change!

I think it would make sense to update it right after #641 is merged, since that PR changes the name of the function covered by one of these tests.

Another point: these tests are currently not being executed. You will need to add a new make target -- my suggestion would be test-unit -- and also update the corresponding GitHub Action to ensure this new target runs when needed.

Done

[arybolovlev]

A tiny change, a giant leap for this project. 👍🏻