[YUNIKORN-2241] Refactor group/user limit e2e test

[pegasas]

What is this PR for?

refactor hard code partition into pre-defined partition const due to its usage on checkusage() to make style more alignable.

other constants like root would be hard to aligned.

What type of PR is it?

• - Bug Fix
• - Improvement
• - Feature
• - Documentation
• - Hot Fix
• - Refactoring

Todos

• - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/YUNIKORN-2241

How should this be tested?

Screenshots (if appropriate)

Questions:

• - The licenses files need update.
• - There is breaking changes for older versions.
• - It needs documentation.

[wilfred-s]

other constants like root would be hard to aligned.

The root queue is something that is fixed and defined in the same constants file: constants.RootQueue.
Same replacement can be made for that one.

[pegasas]

other constants like root would be hard to aligned.

The root queue is something that is fixed and defined in the same constants file: constants.RootQueue. Same replacement can be made for that one.

Hi, @wilfred-s ,
It is a good suggestion.
I make another version of this.
feel free to review when anytime you are available.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2738571) 69.43% compared to head (82ea26f) 69.46%.
Report is 2 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #745      +/-   ##
==========================================
+ Coverage   69.43%   69.46%   +0.02%     
==========================================
  Files          50       50              
  Lines        7993     7993              
==========================================
+ Hits         5550     5552       +2     
+ Misses       2254     2252       -2     
  Partials      189      189              

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

[pbacsko]

@pegasas what is required for this PR? Do you have pending changes?

[pegasas]

@pegasas what is required for this PR? Do you have pending changes?

no, pending for merging now.

[pbacsko]

+1