Added unit tests for status and cronfederatedhpa helper utilities

[anujagrawal699]

Description:
This PR introduces comprehensive unit tests for status and cronfederatedhpa helper utility functions in the pkg/util/helper. These additions aim to increase code reliability, maintainability, and test coverage of the helper utility package.

Additions:

1. pkg/util/helper/cronfederatedhpa_test.go
2. pkg/util/helper/status_test.go

Test Coverage:

1. pkg/util/helper/cronfederatedhpa_test.go : 0% to 100%
2. pkg/util/helper/status_test.go : 0% to 100%

What type of PR is this?
/kind feature

Which issue(s) this PR fixes:
Fixes a part of #5470

Does this PR introduce a user-facing change?:

NONE

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.61%. Comparing base (057cf86) to head (310e4f9).
Report is 67 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #5768      +/-   ##
==========================================
+ Coverage   42.31%   45.61%   +3.30%     
==========================================
  Files         655      658       +3     
  Lines       55756    53921    -1835     
==========================================
+ Hits        23591    24598    +1007     
+ Misses      30651    27724    -2927     
- Partials     1514     1599      +85     

Flag	Coverage Δ
unittests	45.61% <ø> (+3.30%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

---

🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests

[zhzhuang-zju]

others LGTM

[zhzhuang-zju]

thanks, others LGTM

[zhzhuang-zju]

Suggested change

if tt.expectedError == "" {

Lines L322-L327 have already handled the case where expectedError is not empty, so there is no need to add a condition check for tt.expectedError == "".

[zhzhuang-zju]

/lgtm
cc @XiShanYongYe-Chang

[XiShanYongYe-Chang]

There's a code bad taste involved here. The number 3 is a devil's number in the test case, and if we can describe it in one word, we'll be more comfortable naming the test case. For example, we can change the name of the test case to successful history limit is set to bigger than xxx.

If you wish, you can resubmit a pr to modify it.

[anujagrawal699]

Yeah that makes sense, 3 could be defined as a constant at the package level, which would make it easier to change the default value in the future if needed.

[XiShanYongYe-Chang]

/approve

[karmada-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: XiShanYongYe-Chang

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• pkg/util/OWNERS [XiShanYongYe-Chang]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment