[YUNIKORN-2835] Refactor switch statement in UpdateApplication

[blueBlue0102]

What is this PR for?

The application state handling code in the UpdateApplication method of AsyncRMCallback struct (in scheduler_callback.go) could be improved.

The current implementation uses a switch statement with a default case that actually handles two specific states, which may not align with the typical expectations for a default case.

What type of PR is it?

• - Refactoring

What is the Jira issue?

YUNIKORN-2835

How should this be tested?

This has already passed the CI tests(unit test) on my forked repo: Link

[chenyulin0719]

LGTM

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.89%. Comparing base (5bb3890) to head (ab5a184).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #901      +/-   ##
==========================================
+ Coverage   67.87%   67.89%   +0.02%     
==========================================
  Files          70       70              
  Lines        7489     7488       -1     
==========================================
+ Hits         5083     5084       +1     
+ Misses       2194     2192       -2     
  Partials      212      212              

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

[wilfred-s]

LGTM